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

Source/web/ServiceWorkerGlobalScopeProxy.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 186 matching lines @@
 void ServiceWorkerGlobalScopeProxy::postMessageToPageInspector(const String& message)
 {
     m_document.postInspectorTask(FROM_HERE, createCrossThreadTask(&WebEmbeddedWorkerImpl::postMessageToPageInspector, &m_embeddedWorker, message));
 }
 
 void ServiceWorkerGlobalScopeProxy::didEvaluateWorkerScript(bool success)
 {
     m_client.didEvaluateWorkerScript(success);
 }
 
+void ServiceWorkerGlobalScopeProxy::didInitializeWorkerContext()
+{
+    ASSERT(m_workerGlobalScope);
+    ScriptState::Scope scope(m_workerGlobalScope->script()->scriptState());
+    m_client.didInitializeWorkerContext(m_workerGlobalScope->script()->context(), WebURL(m_document.url()));

[kinuko, 2015/08/14 01:37:34]

KURL m_document.url() is not thread safe, I suspect we need to create an
isolated copy of the URL (by calling KURL::copy()) and store it in this proxy's
own field while we're on the main thread (i.e. in ctor), and use the isolated
copy here.

[annekao, 2015/08/14 19:44:23]

Done.

+}
+
 void ServiceWorkerGlobalScopeProxy::workerGlobalScopeStarted(WorkerGlobalScope* workerGlobalScope)
 {
     ASSERT(!m_workerGlobalScope);
     m_workerGlobalScope = static_cast<ServiceWorkerGlobalScope*>(workerGlobalScope);
     m_client.workerContextStarted(this);
 }
 
 void ServiceWorkerGlobalScopeProxy::workerGlobalScopeClosed()
 {
     m_document.postTask(FROM_HERE, createCrossThreadTask(&WebEmbeddedWorkerImpl::terminateWorkerContext, &m_embeddedWorker));
@@ skipping 12 matching lines @@
 
 ServiceWorkerGlobalScopeProxy::ServiceWorkerGlobalScopeProxy(WebEmbeddedWorkerImpl& embeddedWorker, Document& document, WebServiceWorkerContextClient& client)
     : m_embeddedWorker(embeddedWorker)
     , m_document(document)
     , m_client(client)
     , m_workerGlobalScope(0)
 {
 }
 
 } // namespace blink