| Index: content/browser/service_worker/service_worker_dispatcher_host_unittest.cc
|
| diff --git a/content/browser/service_worker/service_worker_dispatcher_host_unittest.cc b/content/browser/service_worker/service_worker_dispatcher_host_unittest.cc
|
| index 461316291d07fa2c1121eb007f7e96ee7494a900..376ffa218fa6cd075a66fd4fa14b662340c75310 100644
|
| --- a/content/browser/service_worker/service_worker_dispatcher_host_unittest.cc
|
| +++ b/content/browser/service_worker/service_worker_dispatcher_host_unittest.cc
|
| @@ -39,13 +39,11 @@ static void SaveStatusCallback(bool* called,
|
| *out = status;
|
| }
|
|
|
| -void SetUpDummyMessagePort(std::vector<TransferredMessagePort>* ports) {
|
| +void SetUpDummyMessagePort(std::vector<int>* ports) {
|
| int port_id = -1;
|
| MessagePortService::GetInstance()->Create(MSG_ROUTING_NONE, nullptr,
|
| &port_id);
|
| - TransferredMessagePort dummy_port;
|
| - dummy_port.id = port_id;
|
| - ports->push_back(dummy_port);
|
| + ports->push_back(port_id);
|
| }
|
|
|
| } // namespace
|
| @@ -225,7 +223,7 @@ class ServiceWorkerDispatcherHostTest : public testing::Test {
|
| scoped_refptr<ServiceWorkerVersion> worker,
|
| const base::string16& message,
|
| const url::Origin& source_origin,
|
| - const std::vector<TransferredMessagePort>& sent_message_ports,
|
| + const std::vector<int>& sent_message_ports,
|
| ServiceWorkerProviderHost* sender_provider_host,
|
| const ServiceWorkerDispatcherHost::StatusCallback& callback) {
|
| dispatcher_host_->DispatchExtendableMessageEvent(
|
| @@ -677,23 +675,23 @@ TEST_F(ServiceWorkerDispatcherHostTest, DispatchExtendableMessageEvent) {
|
| const int ref_count = sender_worker_handle->ref_count();
|
|
|
| // Dispatch ExtendableMessageEvent.
|
| - std::vector<TransferredMessagePort> ports;
|
| + std::vector<int> ports;
|
| SetUpDummyMessagePort(&ports);
|
| bool called = false;
|
| ServiceWorkerStatusCode status = SERVICE_WORKER_ERROR_MAX_VALUE;
|
| DispatchExtendableMessageEvent(
|
| version_, base::string16(), url::Origin(version_->scope().GetOrigin()),
|
| ports, provider_host_, base::Bind(&SaveStatusCallback, &called, &status));
|
| - for (TransferredMessagePort port : ports)
|
| - EXPECT_TRUE(MessagePortService::GetInstance()->AreMessagesHeld(port.id));
|
| + for (int port : ports)
|
| + EXPECT_TRUE(MessagePortService::GetInstance()->AreMessagesHeld(port));
|
| EXPECT_EQ(ref_count + 1, sender_worker_handle->ref_count());
|
| base::RunLoop().RunUntilIdle();
|
| EXPECT_TRUE(called);
|
| EXPECT_EQ(SERVICE_WORKER_OK, status);
|
|
|
| // Messages should be held until ports are created at the destination.
|
| - for (TransferredMessagePort port : ports)
|
| - EXPECT_TRUE(MessagePortService::GetInstance()->AreMessagesHeld(port.id));
|
| + for (int port : ports)
|
| + EXPECT_TRUE(MessagePortService::GetInstance()->AreMessagesHeld(port));
|
|
|
| EXPECT_EQ(ref_count + 1, sender_worker_handle->ref_count());
|
| }
|
| @@ -719,23 +717,23 @@ TEST_F(ServiceWorkerDispatcherHostTest, DispatchExtendableMessageEvent_Fail) {
|
|
|
| // Try to dispatch ExtendableMessageEvent. This should fail to start the
|
| // worker and to dispatch the event.
|
| - std::vector<TransferredMessagePort> ports;
|
| + std::vector<int> ports;
|
| SetUpDummyMessagePort(&ports);
|
| bool called = false;
|
| ServiceWorkerStatusCode status = SERVICE_WORKER_ERROR_MAX_VALUE;
|
| DispatchExtendableMessageEvent(
|
| version_, base::string16(), url::Origin(version_->scope().GetOrigin()),
|
| ports, provider_host_, base::Bind(&SaveStatusCallback, &called, &status));
|
| - for (TransferredMessagePort port : ports)
|
| - EXPECT_TRUE(MessagePortService::GetInstance()->AreMessagesHeld(port.id));
|
| + for (int port : ports)
|
| + EXPECT_TRUE(MessagePortService::GetInstance()->AreMessagesHeld(port));
|
| EXPECT_EQ(ref_count + 1, sender_worker_handle->ref_count());
|
| base::RunLoop().RunUntilIdle();
|
| EXPECT_TRUE(called);
|
| EXPECT_EQ(SERVICE_WORKER_ERROR_START_WORKER_FAILED, status);
|
|
|
| // The error callback should clean up the ports and handle.
|
| - for (TransferredMessagePort port : ports)
|
| - EXPECT_FALSE(MessagePortService::GetInstance()->AreMessagesHeld(port.id));
|
| + for (int port : ports)
|
| + EXPECT_FALSE(MessagePortService::GetInstance()->AreMessagesHeld(port));
|
| EXPECT_EQ(ref_count, sender_worker_handle->ref_count());
|
| }
|
|
|
|
|