Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(186)

Side by Side Diff: third_party/WebKit/Source/modules/compositorworker/CompositorWorkerThread.cpp

Issue 2718643002: Worker: Pass ParentFrameTaskRunners via WorkerThread::start() instead of the ctor (Closed)
Patch Set: Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "modules/compositorworker/CompositorWorkerThread.h" 5 #include "modules/compositorworker/CompositorWorkerThread.h"
6 6
7 #include "core/workers/InProcessWorkerObjectProxy.h" 7 #include "core/workers/InProcessWorkerObjectProxy.h"
8 #include "core/workers/WorkerThreadStartupData.h" 8 #include "core/workers/WorkerThreadStartupData.h"
9 #include "modules/compositorworker/CompositorWorkerGlobalScope.h" 9 #include "modules/compositorworker/CompositorWorkerGlobalScope.h"
10 #include "platform/instrumentation/tracing/TraceEvent.h" 10 #include "platform/instrumentation/tracing/TraceEvent.h"
11 #include "wtf/Assertions.h" 11 #include "wtf/Assertions.h"
12 #include <memory> 12 #include <memory>
13 13
14 namespace blink { 14 namespace blink {
15 15
16 std::unique_ptr<CompositorWorkerThread> CompositorWorkerThread::create( 16 std::unique_ptr<CompositorWorkerThread> CompositorWorkerThread::create(
17 PassRefPtr<WorkerLoaderProxy> workerLoaderProxy, 17 PassRefPtr<WorkerLoaderProxy> workerLoaderProxy,
18 InProcessWorkerObjectProxy& workerObjectProxy, 18 InProcessWorkerObjectProxy& workerObjectProxy,
19 ParentFrameTaskRunners* parentFrameTaskRunners,
20 double timeOrigin) { 19 double timeOrigin) {
21 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("compositor-worker"), 20 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("compositor-worker"),
22 "CompositorWorkerThread::create"); 21 "CompositorWorkerThread::create");
23 ASSERT(isMainThread()); 22 ASSERT(isMainThread());
24 return WTF::wrapUnique(new CompositorWorkerThread( 23 return WTF::wrapUnique(new CompositorWorkerThread(
25 std::move(workerLoaderProxy), workerObjectProxy, parentFrameTaskRunners, 24 std::move(workerLoaderProxy), workerObjectProxy, timeOrigin));
26 timeOrigin));
27 } 25 }
28 26
29 CompositorWorkerThread::CompositorWorkerThread( 27 CompositorWorkerThread::CompositorWorkerThread(
30 PassRefPtr<WorkerLoaderProxy> workerLoaderProxy, 28 PassRefPtr<WorkerLoaderProxy> workerLoaderProxy,
31 InProcessWorkerObjectProxy& workerObjectProxy, 29 InProcessWorkerObjectProxy& workerObjectProxy,
32 ParentFrameTaskRunners* parentFrameTaskRunners,
33 double timeOrigin) 30 double timeOrigin)
34 : AbstractAnimationWorkletThread(std::move(workerLoaderProxy), 31 : AbstractAnimationWorkletThread(std::move(workerLoaderProxy),
35 workerObjectProxy, 32 workerObjectProxy),
36 parentFrameTaskRunners),
37 m_workerObjectProxy(workerObjectProxy), 33 m_workerObjectProxy(workerObjectProxy),
38 m_timeOrigin(timeOrigin) {} 34 m_timeOrigin(timeOrigin) {}
39 35
40 CompositorWorkerThread::~CompositorWorkerThread() {} 36 CompositorWorkerThread::~CompositorWorkerThread() {}
41 37
42 WorkerOrWorkletGlobalScope* CompositorWorkerThread::createWorkerGlobalScope( 38 WorkerOrWorkletGlobalScope* CompositorWorkerThread::createWorkerGlobalScope(
43 std::unique_ptr<WorkerThreadStartupData> startupData) { 39 std::unique_ptr<WorkerThreadStartupData> startupData) {
44 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("compositor-worker"), 40 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("compositor-worker"),
45 "CompositorWorkerThread::createWorkerGlobalScope"); 41 "CompositorWorkerThread::createWorkerGlobalScope");
46 return CompositorWorkerGlobalScope::create(this, std::move(startupData), 42 return CompositorWorkerGlobalScope::create(this, std::move(startupData),
47 m_timeOrigin); 43 m_timeOrigin);
48 } 44 }
49 45
50 } // namespace blink 46 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698