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

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

Issue 2308343002: Replaced PassRefPtr copites with moves in Source/modules. (Closed)
Patch Set: Created 4 years, 3 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/TraceEvent.h" 9 #include "platform/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(PassRefPt r<WorkerLoaderProxy> workerLoaderProxy, WorkerReportingProxy& workerReportingPro xy) 15 std::unique_ptr<AnimationWorkletThread> AnimationWorkletThread::create(PassRefPt r<WorkerLoaderProxy> workerLoaderProxy, WorkerReportingProxy& workerReportingPro xy)
16 { 16 {
17 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("animation-worklet"), "AnimationWorkl etThread::create"); 17 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("animation-worklet"), "AnimationWorkl etThread::create");
18 DCHECK(isMainThread()); 18 DCHECK(isMainThread());
19 return wrapUnique(new AnimationWorkletThread(workerLoaderProxy, workerReport ingProxy)); 19 return wrapUnique(new AnimationWorkletThread(std::move(workerLoaderProxy), w orkerReportingProxy));
20 } 20 }
21 21
22 AnimationWorkletThread::AnimationWorkletThread(PassRefPtr<WorkerLoaderProxy> wor kerLoaderProxy, WorkerReportingProxy& workerReportingProxy) 22 AnimationWorkletThread::AnimationWorkletThread(PassRefPtr<WorkerLoaderProxy> wor kerLoaderProxy, WorkerReportingProxy& workerReportingProxy)
23 : AbstractAnimationWorkletThread(workerLoaderProxy, workerReportingProxy) 23 : AbstractAnimationWorkletThread(std::move(workerLoaderProxy), workerReporti ngProxy)
24 { 24 {
25 } 25 }
26 26
27 AnimationWorkletThread::~AnimationWorkletThread() 27 AnimationWorkletThread::~AnimationWorkletThread()
28 { 28 {
29 } 29 }
30 30
31 WorkerOrWorkletGlobalScope* AnimationWorkletThread::createWorkerGlobalScope(std: :unique_ptr<WorkerThreadStartupData> startupData) 31 WorkerOrWorkletGlobalScope* AnimationWorkletThread::createWorkerGlobalScope(std: :unique_ptr<WorkerThreadStartupData> startupData)
32 { 32 {
33 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("animation-worklet"), "AnimationWorkl etThread::createWorkerGlobalScope"); 33 TRACE_EVENT0(TRACE_DISABLED_BY_DEFAULT("animation-worklet"), "AnimationWorkl etThread::createWorkerGlobalScope");
34 34
35 RefPtr<SecurityOrigin> securityOrigin = SecurityOrigin::create(startupData-> m_scriptURL); 35 RefPtr<SecurityOrigin> securityOrigin = SecurityOrigin::create(startupData-> m_scriptURL);
36 if (startupData->m_starterOriginPrivilegeData) 36 if (startupData->m_starterOriginPrivilegeData)
37 securityOrigin->transferPrivilegesFrom(std::move(startupData->m_starterO riginPrivilegeData)); 37 securityOrigin->transferPrivilegesFrom(std::move(startupData->m_starterO riginPrivilegeData));
38 38
39 // TODO(ikilpatrick): The AnimationWorkletGlobalScope will need to store a 39 // TODO(ikilpatrick): The AnimationWorkletGlobalScope will need to store a
40 // WorkerClients object for using a CompositorProxyClient object. 40 // WorkerClients object for using a CompositorProxyClient object.
41 return AnimationWorkletGlobalScope::create(startupData->m_scriptURL, startup Data->m_userAgent, securityOrigin.release(), this->isolate(), this); 41 return AnimationWorkletGlobalScope::create(startupData->m_scriptURL, startup Data->m_userAgent, securityOrigin.release(), this->isolate(), this);
42 } 42 }
43 43
44 } // namespace blink 44 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698