{"action":"created","issue":{"url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/916","repository_url":"https://api.github.com/repos/slightlyoff/ServiceWorker","labels_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/916/labels{/name}","comments_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/916/comments","events_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/916/events","html_url":"https://github.com/slightlyoff/ServiceWorker/issues/916","id":160840193,"number":916,"title":"More clarity around waiting for an active worker to finish","user":{"login":"mattto","id":1782098,"avatar_url":"https://avatars.githubusercontent.com/u/1782098?v=3","gravatar_id":"","url":"https://api.github.com/users/mattto","html_url":"https://github.com/mattto","followers_url":"https://api.github.com/users/mattto/followers","following_url":"https://api.github.com/users/mattto/following{/other_user}","gists_url":"https://api.github.com/users/mattto/gists{/gist_id}","starred_url":"https://api.github.com/users/mattto/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattto/subscriptions","organizations_url":"https://api.github.com/users/mattto/orgs","repos_url":"https://api.github.com/users/mattto/repos","events_url":"https://api.github.com/users/mattto/events{/privacy}","received_events_url":"https://api.github.com/users/mattto/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/slightlyoff/ServiceWorker/milestones/2","html_url":"https://github.com/slightlyoff/ServiceWorker/milestones/Version%201","labels_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/milestones/2/labels","id":746092,"number":2,"title":"Version 1","description":"","creator":{"login":"tobie","id":3392,"avatar_url":"https://avatars.githubusercontent.com/u/3392?v=3","gravatar_id":"","url":"https://api.github.com/users/tobie","html_url":"https://github.com/tobie","followers_url":"https://api.github.com/users/tobie/followers","following_url":"https://api.github.com/users/tobie/following{/other_user}","gists_url":"https://api.github.com/users/tobie/gists{/gist_id}","starred_url":"https://api.github.com/users/tobie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tobie/subscriptions","organizations_url":"https://api.github.com/users/tobie/orgs","repos_url":"https://api.github.com/users/tobie/repos","events_url":"https://api.github.com/users/tobie/events{/privacy}","received_events_url":"https://api.github.com/users/tobie/received_events","type":"User","site_admin":false},"open_issues":23,"closed_issues":149,"state":"open","created_at":"2014-08-07T10:50:36Z","updated_at":"2016-07-20T14:05:13Z","due_on":"2016-03-30T15:00:00Z","closed_at":null},"comments":22,"created_at":"2016-06-17T08:46:54Z","updated_at":"2016-07-20T14:06:31Z","closed_at":null,"body":"I'm working on a Chrome [bug](https://bugs.chromium.org/p/chromium/issues/detail?id=616331) where we're immediately terminating the incumbent worker during activation instead of waiting for it to finish its \"in-progress requests\" as the spec says. Since update occurs soon after page load, with skipWaiting() we run into this issue in the wild.\r\n\r\nI'm wondering though, what exactly is a \"request\". Is this referring to fetch event only or is it also intended to mean functional events?\r\n\r\nElsewhere the spec also mentions \"in-flight requests\" which are \"triggered by\" a worker, which seems a different thing that \"in-progress requests\" being handled by a worker. \r\n\r\nThe spec also doesn't really mention what to do with new incoming requests: do they go to the incumbent worker or the soon-to-be activating worker?\r\n\r\nMy current plan is to:\r\n- Wait for all events already dispatched to the incumbent worker to finish (subject to our usual timeout)\r\n- All new events are put on hold until the soon-to-be activating worker is activated\r\n\r\nHow does Firefox deal with this issue?"},"comment":{"url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/comments/233959545","html_url":"https://github.com/slightlyoff/ServiceWorker/issues/916#issuecomment-233959545","issue_url":"https://api.github.com/repos/slightlyoff/ServiceWorker/issues/916","id":233959545,"user":{"login":"wanderview","id":1256526,"avatar_url":"https://avatars.githubusercontent.com/u/1256526?v=3","gravatar_id":"","url":"https://api.github.com/users/wanderview","html_url":"https://github.com/wanderview","followers_url":"https://api.github.com/users/wanderview/followers","following_url":"https://api.github.com/users/wanderview/following{/other_user}","gists_url":"https://api.github.com/users/wanderview/gists{/gist_id}","starred_url":"https://api.github.com/users/wanderview/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wanderview/subscriptions","organizations_url":"https://api.github.com/users/wanderview/orgs","repos_url":"https://api.github.com/users/wanderview/repos","events_url":"https://api.github.com/users/wanderview/events{/privacy}","received_events_url":"https://api.github.com/users/wanderview/received_events","type":"User","site_admin":false},"created_at":"2016-07-20T14:06:31Z","updated_at":"2016-07-20T14:06:31Z","body":"@mattto When do you expect this change to ship in chrome?  I'd like to try to get the fix into FF around the same time frame.  Thanks!"}}
{"actor":{"display_login":"wanderview"}}