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

Side by Side Diff: content/browser/service_worker/service_worker_version.cc

Issue 178343011: Support dispatching ServiceWorker FetchEvent and receiving response (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: plubming 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/service_worker/service_worker_version.h" 5 #include "content/browser/service_worker/service_worker_version.h"
6 6
7 #include "base/stl_util.h" 7 #include "base/stl_util.h"
8 #include "content/browser/service_worker/embedded_worker_instance.h" 8 #include "content/browser/service_worker/embedded_worker_instance.h"
9 #include "content/browser/service_worker/embedded_worker_registry.h" 9 #include "content/browser/service_worker/embedded_worker_registry.h"
10 #include "content/browser/service_worker/service_worker_context_core.h" 10 #include "content/browser/service_worker/service_worker_context_core.h"
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 << message.type(); 70 << message.type();
71 callback.Run(SERVICE_WORKER_ERROR_FAILED); 71 callback.Run(SERVICE_WORKER_ERROR_FAILED);
72 return; 72 return;
73 } 73 }
74 callback.Run(SERVICE_WORKER_OK); 74 callback.Run(SERVICE_WORKER_OK);
75 } 75 }
76 76
77 void HandleFetchResponse(const ServiceWorkerVersion::FetchCallback& callback, 77 void HandleFetchResponse(const ServiceWorkerVersion::FetchCallback& callback,
78 ServiceWorkerStatusCode status, 78 ServiceWorkerStatusCode status,
79 const IPC::Message& message) { 79 const IPC::Message& message) {
80 Tuple1<ServiceWorkerFetchResponse> response; 80 Tuple2<ServiceWorkerFetchEventResult, ServiceWorkerResponse> params;
81 if (message.type() != ServiceWorkerHostMsg_FetchEventFinished::ID) { 81 if (message.type() != ServiceWorkerHostMsg_FetchEventFinished::ID) {
82 NOTREACHED() << "Got unexpected response for FetchEvent: " 82 NOTREACHED() << "Got unexpected response for FetchEvent: "
83 << message.type(); 83 << message.type();
84 callback.Run(SERVICE_WORKER_ERROR_FAILED, response.a); 84 callback.Run(SERVICE_WORKER_ERROR_FAILED, params.a, params.b);
85 return; 85 return;
86 } 86 }
87 ServiceWorkerHostMsg_FetchEventFinished::Read(&message, &response); 87 ServiceWorkerHostMsg_FetchEventFinished::Read(&message, &params);
kinuko 2014/03/04 06:39:17 I think for 2+ parameters you can probably do this
falken 2014/03/05 03:35:41 Done.
88 callback.Run(status, response.a); 88 callback.Run(status, params.a, params.b);
89 } 89 }
90 90
91 } // namespace 91 } // namespace
92 92
93 ServiceWorkerVersion::ServiceWorkerVersion( 93 ServiceWorkerVersion::ServiceWorkerVersion(
94 ServiceWorkerRegistration* registration, 94 ServiceWorkerRegistration* registration,
95 EmbeddedWorkerRegistry* worker_registry, 95 EmbeddedWorkerRegistry* worker_registry,
96 int64 version_id) 96 int64 version_id)
97 : version_id_(version_id), 97 : version_id_(version_id),
98 is_shutdown_(false), 98 is_shutdown_(false),
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
258 if (callback) { 258 if (callback) {
259 callback->Run(SERVICE_WORKER_OK, message); 259 callback->Run(SERVICE_WORKER_OK, message);
260 message_callbacks_.Remove(request_id); 260 message_callbacks_.Remove(request_id);
261 return; 261 return;
262 } 262 }
263 NOTREACHED() << "Got unexpected message: " << request_id 263 NOTREACHED() << "Got unexpected message: " << request_id
264 << " " << message.type(); 264 << " " << message.type();
265 } 265 }
266 266
267 } // namespace content 267 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698