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

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

Issue 2746783002: [ServiceWorker] Mojofy InstallEvent of Service Worker (Closed)
Patch Set: Just delete a useless line Created 3 years, 8 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/embedded_worker_test_helper.cc
diff --git a/content/browser/service_worker/embedded_worker_test_helper.cc b/content/browser/service_worker/embedded_worker_test_helper.cc
index fd6cdd6632f3764ef6587e2935b9d02792b83daa..6700f0ab42ecd7fa0b18e751d81a124c678f70f5 100644
--- a/content/browser/service_worker/embedded_worker_test_helper.cc
+++ b/content/browser/service_worker/embedded_worker_test_helper.cc
@@ -27,6 +27,7 @@
#include "content/common/background_fetch/background_fetch_types.h"
#include "content/common/service_worker/embedded_worker_messages.h"
#include "content/common/service_worker/embedded_worker_start_params.h"
+#include "content/common/service_worker/service_worker_event_dispatcher.mojom.h"
#include "content/common/service_worker/service_worker_messages.h"
#include "content/common/service_worker/service_worker_utils.h"
#include "content/public/common/push_event_payload.h"
@@ -158,6 +159,14 @@ class EmbeddedWorkerTestHelper::MockServiceWorkerEventDispatcher
~MockServiceWorkerEventDispatcher() override {}
+ void DispatchInstallEvent(
+ mojom::ServiceWorkerInstallEventMethodsAssociatedPtrInfo client,
+ DispatchInstallEventCallback callback) override {
+ if (!helper_)
+ return;
+ helper_->OnInstallEventStub(std::move(client), std::move(callback));
+ }
+
void DispatchActivateEvent(DispatchActivateEventCallback callback) override {
if (!helper_)
return;
@@ -416,20 +425,6 @@ void EmbeddedWorkerTestHelper::OnStopWorker(int embedded_worker_id) {
SimulateWorkerStopped(embedded_worker_id);
}
-bool EmbeddedWorkerTestHelper::OnMessageToWorker(int thread_id,
- int embedded_worker_id,
- const IPC::Message& message) {
- bool handled = true;
- current_embedded_worker_id_ = embedded_worker_id;
- IPC_BEGIN_MESSAGE_MAP(EmbeddedWorkerTestHelper, message)
- IPC_MESSAGE_HANDLER(ServiceWorkerMsg_InstallEvent, OnInstallEventStub)
- IPC_MESSAGE_UNHANDLED(handled = false)
- IPC_END_MESSAGE_MAP()
- // Record all messages directed to inner script context.
- inner_sink_.OnMessageReceived(message);
- return handled;
-}
-
void EmbeddedWorkerTestHelper::OnActivateEvent(
mojom::ServiceWorkerEventDispatcher::DispatchActivateEventCallback
callback) {
@@ -475,14 +470,13 @@ void EmbeddedWorkerTestHelper::OnExtendableMessageEvent(
std::move(callback).Run(SERVICE_WORKER_OK, base::Time::Now());
}
-void EmbeddedWorkerTestHelper::OnInstallEvent(int embedded_worker_id,
- int request_id) {
- // The installing worker may have been doomed and terminated.
- if (!registry()->GetWorker(embedded_worker_id))
- return;
- SimulateSend(new ServiceWorkerHostMsg_InstallEventFinished(
- embedded_worker_id, request_id,
- blink::kWebServiceWorkerEventResultCompleted, true, base::Time::Now()));
+void EmbeddedWorkerTestHelper::OnInstallEvent(
+ mojom::ServiceWorkerInstallEventMethodsAssociatedPtrInfo client,
+ mojom::ServiceWorkerEventDispatcher::DispatchInstallEventCallback
+ callback) {
+ dispatched_events()->push_back(Event::Install);
+ std::move(callback).Run(SERVICE_WORKER_OK, true /* has_fetch_handler */,
+ base::Time::Now());
}
void EmbeddedWorkerTestHelper::OnFetchEvent(
@@ -670,11 +664,6 @@ void EmbeddedWorkerTestHelper::OnMessageToWorkerStub(
EmbeddedWorkerInstance* worker = registry()->GetWorker(embedded_worker_id);
ASSERT_TRUE(worker);
EXPECT_EQ(worker->thread_id(), thread_id);
- base::ThreadTaskRunnerHandle::Get()->PostTask(
- FROM_HERE,
- base::Bind(
- base::IgnoreResult(&EmbeddedWorkerTestHelper::OnMessageToWorker),
- AsWeakPtr(), thread_id, embedded_worker_id, message));
}
void EmbeddedWorkerTestHelper::OnActivateEventStub(
@@ -738,11 +727,14 @@ void EmbeddedWorkerTestHelper::OnExtendableMessageEventStub(
AsWeakPtr(), base::Passed(&event), base::Passed(&callback)));
}
-void EmbeddedWorkerTestHelper::OnInstallEventStub(int request_id) {
+void EmbeddedWorkerTestHelper::OnInstallEventStub(
+ mojom::ServiceWorkerInstallEventMethodsAssociatedPtrInfo client,
+ mojom::ServiceWorkerEventDispatcher::DispatchInstallEventCallback
+ callback) {
base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE,
base::Bind(&EmbeddedWorkerTestHelper::OnInstallEvent, AsWeakPtr(),
- current_embedded_worker_id_, request_id));
+ base::Passed(&client), base::Passed(&callback)));
}
void EmbeddedWorkerTestHelper::OnFetchEventStub(

Powered by Google App Engine
This is Rietveld 408576698