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

Side by Side Diff: third_party/WebKit/Source/modules/compositorworker/AnimationWorkletThread.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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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/AnimationWorkletThread.h" 5 #include "modules/compositorworker/AnimationWorkletThread.h"
6 6
7 #include "core/workers/WorkerThreadStartupData.h" 7 #include "core/workers/WorkerThreadStartupData.h"
8 #include "modules/compositorworker/AnimationWorkletGlobalScope.h" 8 #include "modules/compositorworker/AnimationWorkletGlobalScope.h"
9 #include "platform/instrumentation/tracing/TraceEvent.h" 9 #include "platform/instrumentation/tracing/TraceEvent.h"
10 #include "platform/weborigin/SecurityOrigin.h" 10 #include "platform/weborigin/SecurityOrigin.h"
11 #include "wtf/PtrUtil.h" 11 #include "wtf/PtrUtil.h"
12 12
13 namespace blink { 13 namespace blink {
14 14
15 std::unique_ptr<AnimationWorkletThread> AnimationWorkletThread::create( 15 std::unique_ptr<AnimationWorkletThread> AnimationWorkletThread::create(
16 PassRefPtr<WorkerLoaderProxy> workerLoaderProxy, 16 PassRefPtr<WorkerLoaderProxy> workerLoaderProxy,
17 WorkerReportingProxy& workerReportingProxy, 17 WorkerReportingProxy& workerReportingProxy) {
18 ParentFrameTaskRunners* parentFrameTaskRunners) {
19 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("animation-worklet"), 18 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("animation-worklet"),
20 "AnimationWorkletThread::create"); 19 "AnimationWorkletThread::create");
21 DCHECK(isMainThread()); 20 DCHECK(isMainThread());
22 return WTF::wrapUnique( 21 return WTF::wrapUnique(new AnimationWorkletThread(
23 new AnimationWorkletThread(std::move(workerLoaderProxy), 22 std::move(workerLoaderProxy), workerReportingProxy));
24 workerReportingProxy, parentFrameTaskRunners));
25 } 23 }
26 24
27 AnimationWorkletThread::AnimationWorkletThread( 25 AnimationWorkletThread::AnimationWorkletThread(
28 PassRefPtr<WorkerLoaderProxy> workerLoaderProxy, 26 PassRefPtr<WorkerLoaderProxy> workerLoaderProxy,
29 WorkerReportingProxy& workerReportingProxy, 27 WorkerReportingProxy& workerReportingProxy)
30 ParentFrameTaskRunners* parentFrameTaskRunners)
31 : AbstractAnimationWorkletThread(std::move(workerLoaderProxy), 28 : AbstractAnimationWorkletThread(std::move(workerLoaderProxy),
32 workerReportingProxy, 29 workerReportingProxy) {}
33 parentFrameTaskRunners) {}
34 30
35 AnimationWorkletThread::~AnimationWorkletThread() {} 31 AnimationWorkletThread::~AnimationWorkletThread() {}
36 32
37 WorkerOrWorkletGlobalScope* AnimationWorkletThread::createWorkerGlobalScope( 33 WorkerOrWorkletGlobalScope* AnimationWorkletThread::createWorkerGlobalScope(
38 std::unique_ptr<WorkerThreadStartupData> startupData) { 34 std::unique_ptr<WorkerThreadStartupData> startupData) {
39 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("animation-worklet"), 35 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("animation-worklet"),
40 "AnimationWorkletThread::createWorkerGlobalScope"); 36 "AnimationWorkletThread::createWorkerGlobalScope");
41 37
42 RefPtr<SecurityOrigin> securityOrigin = 38 RefPtr<SecurityOrigin> securityOrigin =
43 SecurityOrigin::create(startupData->m_scriptURL); 39 SecurityOrigin::create(startupData->m_scriptURL);
44 if (startupData->m_starterOriginPrivilegeData) 40 if (startupData->m_starterOriginPrivilegeData)
45 securityOrigin->transferPrivilegesFrom( 41 securityOrigin->transferPrivilegesFrom(
46 std::move(startupData->m_starterOriginPrivilegeData)); 42 std::move(startupData->m_starterOriginPrivilegeData));
47 43
48 // TODO(ikilpatrick): The AnimationWorkletGlobalScope will need to store a 44 // TODO(ikilpatrick): The AnimationWorkletGlobalScope will need to store a
49 // WorkerClients object for using a CompositorWorkerProxyClient object. 45 // WorkerClients object for using a CompositorWorkerProxyClient object.
50 return AnimationWorkletGlobalScope::create( 46 return AnimationWorkletGlobalScope::create(
51 startupData->m_scriptURL, startupData->m_userAgent, 47 startupData->m_scriptURL, startupData->m_userAgent,
52 securityOrigin.release(), this->isolate(), this); 48 securityOrigin.release(), this->isolate(), this);
53 } 49 }
54 50
55 } // namespace blink 51 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698