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

Side by Side Diff: third_party/WebKit/Source/core/workers/ThreadedMessagingProxyBase.h

Issue 2645613006: Worker: Stop using ExecutionContextTask on WorkerLoaderProxy::postTaskToWorkerGlobalScope (Closed)
Patch Set: address review comments Created 3 years, 11 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 #ifndef ThreadedMessagingProxyBase_h 5 #ifndef ThreadedMessagingProxyBase_h
6 #define ThreadedMessagingProxyBase_h 6 #define ThreadedMessagingProxyBase_h
7 7
8 #include "core/CoreExport.h" 8 #include "core/CoreExport.h"
9 #include "core/frame/UseCounter.h" 9 #include "core/frame/UseCounter.h"
10 #include "core/inspector/ConsoleTypes.h" 10 #include "core/inspector/ConsoleTypes.h"
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 // Returns true if this is called on the parent context thread. 75 // Returns true if this is called on the parent context thread.
76 bool isParentContextThread() const; 76 bool isParentContextThread() const;
77 77
78 // WorkerLoaderProxyProvider 78 // WorkerLoaderProxyProvider
79 // These methods are called on different threads to schedule loading 79 // These methods are called on different threads to schedule loading
80 // requests and to send callbacks back to WorkerGlobalScope. 80 // requests and to send callbacks back to WorkerGlobalScope.
81 void postTaskToLoader(const WebTraceLocation&, 81 void postTaskToLoader(const WebTraceLocation&,
82 std::unique_ptr<ExecutionContextTask>) override; 82 std::unique_ptr<ExecutionContextTask>) override;
83 void postTaskToWorkerGlobalScope( 83 void postTaskToWorkerGlobalScope(
84 const WebTraceLocation&, 84 const WebTraceLocation&,
85 std::unique_ptr<ExecutionContextTask>) override; 85 std::unique_ptr<WTF::CrossThreadClosure>) override;
86 86
87 private: 87 private:
88 friend class InProcessWorkerMessagingProxyForTest; 88 friend class InProcessWorkerMessagingProxyForTest;
89 friend class ThreadedWorkletMessagingProxyForTest; 89 friend class ThreadedWorkletMessagingProxyForTest;
90 90
91 void parentObjectDestroyedInternal(); 91 void parentObjectDestroyedInternal();
92 92
93 Persistent<ExecutionContext> m_executionContext; 93 Persistent<ExecutionContext> m_executionContext;
94 Persistent<WorkerInspectorProxy> m_workerInspectorProxy; 94 Persistent<WorkerInspectorProxy> m_workerInspectorProxy;
95 Persistent<ParentFrameTaskRunners> m_parentFrameTaskRunners; 95 Persistent<ParentFrameTaskRunners> m_parentFrameTaskRunners;
96 96
97 std::unique_ptr<WorkerThread> m_workerThread; 97 std::unique_ptr<WorkerThread> m_workerThread;
98 98
99 RefPtr<WorkerLoaderProxy> m_loaderProxy; 99 RefPtr<WorkerLoaderProxy> m_loaderProxy;
100 100
101 bool m_mayBeDestroyed; 101 bool m_mayBeDestroyed;
102 bool m_askedToTerminate; 102 bool m_askedToTerminate;
103 }; 103 };
104 104
105 } // namespace blink 105 } // namespace blink
106 106
107 #endif // ThreadedMessagingProxyBase_h 107 #endif // ThreadedMessagingProxyBase_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698