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

Source/core/workers/WorkerThread.cpp

Index: Source/core/workers/WorkerThread.cpp
diff --git a/Source/core/workers/WorkerThread.cpp b/Source/core/workers/WorkerThread.cpp
index 632fd6dc9a022ea1126060ffeb5c2263223e12d8..323c4d29014a7effe13214fcfb462190a9e16626 100644
--- a/Source/core/workers/WorkerThread.cpp
+++ b/Source/core/workers/WorkerThread.cpp
@@ -250,6 +250,10 @@ void WorkerThread::initialize(PassOwnPtr<WorkerThreadStartupData> startupData)
     }
     m_workerGlobalScope->workerInspectorController()->workerContextInitialized(startMode == PauseWorkerGlobalScopeOnStart);
 
+    if (m_workerGlobalScope->script()->scriptState()->contextIsValid()) {

[sadrul, 2015/08/11 22:07:31]

Do you think this should first check if
m_workerGlobalScope->script()->isContextIntialized() (or that
script()->scriptState() is non-nul)?

I ask since some related test seemed to start failing after this change (e.g.
the first failure in
http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.8/?numbui...
coincides with this change:
http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.8/builds/...)

+        m_workerReportingProxy.didInitializeWorkerContext();
+    }
+
     OwnPtrWillBeRawPtr<CachedMetadataHandler> handler(workerGlobalScope()->createWorkerScriptCachedMetadataHandler(scriptURL, cachedMetaData.get()));
     bool success = m_workerGlobalScope->script()->evaluate(ScriptSourceCode(sourceCode, scriptURL), nullptr, handler.get(), v8CacheOptions);
     m_workerGlobalScope->didEvaluateWorkerScript();