Worker: Fix broken GC logic on Dedicated Worker while DOMTimer is set

third_party/WebKit/Source/modules/compositorworker/CompositorWorkerMessagingProxy.cpp

 // Copyright 2015 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/compositorworker/CompositorWorkerMessagingProxy.h"
 
+#include "core/workers/InProcessWorkerBase.h"
 #include "core/workers/WorkerThreadStartupData.h"
 #include "modules/compositorworker/CompositorWorkerThread.h"
 #include <memory>
 
 namespace blink {
 
 CompositorWorkerMessagingProxy::CompositorWorkerMessagingProxy(InProcessWorkerBase* worker, WorkerClients* workerClients)
-    : InProcessWorkerMessagingProxy(worker, workerClients)
+    : InProcessWorkerMessagingProxy(worker->getExecutionContext(), worker, workerClients)

[haraken, 2016/08/05 08:20:58]

Maybe is it redundant to pass both |worker| and |worker->getExecutionContext()|?

[nhiroki, 2016/08/08 09:19:14]

Reverted this change (see my reply to kinuko's comment in
InProcessWorkerMessagingProxy.h)

 {
 }
 
 CompositorWorkerMessagingProxy::~CompositorWorkerMessagingProxy()
 {
 }
 
 std::unique_ptr<WorkerThread> CompositorWorkerMessagingProxy::createWorkerThread(double originTime)
 {
     return CompositorWorkerThread::create(loaderProxy(), workerObjectProxy(), originTime);
 }
 
 } // namespace blink