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

Side by Side Diff: third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.h

Issue 1675613002: service worker: use 200 OK for update requests even in the no update case (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: asan and fix win compile? Created 4 years, 10 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 , public WebDevToolsAgentClient 55 , public WebDevToolsAgentClient
56 , private WorkerLoaderProxyProvider { 56 , private WorkerLoaderProxyProvider {
57 WTF_MAKE_NONCOPYABLE(WebEmbeddedWorkerImpl); 57 WTF_MAKE_NONCOPYABLE(WebEmbeddedWorkerImpl);
58 public: 58 public:
59 WebEmbeddedWorkerImpl(PassOwnPtr<WebServiceWorkerContextClient>, PassOwnPtr< WebWorkerContentSettingsClientProxy>); 59 WebEmbeddedWorkerImpl(PassOwnPtr<WebServiceWorkerContextClient>, PassOwnPtr< WebWorkerContentSettingsClientProxy>);
60 ~WebEmbeddedWorkerImpl() override; 60 ~WebEmbeddedWorkerImpl() override;
61 61
62 // WebEmbeddedWorker overrides. 62 // WebEmbeddedWorker overrides.
63 void startWorkerContext(const WebEmbeddedWorkerStartData&) override; 63 void startWorkerContext(const WebEmbeddedWorkerStartData&) override;
64 void terminateWorkerContext() override; 64 void terminateWorkerContext() override;
65 void resumeAfterDownload() override;
65 void attachDevTools(const WebString& hostId, int sessionId) override; 66 void attachDevTools(const WebString& hostId, int sessionId) override;
66 void reattachDevTools(const WebString& hostId, int sessionId, const WebStrin g& savedState) override; 67 void reattachDevTools(const WebString& hostId, int sessionId, const WebStrin g& savedState) override;
67 void detachDevTools() override; 68 void detachDevTools() override;
68 void dispatchDevToolsMessage(int sessionId, const WebString&) override; 69 void dispatchDevToolsMessage(int sessionId, const WebString&) override;
69 70
70 void postMessageToPageInspector(const WTF::String&); 71 void postMessageToPageInspector(const WTF::String&);
71 72
72 private: 73 private:
73 void prepareShadowPageForLoader(); 74 void prepareShadowPageForLoader();
74 void loadShadowPage(); 75 void loadShadowPage();
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 // Both WebView and WebFrame objects are close()'ed (where they're 115 // Both WebView and WebFrame objects are close()'ed (where they're
115 // deref'ed) when this EmbeddedWorkerImpl is destructed, therefore they 116 // deref'ed) when this EmbeddedWorkerImpl is destructed, therefore they
116 // are guaranteed to exist while this object is around. 117 // are guaranteed to exist while this object is around.
117 WebView* m_webView; 118 WebView* m_webView;
118 RefPtrWillBePersistent<WebLocalFrameImpl> m_mainFrame; 119 RefPtrWillBePersistent<WebLocalFrameImpl> m_mainFrame;
119 120
120 bool m_loadingShadowPage; 121 bool m_loadingShadowPage;
121 bool m_askedToTerminate; 122 bool m_askedToTerminate;
122 123
123 enum WaitingForDebuggerState { 124 enum WaitingForDebuggerState {
124 WaitingForDebuggerBeforeLoadingScript, 125 WaitingForDebugger,
125 WaitingForDebuggerAfterScriptLoaded,
126 NotWaitingForDebugger 126 NotWaitingForDebugger
127 }; 127 };
128 128
129 enum {
130 DontPauseAfterDownload,
131 DoPauseAfterDownload,
132 IsPausedAfterDownload
133 } m_pauseAfterDownloadState;
134
129 WaitingForDebuggerState m_waitingForDebuggerState; 135 WaitingForDebuggerState m_waitingForDebuggerState;
130 }; 136 };
131 137
132 } // namespace blink 138 } // namespace blink
133 139
134 #endif // WebEmbeddedWorkerImpl_h 140 #endif // WebEmbeddedWorkerImpl_h
OLDNEW
« no previous file with comments | « content/renderer/service_worker/embedded_worker_dispatcher.cc ('k') | third_party/WebKit/Source/web/WebEmbeddedWorkerImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698