GitHub PullRequestReviewCommentEvent at 2014-12-01 16:47:21
Repository: navigator-connect
Actor: jakeleichtling
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/comments/21100982","id":21100982,"diff_hunk":"@@ -142,6 +142,34 @@\n           without-<code><a>navigator.connect</a></code> implementations.\n         </p>\n       </section>\n+      <section id='extension'>\n+        <h2>\n+          Extending the platform\n+        </h2>\n+        <p>\n+          <em>This use case is unlikely to drive aspects of the proposed API. We will need detailed\n+          analysis of how to make this use safe for the web before recommending it.</em>\n+        </p>\n+        <p>\n+          Browser vendors would like to have a way to experiment with APIs that access new platform\n+          features. Dimitri Glazkov has a <a href=\"https://github.com/dglazkov/tubes\">proposal</a>\n+          to use a <code><a>navigator.connect</a></code>-like API to \"drill through the bedrock\" by\n+          connecting to a system-provided endpoint. The idea is that, for proposed APIs that can\n+          afford <code>postMessage</code> overhead on every call, the browser would define that\n+          <code>navigator.connect('https://browser.example.com/<var>raiQu6up</var>/theNewApiName')</code>","path":"use-cases.html","position":18,"original_position":18,"commit_id":"2bbac6c888fceef0a8731bc5a10769fbf2507d38","original_commit_id":"2bbac6c888fceef0a8731bc5a10769fbf2507d38","user":{"login":"jakeleichtling","id":448920,"avatar_url":"https://avatars.githubusercontent.com/u/448920?v=3","gravatar_id":"","url":"https://api.github.com/users/jakeleichtling","html_url":"https://github.com/jakeleichtling","followers_url":"https://api.github.com/users/jakeleichtling/followers","following_url":"https://api.github.com/users/jakeleichtling/following{/other_user}","gists_url":"https://api.github.com/users/jakeleichtling/gists{/gist_id}","starred_url":"https://api.github.com/users/jakeleichtling/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jakeleichtling/subscriptions","organizations_url":"https://api.github.com/users/jakeleichtling/orgs","repos_url":"https://api.github.com/users/jakeleichtling/repos","events_url":"https://api.github.com/users/jakeleichtling/events{/privacy}","received_events_url":"https://api.github.com/users/jakeleichtling/received_events","type":"User","site_admin":false},"body":"As a small side note within the larger context, I think the rotating value provides a nice sample idea that emphasizes the experimental nature of browser APIs exposed via n.c. It underlines that proprietary APIs are not the motivating factor here. My vote would be to keep it in, although I would be alright without it as well.","created_at":"2014-12-01T16:47:21Z","updated_at":"2014-12-01T16:47:21Z","html_url":"https://github.com/jyasskin/navigator-connect/pull/1#discussion_r21100982","pull_request_url":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/1","_links":{"self":{"href":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/comments/21100982"},"html":{"href":"https://github.com/jyasskin/navigator-connect/pull/1#discussion_r21100982"},"pull_request":{"href":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/1"}}},"pull_request":{"url":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/1","id":25219124,"html_url":"https://github.com/jyasskin/navigator-connect/pull/1","diff_url":"https://github.com/jyasskin/navigator-connect/pull/1.diff","patch_url":"https://github.com/jyasskin/navigator-connect/pull/1.patch","issue_url":"https://api.github.com/repos/jyasskin/navigator-connect/issues/1","number":1,"state":"open","locked":false,"title":"Add a description of the \"Tubes\" use of navigator.connect.","user":{"login":"jyasskin","id":83420,"avatar_url":"https://avatars.githubusercontent.com/u/83420?v=3","gravatar_id":"","url":"https://api.github.com/users/jyasskin","html_url":"https://github.com/jyasskin","followers_url":"https://api.github.com/users/jyasskin/followers","following_url":"https://api.github.com/users/jyasskin/following{/other_user}","gists_url":"https://api.github.com/users/jyasskin/gists{/gist_id}","starred_url":"https://api.github.com/users/jyasskin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jyasskin/subscriptions","organizations_url":"https://api.github.com/users/jyasskin/orgs","repos_url":"https://api.github.com/users/jyasskin/repos","events_url":"https://api.github.com/users/jyasskin/events{/privacy}","received_events_url":"https://api.github.com/users/jyasskin/received_events","type":"User","site_admin":false},"body":"@slightlyoff @dglazkov @jakeleichtling, I'd like to include this use case so that folks don't feel like we're trying to sneak it through. Can you help me describe it correctly?","created_at":"2014-11-29T04:47:51Z","updated_at":"2014-12-01T16:47:21Z","closed_at":null,"merged_at":null,"merge_commit_sha":"b4734c374d37a600b5951747c6291a96cc17a4f2","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/1/commits","review_comments_url":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/1/comments","review_comment_url":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/comments/{number}","comments_url":"https://api.github.com/repos/jyasskin/navigator-connect/issues/1/comments","statuses_url":"https://api.github.com/repos/jyasskin/navigator-connect/statuses/2bbac6c888fceef0a8731bc5a10769fbf2507d38","head":{"label":"jyasskin:through-bedrock","ref":"through-bedrock","sha":"2bbac6c888fceef0a8731bc5a10769fbf2507d38","user":{"login":"jyasskin","id":83420,"avatar_url":"https://avatars.githubusercontent.com/u/83420?v=3","gravatar_id":"","url":"https://api.github.com/users/jyasskin","html_url":"https://github.com/jyasskin","followers_url":"https://api.github.com/users/jyasskin/followers","following_url":"https://api.github.com/users/jyasskin/following{/other_user}","gists_url":"https://api.github.com/users/jyasskin/gists{/gist_id}","starred_url":"https://api.github.com/users/jyasskin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jyasskin/subscriptions","organizations_url":"https://api.github.com/users/jyasskin/orgs","repos_url":"https://api.github.com/users/jyasskin/repos","events_url":"https://api.github.com/users/jyasskin/events{/privacy}","received_events_url":"https://api.github.com/users/jyasskin/received_events","type":"User","site_admin":false},"repo":{"id":27152686,"name":"navigator-connect","full_name":"jyasskin/navigator-connect","owner":{"login":"jyasskin","id":83420,"avatar_url":"https://avatars.githubusercontent.com/u/83420?v=3","gravatar_id":"","url":"https://api.github.com/users/jyasskin","html_url":"https://github.com/jyasskin","followers_url":"https://api.github.com/users/jyasskin/followers","following_url":"https://api.github.com/users/jyasskin/following{/other_user}","gists_url":"https://api.github.com/users/jyasskin/gists{/gist_id}","starred_url":"https://api.github.com/users/jyasskin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jyasskin/subscriptions","organizations_url":"https://api.github.com/users/jyasskin/orgs","repos_url":"https://api.github.com/users/jyasskin/repos","events_url":"https://api.github.com/users/jyasskin/events{/privacy}","received_events_url":"https://api.github.com/users/jyasskin/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/jyasskin/navigator-connect","description":"","fork":true,"url":"https://api.github.com/repos/jyasskin/navigator-connect","forks_url":"https://api.github.com/repos/jyasskin/navigator-connect/forks","keys_url":"https://api.github.com/repos/jyasskin/navigator-connect/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jyasskin/navigator-connect/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jyasskin/navigator-connect/teams","hooks_url":"https://api.github.com/repos/jyasskin/navigator-connect/hooks","issue_events_url":"https://api.github.com/repos/jyasskin/navigator-connect/issues/events{/number}","events_url":"https://api.github.com/repos/jyasskin/navigator-connect/events","assignees_url":"https://api.github.com/repos/jyasskin/navigator-connect/assignees{/user}","branches_url":"https://api.github.com/repos/jyasskin/navigator-connect/branches{/branch}","tags_url":"https://api.github.com/repos/jyasskin/navigator-connect/tags","blobs_url":"https://api.github.com/repos/jyasskin/navigator-connect/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jyasskin/navigator-connect/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jyasskin/navigator-connect/git/refs{/sha}","trees_url":"https://api.github.com/repos/jyasskin/navigator-connect/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jyasskin/navigator-connect/statuses/{sha}","languages_url":"https://api.github.com/repos/jyasskin/navigator-connect/languages","stargazers_url":"https://api.github.com/repos/jyasskin/navigator-connect/stargazers","contributors_url":"https://api.github.com/repos/jyasskin/navigator-connect/contributors","subscribers_url":"https://api.github.com/repos/jyasskin/navigator-connect/subscribers","subscription_url":"https://api.github.com/repos/jyasskin/navigator-connect/subscription","commits_url":"https://api.github.com/repos/jyasskin/navigator-connect/commits{/sha}","git_commits_url":"https://api.github.com/repos/jyasskin/navigator-connect/git/commits{/sha}","comments_url":"https://api.github.com/repos/jyasskin/navigator-connect/comments{/number}","issue_comment_url":"https://api.github.com/repos/jyasskin/navigator-connect/issues/comments/{number}","contents_url":"https://api.github.com/repos/jyasskin/navigator-connect/contents/{+path}","compare_url":"https://api.github.com/repos/jyasskin/navigator-connect/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jyasskin/navigator-connect/merges","archive_url":"https://api.github.com/repos/jyasskin/navigator-connect/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jyasskin/navigator-connect/downloads","issues_url":"https://api.github.com/repos/jyasskin/navigator-connect/issues{/number}","pulls_url":"https://api.github.com/repos/jyasskin/navigator-connect/pulls{/number}","milestones_url":"https://api.github.com/repos/jyasskin/navigator-connect/milestones{/number}","notifications_url":"https://api.github.com/repos/jyasskin/navigator-connect/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jyasskin/navigator-connect/labels{/name}","releases_url":"https://api.github.com/repos/jyasskin/navigator-connect/releases{/id}","created_at":"2014-11-26T00:29:59Z","updated_at":"2014-11-26T00:29:59Z","pushed_at":"2014-11-29T04:44:44Z","git_url":"git://github.com/jyasskin/navigator-connect.git","ssh_url":"git@github.com:jyasskin/navigator-connect.git","clone_url":"https://github.com/jyasskin/navigator-connect.git","svn_url":"https://github.com/jyasskin/navigator-connect","homepage":null,"size":166,"stargazers_count":0,"watchers_count":0,"language":"JavaScript","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":0,"mirror_url":null,"open_issues_count":1,"forks":0,"open_issues":1,"watchers":0,"default_branch":"gh-pages"}},"base":{"label":"jyasskin:use-cases","ref":"use-cases","sha":"84efa0a66a9966712053d59e9dea8f07bf6a5c33","user":{"login":"jyasskin","id":83420,"avatar_url":"https://avatars.githubusercontent.com/u/83420?v=3","gravatar_id":"","url":"https://api.github.com/users/jyasskin","html_url":"https://github.com/jyasskin","followers_url":"https://api.github.com/users/jyasskin/followers","following_url":"https://api.github.com/users/jyasskin/following{/other_user}","gists_url":"https://api.github.com/users/jyasskin/gists{/gist_id}","starred_url":"https://api.github.com/users/jyasskin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jyasskin/subscriptions","organizations_url":"https://api.github.com/users/jyasskin/orgs","repos_url":"https://api.github.com/users/jyasskin/repos","events_url":"https://api.github.com/users/jyasskin/events{/privacy}","received_events_url":"https://api.github.com/users/jyasskin/received_events","type":"User","site_admin":false},"repo":{"id":27152686,"name":"navigator-connect","full_name":"jyasskin/navigator-connect","owner":{"login":"jyasskin","id":83420,"avatar_url":"https://avatars.githubusercontent.com/u/83420?v=3","gravatar_id":"","url":"https://api.github.com/users/jyasskin","html_url":"https://github.com/jyasskin","followers_url":"https://api.github.com/users/jyasskin/followers","following_url":"https://api.github.com/users/jyasskin/following{/other_user}","gists_url":"https://api.github.com/users/jyasskin/gists{/gist_id}","starred_url":"https://api.github.com/users/jyasskin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jyasskin/subscriptions","organizations_url":"https://api.github.com/users/jyasskin/orgs","repos_url":"https://api.github.com/users/jyasskin/repos","events_url":"https://api.github.com/users/jyasskin/events{/privacy}","received_events_url":"https://api.github.com/users/jyasskin/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/jyasskin/navigator-connect","description":"","fork":true,"url":"https://api.github.com/repos/jyasskin/navigator-connect","forks_url":"https://api.github.com/repos/jyasskin/navigator-connect/forks","keys_url":"https://api.github.com/repos/jyasskin/navigator-connect/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jyasskin/navigator-connect/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jyasskin/navigator-connect/teams","hooks_url":"https://api.github.com/repos/jyasskin/navigator-connect/hooks","issue_events_url":"https://api.github.com/repos/jyasskin/navigator-connect/issues/events{/number}","events_url":"https://api.github.com/repos/jyasskin/navigator-connect/events","assignees_url":"https://api.github.com/repos/jyasskin/navigator-connect/assignees{/user}","branches_url":"https://api.github.com/repos/jyasskin/navigator-connect/branches{/branch}","tags_url":"https://api.github.com/repos/jyasskin/navigator-connect/tags","blobs_url":"https://api.github.com/repos/jyasskin/navigator-connect/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jyasskin/navigator-connect/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jyasskin/navigator-connect/git/refs{/sha}","trees_url":"https://api.github.com/repos/jyasskin/navigator-connect/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jyasskin/navigator-connect/statuses/{sha}","languages_url":"https://api.github.com/repos/jyasskin/navigator-connect/languages","stargazers_url":"https://api.github.com/repos/jyasskin/navigator-connect/stargazers","contributors_url":"https://api.github.com/repos/jyasskin/navigator-connect/contributors","subscribers_url":"https://api.github.com/repos/jyasskin/navigator-connect/subscribers","subscription_url":"https://api.github.com/repos/jyasskin/navigator-connect/subscription","commits_url":"https://api.github.com/repos/jyasskin/navigator-connect/commits{/sha}","git_commits_url":"https://api.github.com/repos/jyasskin/navigator-connect/git/commits{/sha}","comments_url":"https://api.github.com/repos/jyasskin/navigator-connect/comments{/number}","issue_comment_url":"https://api.github.com/repos/jyasskin/navigator-connect/issues/comments/{number}","contents_url":"https://api.github.com/repos/jyasskin/navigator-connect/contents/{+path}","compare_url":"https://api.github.com/repos/jyasskin/navigator-connect/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jyasskin/navigator-connect/merges","archive_url":"https://api.github.com/repos/jyasskin/navigator-connect/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jyasskin/navigator-connect/downloads","issues_url":"https://api.github.com/repos/jyasskin/navigator-connect/issues{/number}","pulls_url":"https://api.github.com/repos/jyasskin/navigator-connect/pulls{/number}","milestones_url":"https://api.github.com/repos/jyasskin/navigator-connect/milestones{/number}","notifications_url":"https://api.github.com/repos/jyasskin/navigator-connect/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jyasskin/navigator-connect/labels{/name}","releases_url":"https://api.github.com/repos/jyasskin/navigator-connect/releases{/id}","created_at":"2014-11-26T00:29:59Z","updated_at":"2014-11-26T00:29:59Z","pushed_at":"2014-11-29T04:44:44Z","git_url":"git://github.com/jyasskin/navigator-connect.git","ssh_url":"git@github.com:jyasskin/navigator-connect.git","clone_url":"https://github.com/jyasskin/navigator-connect.git","svn_url":"https://github.com/jyasskin/navigator-connect","homepage":null,"size":166,"stargazers_count":0,"watchers_count":0,"language":"JavaScript","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":0,"mirror_url":null,"open_issues_count":1,"forks":0,"open_issues":1,"watchers":0,"default_branch":"gh-pages"}},"_links":{"self":{"href":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/1"},"html":{"href":"https://github.com/jyasskin/navigator-connect/pull/1"},"issue":{"href":"https://api.github.com/repos/jyasskin/navigator-connect/issues/1"},"comments":{"href":"https://api.github.com/repos/jyasskin/navigator-connect/issues/1/comments"},"review_comments":{"href":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/1/comments"},"review_comment":{"href":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/jyasskin/navigator-connect/pulls/1/commits"},"statuses":{"href":"https://api.github.com/repos/jyasskin/navigator-connect/statuses/2bbac6c888fceef0a8731bc5a10769fbf2507d38"}}}}