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

Unified Diff: third_party/WebKit/Source/core/workers/WorkerLoaderProxy.h

Issue 2254303004: Worker: Stop WorkerLoaderProxy::postTaskToWorkerGlobalScope from returning bool (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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/workers/WorkerLoaderProxy.h
diff --git a/third_party/WebKit/Source/core/workers/WorkerLoaderProxy.h b/third_party/WebKit/Source/core/workers/WorkerLoaderProxy.h
index 225a0aaca80a19c0dd2cb1768dd8813e6b62942b..1614fcc90bfb58155a81ae0ae24160297d9b23a3 100644
--- a/third_party/WebKit/Source/core/workers/WorkerLoaderProxy.h
+++ b/third_party/WebKit/Source/core/workers/WorkerLoaderProxy.h
@@ -64,8 +64,7 @@ public:
virtual void postTaskToLoader(const WebTraceLocation&, std::unique_ptr<ExecutionContextTask>) = 0;
// Posts callbacks from loading code to the WorkerGlobalScope.
- // Returns true if the task was posted successfully.
- virtual bool postTaskToWorkerGlobalScope(const WebTraceLocation&, std::unique_ptr<ExecutionContextTask>) = 0;
+ virtual void postTaskToWorkerGlobalScope(const WebTraceLocation&, std::unique_ptr<ExecutionContextTask>) = 0;
};
class CORE_EXPORT WorkerLoaderProxy final : public ThreadSafeRefCounted<WorkerLoaderProxy> {
@@ -78,7 +77,7 @@ public:
~WorkerLoaderProxy();
void postTaskToLoader(const WebTraceLocation&, std::unique_ptr<ExecutionContextTask>);
- bool postTaskToWorkerGlobalScope(const WebTraceLocation&, std::unique_ptr<ExecutionContextTask>);
+ void postTaskToWorkerGlobalScope(const WebTraceLocation&, std::unique_ptr<ExecutionContextTask>);
// Notification from the provider that it can no longer be
// accessed. An implementation of WorkerLoaderProxyProvider is

Powered by Google App Engine
This is Rietveld 408576698