New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 40 users
Status: Fixed
Owner:
Closed: Nov 2014
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Launch-OWP
Launch-Accessibility: ----
Launch-Legal: ----
Launch-M-Approved: ----
Launch-M-Target: ----
Launch-Privacy: Yes
Launch-Security: Yes
Launch-Status: ----
Launch-Test: ----
Launch-UI: NA
Product-Review: ----



Sign in to add a comment
[OWP Launch Tracking] ServiceWorker
Project Member Reported by meh@chromium.org, Apr 17 2014 Back to list
Change description:
A ServiceWorker is a scriptable caching proxy that sits between a web page and the network.  It has two parts:

1. A shared worker-like thread that intercepts some/all resource requests for a given page that matches a developer-chosen pattern.
2. A programmable HTTP cache that can be updated atomically.

These APIs are complementary such that it is easy for the developer to deliver a specific resource from the cache given a specific request. However, being able to intercept the requests before the cache is involved gives the developer great freedom in delivering or even generating resources. Formerly known as NavigationController.

Links:
Spec: https://slightlyoff.github.io/ServiceWorker/spec/service_worker/
Intent to Implement: https://groups.google.com/a/chromium.org/d/msg/blink-dev/Du9lhfui1Mo/HxL_pS7Cl-AJ

Support in other browsers:
Internet Explorer: No public signals
Firefox: Implementation started https://bugzilla.mozilla.org/show_bug.cgi?id=898524
Safari: Interest expressed http://lists.w3.org/Archives/Public/public-webapps/2014JanMar/0397.html

*Make sure to fill in any labels with a -?, including all OSes this change
affects. Feel free to leave other labels at the defaults.

 
Comment 1 by meh@chromium.org, Apr 17 2014
Labels: Cr-Blink-Storage Cr-Blink-ServiceWorker
Comment 2 by meh@chromium.org, Apr 17 2014
Labels: Launch-Privacy-No Launch-Security-No
Cc: igrigo...@chromium.org
Labels: -M-37 M-38
In M37 planning last week we discussed the benefit of slipping this to M38. This was a combination of triage (we don't think we will have the fetch() method working in time for M37) and driving the platform forward (the alternative to fetch() is to use XHR, but we would like to make XHR simply unavailable in a Service Worker. fetch() ultimately makes it redundant.)
Cc: dominicc@chromium.org
Labels: Launch-Strings-NA Launch-UI-NA
Owner: kenjibaheux@chromium.org
Let me take care of the mundane tasks of the launch process.

No new user facing strings/UI for the MVP scope => N/A those bits.
Labels: -M-38 M-39
Labels: -M-39 -Pri-2 M-40 Pri-1
Status: Started
Cc: markus@chromium.org
Markus, the team has fixed all the privacy ship blockers. Would you mind flipping the privacy bit? Thanks!
Comment 9 by markus@chromium.org, Oct 21 2014
I think you have the wrong guy. Unfortunately, I don't know who you need to talk to or otherwise I would have forwarded your request.
Cc: -markus@chromium.org markusheintz@chromium.org
Sorry about that ;)
This time I have the right Markus in the cc line.

"Markus, the team has fixed all the privacy ship blockers. Would you mind flipping the privacy bit? Thanks!"
Project Member Comment 11 by bugdroid1@chromium.org, Oct 24 2014
The following revision refers to this bug:
  http://src.chromium.org/viewvc/blink?view=rev&rev=184340

------------------------------------------------------------------
r184340 | dominicc@chromium.org | 2014-10-24T09:42:40.147152Z

Changed paths:
   M http://src.chromium.org/viewvc/blink/trunk/Source/platform/RuntimeEnabledFeatures.in?r1=184340&r2=184339&pathrev=184340
   M http://src.chromium.org/viewvc/blink/trunk/LayoutTests/virtual/stable/webexposed/global-constructors-listing-expected.txt?r1=184340&r2=184339&pathrev=184340

Enable Service Workers, including the onfetch event, by default.

BUG= 364627 
TEST=virtual/stable/webexposed

Review URL: https://codereview.chromium.org/680473002
-----------------------------------------------------------------
Project Member Comment 12 by bugdroid1@chromium.org, Oct 25 2014
The following revision refers to this bug:
  http://src.chromium.org/viewvc/blink?view=rev&rev=184400

------------------------------------------------------------------
r184400 | horo@chromium.org | 2014-10-25T01:36:11.756988Z

Changed paths:
   M http://src.chromium.org/viewvc/blink/trunk/Source/platform/RuntimeEnabledFeatures.in?r1=184400&r2=184399&pathrev=184400
   M http://src.chromium.org/viewvc/blink/trunk/LayoutTests/virtual/stable/webexposed/global-constructors-listing-expected.txt?r1=184400&r2=184399&pathrev=184400

Revert of Enable Service Workers, including the onfetch event, by default. (patchset #1 id:1 of https://codereview.chromium.org/680473002/)

Reason for revert:
Directory restriction of ServiceWorker sope patch (https://codereview.chromium.org/672383003/) is reverted.

We should not enable ServiceWorker without this restriction for security reason.

I will revert this patch after these patches will land.
[1/3 blink] https://codereview.chromium.org/674133004/
 Update LayoutTests to be subject to the directory restrictions.
[2/3 chromium] https://codereview.chromium.org/674133004/
 Introduce the directory restriction in Chromium side.
[3/3 blink] https://codereview.chromium.org/681483004/
 Introduce the directory restriction in Blink side and update LayoutTest.


Original issue's description:
> Enable Service Workers, including the onfetch event, by default.
> 
> BUG= 364627 
> TEST=virtual/stable/webexposed
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184340

TBR=jochen@chromium.org,tkent@chromium.org,falken@chromium.org,dominicc@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG= 364627 

Review URL: https://codereview.chromium.org/678843002
-----------------------------------------------------------------
Project Member Comment 13 by bugdroid1@chromium.org, Oct 26 2014
The following revision refers to this bug:
  http://src.chromium.org/viewvc/blink?view=rev&rev=184410

------------------------------------------------------------------
r184410 | horo@chromium.org | 2014-10-26T01:28:19.896318Z

Changed paths:
   M http://src.chromium.org/viewvc/blink/trunk/LayoutTests/virtual/stable/webexposed/global-constructors-listing-expected.txt?r1=184410&r2=184409&pathrev=184410
   M http://src.chromium.org/viewvc/blink/trunk/Source/platform/RuntimeEnabledFeatures.in?r1=184410&r2=184409&pathrev=184410

Revert of Revert of Enable Service Workers, including the onfetch event, by default. (patchset #2 id:70001 of https://codereview.chromium.org/678843002/)

Reason for revert:
It is ok to enable SW.
We will fix this issue before launch.

Original issue's description:
> Revert of Enable Service Workers, including the onfetch event, by default. (patchset #1 id:1 of https://codereview.chromium.org/680473002/)
> 
> Reason for revert:
> Directory restriction of ServiceWorker sope patch (https://codereview.chromium.org/672383003/) is reverted.
> 
> We should not enable ServiceWorker without this restriction for security reason.
> 
> I will revert this patch after these patches will land.
> [1/3 blink] https://codereview.chromium.org/674133004/
>  Update LayoutTests to be subject to the directory restrictions.
> [2/3 chromium] https://codereview.chromium.org/674133004/
>  Introduce the directory restriction in Chromium side.
> [3/3 blink] https://codereview.chromium.org/681483004/
>  Introduce the directory restriction in Blink side and update LayoutTest.
> 
> 
> Original issue's description:
> > Enable Service Workers, including the onfetch event, by default.
> > 
> > BUG= 364627 
> > TEST=virtual/stable/webexposed
> > 
> > Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184340
> 
> TBR=jochen@chromium.org,tkent@chromium.org,falken@chromium.org,dominicc@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG= 364627 
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184400

TBR=jochen@chromium.org,tkent@chromium.org,falken@chromium.org,dominicc@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG= 364627 

Review URL: https://codereview.chromium.org/680643004
-----------------------------------------------------------------
Is blink susceptible to spec issues  #539  and  #548 ? They affect the registration process which is in the subset that Blink intends to ship soon.

(P. S. I'm offline till Monday)
Cc: jwd@chromium.org
+jwd for the security bit flip

+markusheintz: ping.
Comment 16 by jww@chromium.org, Nov 14 2014
Labels: -Launch-Security-No Launch-Security-Yes
Based on our latest discussions, Security is happy with the current plans and approach for SWs and approves launch. 
Comment 17 Deleted
Comment 18 Deleted
Labels: -Launch-Privacy-No Launch-Privacy-Yes
Status: Fixed
and we are done for M40 :)
Project Member Comment 21 by bugdroid1@chromium.org, Jul 6 2015
The following revision refers to this bug:
  http://src.chromium.org/viewvc/blink?view=rev&rev=198306

------------------------------------------------------------------
r198306 | jeremyarcher@google.com | 2015-07-06T08:20:19.230934Z

Changed paths:
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/Clients.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/ServiceWorkerMessageEvent.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/fetch/Request.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/web/FrameLoaderClientImpl.cpp?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/ServiceWorkerGlobalScope.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/NavigatorServiceWorker.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/public/web/WebRuntimeFeatures.h?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/web/ServiceWorkerGlobalScopeProxy.cpp?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/fetch/Body.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/fetch/Response.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/ExtendableEvent.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/Client.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/web/WebRuntimeFeatures.cpp?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/ServiceWorkerRegistration.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/fetch/Headers.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/RespondWithObserver.cpp?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/ServiceWorker.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/platform/RuntimeEnabledFeatures.in?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/FetchEvent.idl?r1=198306&r2=198305&pathrev=198306
   M http://src.chromium.org/viewvc/blink/trunk/Source/modules/serviceworkers/ServiceWorkerContainer.idl?r1=198306&r2=198305&pathrev=198306

Remove legacy ServiceWorker and ServiceWorkerOnFetch flags.

ServiceWorker and ServiceWorkerOnFetch were previously marked
as stable and so can safely be removed.

(No test added: (virtual/stable/)http/serviceworker/webexposed
and (virtual/stable/)webexposed still pass.)

BUG= 364627 

Review URL: https://codereview.chromium.org/1223793005
-----------------------------------------------------------------
Sign in to add a comment