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

Unified Diff: third_party/WebKit/Source/core/workers/ThreadedMessagingProxyBase.cpp

Issue 2840473002: Worker: Remove WorkerLoaderProxy for clean-up (Closed)
Patch Set: clean up Created 3 years, 7 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/ThreadedMessagingProxyBase.cpp
diff --git a/third_party/WebKit/Source/core/workers/ThreadedMessagingProxyBase.cpp b/third_party/WebKit/Source/core/workers/ThreadedMessagingProxyBase.cpp
index e1227217634544c956fce4a720bfcaf73bcb4b5c..beed666370c999ea3b9288aff5af1bc9303aad40 100644
--- a/third_party/WebKit/Source/core/workers/ThreadedMessagingProxyBase.cpp
+++ b/third_party/WebKit/Source/core/workers/ThreadedMessagingProxyBase.cpp
@@ -36,8 +36,6 @@ ThreadedMessagingProxyBase::ThreadedMessagingProxyBase(
ThreadedMessagingProxyBase::~ThreadedMessagingProxyBase() {
DCHECK(IsParentContextThread());
- if (loader_proxy_)
- loader_proxy_->DetachProvider(this);
g_live_messaging_proxy_count--;
}
@@ -61,7 +59,6 @@ void ThreadedMessagingProxyBase::InitializeWorkerThread(
? document->Loader()->GetTiming().ReferenceMonotonicTime()
: MonotonicallyIncreasingTime();
- loader_proxy_ = WorkerLoaderProxy::Create(this);
worker_thread_ = CreateWorkerThread(origin_time);
worker_thread_->Start(std::move(startup_data), GetParentFrameTaskRunners());
WorkerThreadCreated();
@@ -70,11 +67,7 @@ void ThreadedMessagingProxyBase::InitializeWorkerThread(
ThreadableLoadingContext*
ThreadedMessagingProxyBase::GetThreadableLoadingContext() {
DCHECK(IsParentContextThread());
- if (!loading_context_) {
- loading_context_ =
- ThreadableLoadingContext::Create(*ToDocument(execution_context_));
- }
- return loading_context_;
+ return ThreadableLoadingContext::Create(*ToDocument(execution_context_));
}
void ThreadedMessagingProxyBase::CountFeature(UseCounter::Feature feature) {

Powered by Google App Engine
This is Rietveld 408576698