Add browser test for V8 code caching for ServiceWorker scripts.

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 ede55528807ba8d7cfd4b30c191196b5b354be17..a4eb0d8323eea8b7e13934056f2c307594d62f83 100644
--- a/content/browser/service_worker/service_worker_version.cc
+++ b/content/browser/service_worker/service_worker_version.cc
@@ -272,18 +272,6 @@ void KillEmbeddedWorkerProcess(int process_id, ResultCode code) {
     render_process_host->ReceivedBadMessage();
 }
 
-void DidSetCachedMetadata(int64 callback_id, int result) {
-  TRACE_EVENT_ASYNC_END1("ServiceWorker",
-                         "ServiceWorkerVersion::OnSetCachedMetadata",
-                         callback_id, "result", result);
-}
-
-void DidClearCachedMetadata(int64 callback_id, int result) {
-  TRACE_EVENT_ASYNC_END1("ServiceWorker",
-                         "ServiceWorkerVersion::OnClearCachedMetadata",
-                         callback_id, "result", result);
-}
-
 }  // namespace
 
 ServiceWorkerVersion::ServiceWorkerVersion(
@@ -1244,7 +1232,16 @@ void ServiceWorkerVersion::OnSetCachedMetadata(const GURL& url,
                            "ServiceWorkerVersion::OnSetCachedMetadata",
                            callback_id, "URL", url.spec());
   script_cache_map_.WriteMetadata(
-      url, data, base::Bind(&DidSetCachedMetadata, callback_id));
+      url, data, base::Bind(&ServiceWorkerVersion::OnSetCachedMetadataFinished,
+                            weak_factory_.GetWeakPtr(), callback_id));
+}
+
+void ServiceWorkerVersion::OnSetCachedMetadataFinished(int64 callback_id,
+                                                       int result) {
+  TRACE_EVENT_ASYNC_END1("ServiceWorker",
+                         "ServiceWorkerVersion::OnSetCachedMetadata",
+                         callback_id, "result", result);
+  FOR_EACH_OBSERVER(Listener, listeners_, OnCachedMetadataUpdated(this));
 }
 
 void ServiceWorkerVersion::OnClearCachedMetadata(const GURL& url) {
@@ -1253,7 +1250,16 @@ void ServiceWorkerVersion::OnClearCachedMetadata(const GURL& url) {
                            "ServiceWorkerVersion::OnClearCachedMetadata",
                            callback_id, "URL", url.spec());
   script_cache_map_.ClearMetadata(
-      url, base::Bind(&DidClearCachedMetadata, callback_id));
+      url, base::Bind(&ServiceWorkerVersion::OnClearCachedMetadataFinished,
+                      weak_factory_.GetWeakPtr(), callback_id));
+}
+
+void ServiceWorkerVersion::OnClearCachedMetadataFinished(int64 callback_id,
+                                                         int result) {
+  TRACE_EVENT_ASYNC_END1("ServiceWorker",
+                         "ServiceWorkerVersion::OnClearCachedMetadata",
+                         callback_id, "result", result);
+  FOR_EACH_OBSERVER(Listener, listeners_, OnCachedMetadataUpdated(this));
 }
 
 void ServiceWorkerVersion::OnPostMessageToDocument(