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

Unified Diff: content/renderer/service_worker/embedded_worker_context_client.cc

Issue 205563006: Add a status code to install event handled message from Service Worker (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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: content/renderer/service_worker/embedded_worker_context_client.cc
diff --git a/content/renderer/service_worker/embedded_worker_context_client.cc b/content/renderer/service_worker/embedded_worker_context_client.cc
index c0e1191388f8a5aa6e3504e87006e92694365446..d3cd8ee7d483481ef0d2075de97c94cf5d455247 100644
--- a/content/renderer/service_worker/embedded_worker_context_client.cc
+++ b/content/renderer/service_worker/embedded_worker_context_client.cc
@@ -121,8 +121,16 @@ void EmbeddedWorkerContextClient::workerContextDestroyed() {
}
void EmbeddedWorkerContextClient::didHandleInstallEvent(int request_id) {
+ didHandleInstallEvent(request_id, false /* rejected */);
+}
+
+void EmbeddedWorkerContextClient::didHandleInstallEvent(int request_id,
+ bool rejected) {
DCHECK(script_context_);
- script_context_->DidHandleInstallEvent(request_id);
+ script_context_->DidHandleInstallEvent(
+ request_id,
+ rejected ? SERVICE_WORKER_INSTALL_EVENT_RESULT_REJECTED
+ : SERVICE_WORKER_INSTALL_EVENT_RESULT_COMPLETED);
kinuko 2014/03/25 11:16:49 It looks like we'll also need this rejected or com
falken 2014/03/26 07:53:56 Done.
}
void EmbeddedWorkerContextClient::didHandleFetchEvent(int request_id) {

Powered by Google App Engine
This is Rietveld 408576698