[ServiceWorker] Mojofy extendable message event.

content/browser/service_worker/service_worker_version.cc

Index: content/browser/service_worker/service_worker_version.cc
diff --git a/content/browser/service_worker/service_worker_version.cc b/content/browser/service_worker/service_worker_version.cc
index 3dd451bf2aab7e84e417b3553b33d3f6485c74a2..822908ffc406e8a9eadbd1bbc9dd44e97219b4ca 100644
--- a/content/browser/service_worker/service_worker_version.cc
+++ b/content/browser/service_worker/service_worker_version.cc
@@ -1073,6 +1073,20 @@ void ServiceWorkerVersion::OnGetClientsFinished(int request_id,
       ServiceWorkerMsg_DidGetClients(request_id, *clients));
 }
 
+void ServiceWorkerVersion::OnSimpleEventFinished(
+    int request_id,
+    ServiceWorkerStatusCode status,
+    base::Time dispatch_event_time) {
+  // Copy error callback before calling FinishRequest.
+  PendingRequest* request = pending_requests_.Lookup(request_id);
+  DCHECK(request) << "Invalid request id";
+  StatusCallback callback = request->error_callback;
+
+  FinishRequest(request_id, status == SERVICE_WORKER_OK, dispatch_event_time);
+
+  callback.Run(status);
+}
+
 void ServiceWorkerVersion::OnSimpleEventResponse(

[shimazu, 2016/12/02 04:00:33]

How about deduping the codes by calling OnSimpleEventFinished in
OnSimpleEventResponse?

and I think the names should be the same because there is no difference between
the two. I prefer OnSimpleEventFinished to OnSimpleEventResponse because it's
clearer.

[leonhsl(Using Gerrit), 2016/12/02 08:36:44]

Acknowledged.

[leonhsl(Using Gerrit), 2016/12/02 15:43:27]

Put the duplicated codes into OnSimpleEventFinished and let
OnSimpleEventResponse call it. But for the naming change, if we renamed
OnSimpleEventResponse to OnSimpleEventFinished, at [1] base::Bind() can't deduce
which OnSimpleEventFinished should be bound.

[1]
https://cs.chromium.org/chromium/src/content/browser/service_worker/service_w...

[shimazu, 2016/12/05 03:40:21]

Thanks, I could understand why the names are different. How about adding a
comment about that (I put another comment to SWVersion) instead of renaming?

     int request_id,
     blink::WebServiceWorkerEventResult result,