Worker: Pass ParentFrameTaskRunners via WorkerThread::start() instead of the ctor

third_party/WebKit/Source/modules/webaudio/AudioWorkletThreadTest.cpp

 // Copyright 2016 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 "modules/webaudio/AudioWorkletThread.h"
 
 #include "bindings/core/v8/ScriptSourceCode.h"
 #include "bindings/core/v8/SourceLocation.h"
 #include "bindings/core/v8/V8GCController.h"
 #include "bindings/core/v8/WorkerOrWorkletScriptController.h"
@@ skipping 45 matching lines @@
  private:
   TestAudioWorkletReportingProxy() {}
 };
 
 }  // namespace
 
 class AudioWorkletThreadTest : public ::testing::Test {
  public:
   void SetUp() override {
     AudioWorkletThread::createSharedBackingThreadForTest();
-    m_parentFrameTaskRunners = ParentFrameTaskRunners::create(nullptr);
     m_reportingProxy = TestAudioWorkletReportingProxy::create();
     m_securityOrigin =
         SecurityOrigin::create(KURL(ParsedURLString, "http://fake.url/"));
   }
 
   void TearDown() override { AudioWorkletThread::clearSharedBackingThread(); }
 
   std::unique_ptr<AudioWorkletThread> createAudioWorkletThread() {
-    std::unique_ptr<AudioWorkletThread> thread = AudioWorkletThread::create(
-        nullptr, *m_reportingProxy, m_parentFrameTaskRunners.get());
-    thread->start(WorkerThreadStartupData::create(
-        KURL(ParsedURLString, "http://fake.url/"), "fake user agent", "",
-        nullptr, DontPauseWorkerGlobalScopeOnStart, nullptr, "",
-        m_securityOrigin.get(), nullptr, WebAddressSpaceLocal, nullptr, nullptr,
-        WorkerV8Settings::Default()));
+    std::unique_ptr<AudioWorkletThread> thread =
+        AudioWorkletThread::create(nullptr, *m_reportingProxy);
+    thread->start(
+        WorkerThreadStartupData::create(
+            KURL(ParsedURLString, "http://fake.url/"), "fake user agent", "",
+            nullptr, DontPauseWorkerGlobalScopeOnStart, nullptr, "",
+            m_securityOrigin.get(), nullptr, WebAddressSpaceLocal, nullptr,
+            nullptr, WorkerV8Settings::Default()),
+        ParentFrameTaskRunners::create(nullptr));
     return thread;
   }
 
   // Attempts to run some simple script for |thread|.
   void checkWorkletCanExecuteScript(WorkerThread* thread) {
     WaitableEvent waitEvent;
     thread->workerBackingThread().backingThread().postTask(
         BLINK_FROM_HERE,
         crossThreadBind(&AudioWorkletThreadTest::executeScriptInWorklet,
                         crossThreadUnretained(this),
                         crossThreadUnretained(thread),
                         crossThreadUnretained(&waitEvent)));
     waitEvent.wait();
   }
 
  private:
   void executeScriptInWorklet(WorkerThread* thread, WaitableEvent* waitEvent) {
     WorkerOrWorkletScriptController* scriptController =
         thread->globalScope()->scriptController();
     scriptController->evaluate(ScriptSourceCode("var counter = 0; ++counter;"));
     waitEvent->signal();
   }
 
   RefPtr<SecurityOrigin> m_securityOrigin;
   std::unique_ptr<WorkerReportingProxy> m_reportingProxy;
-  Persistent<ParentFrameTaskRunners> m_parentFrameTaskRunners;
 };
 
 TEST_F(AudioWorkletThreadTest, Basic) {
   std::unique_ptr<AudioWorkletThread> worklet = createAudioWorkletThread();
   checkWorkletCanExecuteScript(worklet.get());
   worklet->terminateAndWait();
 }
 
 // Tests that the same WebThread is used for new worklets if the WebThread is
 // still alive.
@@ skipping 72 matching lines @@
   ASSERT_TRUE(secondIsolate);
   EXPECT_EQ(firstIsolate, secondIsolate);
 
   // Verify that the isolate can run some scripts correctly in the second
   // worklet.
   checkWorkletCanExecuteScript(secondWorklet.get());
   secondWorklet->terminateAndWait();
 }
 
 }  // namespace blink