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

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

Issue 580993002: Revert of [ServivceWorker] Treat rejecting respondWith as a Network Error (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: 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 e4a7f32fed23e1de0478aa4edaaecfa95ab82162..24172c365b61169e8e16aa52089f964c158c7942 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,10 +59,8 @@
virtual void didHandleActivateEvent(int eventID, WebServiceWorkerEventResult) = 0;
virtual void didHandleInstallEvent(int installEventID, WebServiceWorkerEventResult) = 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;
+ // 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;
virtual void didHandleSyncEvent(int syncEventID) = 0;
virtual void postMessageToClient(int clientID, const WebString& message, PassOwnPtr<WebMessagePortChannelArray>) = 0;
« no previous file with comments | « Source/modules/serviceworkers/RespondWithObserver.cpp ('k') | Source/web/ServiceWorkerGlobalScopeClientImpl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698