Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1411)

Unified Diff: Source/modules/serviceworkers/ServiceWorkerGlobalScopeClient.h

Issue 571843003: [ServivceWorker] Treat rejecting respondWith as a Network Error (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: incorporated yhirano's comment Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/modules/serviceworkers/ServiceWorkerGlobalScopeClient.h
diff --git a/Source/modules/serviceworkers/ServiceWorkerGlobalScopeClient.h b/Source/modules/serviceworkers/ServiceWorkerGlobalScopeClient.h
index 24172c365b61169e8e16aa52089f964c158c7942..e4a7f32fed23e1de0478aa4edaaecfa95ab82162 100644
--- a/Source/modules/serviceworkers/ServiceWorkerGlobalScopeClient.h
+++ b/Source/modules/serviceworkers/ServiceWorkerGlobalScopeClient.h
@@ -43,8 +43,8 @@
namespace blink {
class ExecutionContext;
-class Response;
class WebServiceWorkerCacheStorage;
+class WebServiceWorkerResponse;
class WebURL;
class WorkerClients;
@@ -59,8 +59,10 @@ public:
virtual void didHandleActivateEvent(int eventID, WebServiceWorkerEventResult) = 0;
virtual void didHandleInstallEvent(int installEventID, WebServiceWorkerEventResult) = 0;
- // A null response means no valid response was provided by the service worker, so fallback to native.
- virtual void didHandleFetchEvent(int fetchEventID, Response* = 0) = 0;
+ // Calling didHandleFetchEvent without response means no response was
+ // provided by the service worker in the fetch events, so fallback to native.
+ virtual void didHandleFetchEvent(int fetchEventID) = 0;
+ virtual void didHandleFetchEvent(int fetchEventID, const WebServiceWorkerResponse&) = 0;
virtual void didHandleSyncEvent(int syncEventID) = 0;
virtual void postMessageToClient(int clientID, const WebString& message, PassOwnPtr<WebMessagePortChannelArray>) = 0;

Powered by Google App Engine
This is Rietveld 408576698