Commit 3e6aeeec authored by Kenichi Ishibashi's avatar Kenichi Ishibashi Committed by Commit Bot

Remove WebServiceWorkerContextClient::WorkerScriptLoadedOnInitiatorThread()

This method was used legacy on-the-main-thread script fetch which was
removed a while ago. Remove this leftover.

Bug: 967272
Change-Id: Ia57e7ec143bf08a245bc9ae83ac5522604344081
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1771101Reviewed-by: default avatarHiroki Nakagawa <nhiroki@chromium.org>
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#690272}
parent dc4e9e38
...@@ -233,13 +233,6 @@ void ServiceWorkerContextClient::FailedToFetchModuleScript() { ...@@ -233,13 +233,6 @@ void ServiceWorkerContextClient::FailedToFetchModuleScript() {
// eventually destroys |this|. // eventually destroys |this|.
} }
void ServiceWorkerContextClient::WorkerScriptLoadedOnInitiatorThread() {
DCHECK(initiator_thread_task_runner_->RunsTasksInCurrentSequence());
DCHECK(!is_starting_installed_worker_);
instance_host_->OnScriptLoaded();
TRACE_EVENT_NESTABLE_ASYNC_END0("ServiceWorker", "LOAD_SCRIPT", this);
}
void ServiceWorkerContextClient::WorkerScriptLoadedOnWorkerThread() { void ServiceWorkerContextClient::WorkerScriptLoadedOnWorkerThread() {
DCHECK(worker_task_runner_->RunsTasksInCurrentSequence()); DCHECK(worker_task_runner_->RunsTasksInCurrentSequence());
instance_host_->OnScriptLoaded(); instance_host_->OnScriptLoaded();
......
...@@ -124,7 +124,6 @@ class CONTENT_EXPORT ServiceWorkerContextClient ...@@ -124,7 +124,6 @@ class CONTENT_EXPORT ServiceWorkerContextClient
void WorkerContextFailedToStartOnInitiatorThread() override; void WorkerContextFailedToStartOnInitiatorThread() override;
void FailedToLoadClassicScript() override; void FailedToLoadClassicScript() override;
void FailedToFetchModuleScript() override; void FailedToFetchModuleScript() override;
void WorkerScriptLoadedOnInitiatorThread() override;
void WorkerScriptLoadedOnWorkerThread() override; void WorkerScriptLoadedOnWorkerThread() override;
void WorkerContextStarted( void WorkerContextStarted(
blink::WebServiceWorkerContextProxy* proxy, blink::WebServiceWorkerContextProxy* proxy,
......
...@@ -95,13 +95,6 @@ class WebServiceWorkerContextClient { ...@@ -95,13 +95,6 @@ class WebServiceWorkerContextClient {
// failed. // failed.
virtual void FailedToFetchModuleScript() {} virtual void FailedToFetchModuleScript() {}
// The worker script was successfully loaded by ResourceLoader. Called on the
// initiator thread.
//
// This is called before WorkerContextStarted(). Script evaluation does not
// start until WillEvaluateScript().
virtual void WorkerScriptLoadedOnInitiatorThread() {}
// The worker script was successfully loaded on the worker thread. // The worker script was successfully loaded on the worker thread.
// When off-the-main-thread script fetch is on, this is called for both // When off-the-main-thread script fetch is on, this is called for both
// new-script and installed-script cases. If off-the-main-thread script fetch // new-script and installed-script cases. If off-the-main-thread script fetch
......
...@@ -128,7 +128,6 @@ class MockServiceWorkerContextClient final ...@@ -128,7 +128,6 @@ class MockServiceWorkerContextClient final
void(mojo::ScopedMessagePipeHandle, void(mojo::ScopedMessagePipeHandle,
mojo::ScopedMessagePipeHandle)); mojo::ScopedMessagePipeHandle));
MOCK_METHOD0(WorkerContextFailedToStartOnInitiatorThread, void()); MOCK_METHOD0(WorkerContextFailedToStartOnInitiatorThread, void());
MOCK_METHOD0(WorkerScriptLoadedOnInitiatorThread, void());
void WorkerContextStarted(WebServiceWorkerContextProxy* proxy, void WorkerContextStarted(WebServiceWorkerContextProxy* proxy,
scoped_refptr<base::SequencedTaskRunner>) override { scoped_refptr<base::SequencedTaskRunner>) override {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment