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

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

Issue 2490623005: Remove InterfaceRegistry/Provider from service workers (Closed)
Patch Set: Created 4 years, 1 month 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_job_unittest.cc
diff --git a/content/browser/service_worker/service_worker_job_unittest.cc b/content/browser/service_worker/service_worker_job_unittest.cc
index 5ea9d11a0724baac9a6ef88b1157dee414d3094b..77be6ef3c7a18b3f8b816d5a15ff9029aee6770f 100644
--- a/content/browser/service_worker/service_worker_job_unittest.cc
+++ b/content/browser/service_worker/service_worker_job_unittest.cc
@@ -1624,15 +1624,13 @@ class CheckPauseAfterDownloadEmbeddedWorkerInstanceClient
protected:
void StartWorker(
- const EmbeddedWorkerStartParams& params,
- service_manager::mojom::InterfaceProviderPtr browser_interfaces,
- service_manager::mojom::InterfaceProviderRequest renderer_request)
- override {
+ const EmbeddedWorkerStartParamsInternal& params,
+ mojom::ServiceWorkerEventDispatcherRequest request) override {
ASSERT_TRUE(next_pause_after_download_.has_value());
EXPECT_EQ(next_pause_after_download_.value(), params.pause_after_download);
num_of_startworker_++;
EmbeddedWorkerTestHelper::MockEmbeddedWorkerInstanceClient::StartWorker(
- params, std::move(browser_interfaces), std::move(renderer_request));
+ params, std::move(request));
}
private:
@@ -1669,7 +1667,7 @@ TEST_P(ServiceWorkerJobTestP, Update_PauseAfterDownload) {
ASSERT_TRUE(start_msg);
EmbeddedWorkerMsg_StartWorker::Param param;
EmbeddedWorkerMsg_StartWorker::Read(start_msg, &param);
- const EmbeddedWorkerStartParams& start_params = std::get<0>(param);
+ const EmbeddedWorkerStartParamsInternal& start_params = std::get<0>(param);
EXPECT_FALSE(start_params.pause_after_download);
sink->ClearMessages();
}
@@ -1688,7 +1686,7 @@ TEST_P(ServiceWorkerJobTestP, Update_PauseAfterDownload) {
ASSERT_TRUE(start_msg);
EmbeddedWorkerMsg_StartWorker::Param param;
EmbeddedWorkerMsg_StartWorker::Read(start_msg, &param);
- const EmbeddedWorkerStartParams& start_params = std::get<0>(param);
+ const EmbeddedWorkerStartParamsInternal& start_params = std::get<0>(param);
EXPECT_TRUE(start_params.pause_after_download);
sink->ClearMessages();
}

Powered by Google App Engine
This is Rietveld 408576698