Index: content/renderer/service_worker/service_worker_script_context.cc |
diff --git a/content/renderer/service_worker/service_worker_script_context.cc b/content/renderer/service_worker/service_worker_script_context.cc |
index 3edd887bac377321d48410a1d8ff15bd1f464713..8e4b8af26b2de984c17db4a13e0278624cb7e64d 100644 |
--- a/content/renderer/service_worker/service_worker_script_context.cc |
+++ b/content/renderer/service_worker/service_worker_script_context.cc |
@@ -4,6 +4,7 @@ |
#include "content/renderer/service_worker/service_worker_script_context.h" |
+#include "base/debug/stack_trace.h" |
nhiroki
2014/02/14 09:03:21
It looks like this is not used for now. Are you pl
kinuko
2014/02/14 09:19:13
This is WIP patch, I believe it's for his local de
nhiroki
2014/02/14 09:41:50
Ah, that make sense.
falken
2014/02/19 05:32:19
Right it was just for debugging. Removed.
|
#include "base/logging.h" |
#include "content/common/service_worker/service_worker_messages.h" |
#include "content/renderer/service_worker/embedded_worker_context_client.h" |
@@ -41,6 +42,12 @@ void ServiceWorkerScriptContext::DidHandleInstallEvent(int request_id) { |
Send(request_id, ServiceWorkerHostMsg_InstallEventFinished()); |
} |
+void ServiceWorkerScriptContext::DidHandleFetchEvent( |
+ int request_id, |
+ const ServiceWorkerFetchResponse& response) { |
+ Send(request_id, ServiceWorkerHostMsg_FetchEventFinished(response)); |
+} |
+ |
void ServiceWorkerScriptContext::Send(int request_id, |
const IPC::Message& message) { |
embedded_context_->SendMessageToBrowser(request_id, message); |
@@ -53,7 +60,10 @@ void ServiceWorkerScriptContext::OnInstallEvent( |
void ServiceWorkerScriptContext::OnFetchEvent( |
const ServiceWorkerFetchRequest& request) { |
- NOTIMPLEMENTED(); |
+ // TODO(falken): Dispatch the event to Blink and wait for the response from |
+ // respondWith. This is just a dummy response now. |
+ DidHandleFetchEvent(current_request_id_, ServiceWorkerFetchResponse( |
+ 200, "OK", "GET", std::map<std::string, std::string>())); |
} |
} // namespace content |