Inform the client when the Service Worker rejects an install event

public/web/WebServiceWorkerContextClient.h

Index: public/web/WebServiceWorkerContextClient.h
diff --git a/public/web/WebServiceWorkerContextClient.h b/public/web/WebServiceWorkerContextClient.h
index 540cb061239c36da46c3e21c376eec706843c03c..333b070d6de878a4b3f0c861d5eb5f83b7ac6869 100644
--- a/public/web/WebServiceWorkerContextClient.h
+++ b/public/web/WebServiceWorkerContextClient.h
@@ -79,8 +79,9 @@ public:
 
     // ServiceWorker specific method. Called after InstallEvent (dispatched
     // via WebServiceWorkerContextProxy) is handled by the ServiceWorker's
-    // script context.
-    virtual void didHandleInstallEvent(int installEventID) { }
+    // script context. |rejected| is true if the event handler passed a promise
+    // to waitUntil() that rejected.
+    virtual void didHandleInstallEvent(int installEventID, bool rejected) { }

[kinuko, 2014/03/25 11:26:51]

If you're going to make yet another blink patch (which needs to land before your
first chrome patch) you can include this public interface change code into it
(and probably you can skip the second chrome patch instead)


1st patch (blink): add another virtual void didHandleInstallEvent(...) { } which
internally calls old (single-arg) didHandleInstallEvent()

2nd patch (chrome): implement the new didHandleInstallEvent without disabling
chrome test

3rd patch (blink): remove the old didHandleInstallEvent() interface

[falken, 2014/03/26 07:26:36]

Thanks for the idea. Taking this approach now.

 
     // ServiceWorker specific methods. Called after FetchEvent is handled by the
     // ServiceWorker's script context. When no response is provided, the browser