Revert of Pass the v8::Context through the willDestroyServiceWorkerContextOnWorkerThread event.

extensions/renderer/dispatcher.cc

 // Copyright 2014 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 "extensions/renderer/dispatcher.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/containers/scoped_ptr_map.h"
@@ skipping 179 matching lines @@
   void GetChrome(const v8::FunctionCallbackInfo<v8::Value>& args) {
     args.GetReturnValue().Set(GetOrCreateChrome(context()));
   }
 };
 
 class ServiceWorkerScriptContextSet {
  public:
   ServiceWorkerScriptContextSet() {}
   ~ServiceWorkerScriptContextSet() {}
 
-  void Insert(scoped_ptr<ScriptContext> context) {
+  void Insert(const GURL& url, scoped_ptr<ScriptContext> context) {
     base::AutoLock lock(lock_);
-    CHECK(FindScriptContext(context->v8_context()) == contexts_.end());
-    contexts_.push_back(context.Pass());
+    CHECK(script_contexts_.find(url) == script_contexts_.end());
+    script_contexts_.set(url, context.Pass());
   }
 
-  void Remove(v8::Local<v8::Context> v8_context) {
+  void Remove(const GURL& url) {
     base::AutoLock lock(lock_);
-    ScriptContextList::iterator context_it = FindScriptContext(v8_context);
-    // TODO(kalman): It would be good to CHECK(context_it != contexts_.end())
-    // here, but service workers can be started before the extension has been
-    // installed. See the length comment explaining why this happens, and
-    // how to solve it, in DidInitializeServiceWorkerContextOnWorkerThread.
-    // This does need to be fixed eventually, but for now, at least don't crash.
-    if (context_it == contexts_.end())
-      return;
-    (*context_it)->Invalidate();
-    contexts_.erase(context_it);
+    scoped_ptr<ScriptContext> context = script_contexts_.take_and_erase(url);
+    CHECK(context);
+    context->Invalidate();
   }
 
  private:
-  using ScriptContextList = ScopedVector<ScriptContext>;
-
-  // Returns an iterator to the ScriptContext associated with |v8_context|, or
-  // contexts_.end() if not found.
-  ScriptContextList::iterator FindScriptContext(
-      v8::Local<v8::Context> v8_context) {
-    for (auto it = contexts_.begin(); it != contexts_.end(); ++it) {
-      if ((*it)->v8_context() == v8_context)
-        return it;
-    }
-    return contexts_.end();
-  }
-
-  ScriptContextList contexts_;
+  base::ScopedPtrMap<GURL, scoped_ptr<ScriptContext>> script_contexts_;
 
   mutable base::Lock lock_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerScriptContextSet);
 };
 
 base::LazyInstance<ServiceWorkerScriptContextSet>
     g_service_worker_script_context_set = LAZY_INSTANCE_INITIALIZER;
 
 }  // namespace
@@ skipping 151 matching lines @@
 
 // static
 void Dispatcher::DidInitializeServiceWorkerContextOnWorkerThread(
     v8::Local<v8::Context> v8_context,
     const GURL& url) {
   const base::TimeTicks start_time = base::TimeTicks::Now();
 
   const Extension* extension =
       RendererExtensionRegistry::Get()->GetExtensionOrAppByURL(url);
 
-  if (!extension) {
-    // TODO(kalman): This is no good. Instead we need to either:
-    //
-    // - Hold onto the v8::Context and create the ScriptContext and install
-    //   our bindings when this extension is loaded.
-    // - Deal with there being an extension ID (url.host()) but no
-    //   extension associated with it, then document that getBackgroundClient
-    //   may fail if the extension hasn't loaded yet.
-    //
-    // The former is safer, but is unfriendly to caching (e.g. session restore).
-    // It seems to contradict the service worker idiom.
-    //
-    // The latter is friendly to caching, but running extension code without an
-    // installed extension makes me nervous, and means that we won't be able to
-    // expose arbitrary (i.e. capability-checked) extension APIs to service
-    // workers. We will probably need to relax some assertions - we just need
-    // to find them.
-    //
-    // Perhaps this could be solved with our own event on the service worker
-    // saying that an extension is ready, and documenting that extension APIs
-    // won't work before that event has fired?
+  if (!extension)
     return;
-  }
 
   ScriptContext* context = new ScriptContext(
       v8_context, nullptr, extension, Feature::SERVICE_WORKER_CONTEXT,
       extension, Feature::SERVICE_WORKER_CONTEXT);
 
-  g_service_worker_script_context_set.Get().Insert(make_scoped_ptr(context));
+  g_service_worker_script_context_set.Get().Insert(url,
+                                                   make_scoped_ptr(context));
 
   v8::Isolate* isolate = context->isolate();
 
   // Fetch the source code for service_worker_bindings.js.
   base::StringPiece script_resource =
       ResourceBundle::GetSharedInstance().GetRawDataResource(
           IDR_SERVICE_WORKER_BINDINGS_JS);
   v8::Local<v8::String> script = v8::String::NewExternal(
       isolate, new StaticV8ExternalOneByteStringResource(script_resource));
 
@@ skipping 26 matching lines @@
   // TODO(kalman): Make |request_sender| use |context->AddInvalidationObserver|.
   // In fact |request_sender_| should really be owned by ScriptContext.
   request_sender_->InvalidateSource(context);
 
   script_context_set_->Remove(context);
   VLOG(1) << "Num tracked contexts: " << script_context_set_->size();
 }
 
 // static
 void Dispatcher::WillDestroyServiceWorkerContextOnWorkerThread(
-    v8::Local<v8::Context> v8_context,
     const GURL& url) {
-  if (url.SchemeIs(kExtensionScheme))
-    g_service_worker_script_context_set.Get().Remove(v8_context);
+  if (RendererExtensionRegistry::Get()->GetExtensionOrAppByURL(url))
+    g_service_worker_script_context_set.Get().Remove(url);
 }
 
 void Dispatcher::DidCreateDocumentElement(blink::WebLocalFrame* frame) {
   // Note: use GetEffectiveDocumentURL not just frame->document()->url()
   // so that this also injects the stylesheet on about:blank frames that
   // are hosted in the extension process.
   GURL effective_document_url = ScriptContext::GetEffectiveDocumentURL(
       frame, frame->document().url(), true /* match_about_blank */);
 
   const Extension* extension =
@@ skipping 1115 matching lines @@
 void Dispatcher::AddChannelSpecificFeatures() {
   // chrome-extension: resources should be allowed to register a Service Worker.
   if (FeatureProvider::GetBehaviorFeature(BehaviorFeature::kServiceWorker)
           ->IsAvailableToEnvironment()
           .is_available())
     WebSecurityPolicy::registerURLSchemeAsAllowingServiceWorkers(
         WebString::fromUTF8(kExtensionScheme));
 }
 
 }  // namespace extensions