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

chrome/renderer/chrome_content_renderer_client.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/renderer/chrome_content_renderer_client.h"
 
 #include "base/command_line.h"
 #include "base/debug/crash_logging.h"
 #include "base/logging.h"
 #include "base/metrics/field_trial.h"
@@ skipping 1612 matching lines @@
   WebString header_key(ASCIIToUTF16(
       data_reduction_proxy::chrome_proxy_header()));
   if (!response.httpHeaderField(header_key).isNull() &&
       response.httpHeaderField(header_key).utf8().find(
           data_reduction_proxy::chrome_proxy_lo_fi_directive()) !=
               std::string::npos) {
     (*properties)[data_reduction_proxy::chrome_proxy_header()] =
         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);
+  }
+}