[Extension ServiceWorkers] Chrome: Passing v8::context to extensions dispatcher

chrome/renderer/chrome_content_renderer_client.cc

Index: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index 481b439e666696669ca478967ff2dfcf75f96dff..40560d9a4e19f5e1a9aa5396022cfe24d4509b50 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -1630,3 +1630,13 @@ void ChromeContentRendererClient::AddImageContextMenuProperties(
         data_reduction_proxy::chrome_proxy_lo_fi_directive();
   }
 }
+
+void ChromeContentRendererClient::
+    DidInitializeServiceWorkerContextOnWorkerThread(
+        v8::Local<v8::Context> context,

[Devlin, 2015/07/30 22:58:51]

This indentation looks funny - was it git cl format's doing?

[annekao, 2015/07/30 23:57:39]

Yup.

+        const GURL& url) {
+  if (url.SchemeIs(extensions::kExtensionScheme)) {

[Devlin, 2015/07/30 22:58:51]

Eh, I think the dispatcher should handle this check. Best to have as little
extension-y code here as possible.  On that note, we should #ifdef out the body
of this method with ENABLE_EXTENSIONS.

[annekao, 2015/07/30 23:57:39]

Done.

+    extension_dispatcher_->DidInitializeServiceWorkerContextOnWorkerThread(

[Devlin, 2015/07/30 22:58:51]

This should be called statically.

[annekao, 2015/07/30 23:57:39]

Done.

+        context);
+  }
+}