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

Unified Diff: third_party/WebKit/Source/core/loader/WorkerThreadableLoader.cpp

Issue 2761693002: Wrapped PassRefPtrs in move where passed to RefPtr constructor. (Closed)
Patch Set: Added move wraps for multiple instances in 1 line. Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/loader/WorkerThreadableLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/WorkerThreadableLoader.cpp b/third_party/WebKit/Source/core/loader/WorkerThreadableLoader.cpp
index 9af8d3db0a042b4b691d13003f1a63c2a3a4d07f..b3959f87d9e33940afa5af82b413aeb967fd5b11 100644
--- a/third_party/WebKit/Source/core/loader/WorkerThreadableLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/WorkerThreadableLoader.cpp
@@ -66,7 +66,7 @@ class WorkerThreadableLoader::AsyncTaskForwarder final
: public WorkerThreadableLoader::TaskForwarder {
public:
explicit AsyncTaskForwarder(PassRefPtr<WorkerLoaderProxy> loaderProxy)
- : m_loaderProxy(loaderProxy) {
+ : m_loaderProxy(std::move(loaderProxy)) {
DCHECK(isMainThread());
}
~AsyncTaskForwarder() override { DCHECK(isMainThread()); }
@@ -162,7 +162,7 @@ class WorkerThreadableLoader::SyncTaskForwarder final
: public WorkerThreadableLoader::TaskForwarder {
public:
explicit SyncTaskForwarder(PassRefPtr<WaitableEventWithTasks> eventWithTasks)
- : m_eventWithTasks(eventWithTasks) {
+ : m_eventWithTasks(std::move(eventWithTasks)) {
DCHECK(isMainThread());
}
~SyncTaskForwarder() override { DCHECK(isMainThread()); }

Powered by Google App Engine
This is Rietveld 408576698