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

Unified Diff: content/browser/service_worker/service_worker_url_request_job_unittest.cc

Issue 2958753003: Revert "Create ServiceWorkerProviderHost before starting worker" (Closed)
Patch Set: Created 3 years, 6 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: content/browser/service_worker/service_worker_url_request_job_unittest.cc
diff --git a/content/browser/service_worker/service_worker_url_request_job_unittest.cc b/content/browser/service_worker/service_worker_url_request_job_unittest.cc
index b6489eb0520a1cbe5a489ae27cb35b9a4ddffa47..54ab7313d3931fbcc209818604b90c93e2486106 100644
--- a/content/browser/service_worker/service_worker_url_request_job_unittest.cc
+++ b/content/browser/service_worker/service_worker_url_request_job_unittest.cc
@@ -439,7 +439,7 @@ class DelayHelper : public EmbeddedWorkerTestHelper {
EmbeddedWorkerTestHelper::OnStartWorker(
embedded_worker_id_, service_worker_version_id_, scope_, script_url_,
pause_after_download_, std::move(start_worker_request_),
- std::move(start_worker_instance_host_), std::move(provider_info_));
+ std::move(start_worker_instance_host_));
}
void Respond() {
@@ -458,16 +458,14 @@ class DelayHelper : public EmbeddedWorkerTestHelper {
}
protected:
- void OnStartWorker(
- int embedded_worker_id,
- int64_t service_worker_version_id,
- const GURL& scope,
- const GURL& script_url,
- bool pause_after_download,
- mojom::ServiceWorkerEventDispatcherRequest request,
- mojom::EmbeddedWorkerInstanceHostAssociatedPtrInfo instance_host,
- mojom::ServiceWorkerProviderInfoForStartWorkerPtr provider_info)
- override {
+ void OnStartWorker(int embedded_worker_id,
+ int64_t service_worker_version_id,
+ const GURL& scope,
+ const GURL& script_url,
+ bool pause_after_download,
+ mojom::ServiceWorkerEventDispatcherRequest request,
+ mojom::EmbeddedWorkerInstanceHostAssociatedPtrInfo
+ instance_host) override {
embedded_worker_id_ = embedded_worker_id;
service_worker_version_id_ = service_worker_version_id;
scope_ = scope;
@@ -475,7 +473,6 @@ class DelayHelper : public EmbeddedWorkerTestHelper {
pause_after_download_ = pause_after_download;
start_worker_request_ = std::move(request);
start_worker_instance_host_ = std::move(instance_host);
- provider_info_ = std::move(provider_info);
}
void OnFetchEvent(
@@ -500,7 +497,6 @@ class DelayHelper : public EmbeddedWorkerTestHelper {
mojom::ServiceWorkerEventDispatcherRequest start_worker_request_;
mojom::EmbeddedWorkerInstanceHostAssociatedPtrInfo
start_worker_instance_host_;
- mojom::ServiceWorkerProviderInfoForStartWorkerPtr provider_info_;
int embedded_worker_id_ = 0;
int fetch_event_id_ = 0;
mojom::ServiceWorkerFetchResponseCallbackPtr response_callback_;
« no previous file with comments | « content/browser/service_worker/service_worker_test_utils.cc ('k') | content/browser/service_worker/service_worker_version.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698