[ServiceWorker] Add EmbeddedWorkerInstanceHost Interface.

content/browser/service_worker/service_worker_context_unittest.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/public/browser/service_worker_context.h"
 
 #include <stdint.h>
 
 #include "base/files/scoped_temp_dir.h"
 #include "base/logging.h"
@@ skipping 158 matching lines @@
  public:
   enum class Message { StartWorker, StopWorker };
 
   explicit RecordableEmbeddedWorkerInstanceClient(
       base::WeakPtr<EmbeddedWorkerTestHelper> helper)
       : EmbeddedWorkerTestHelper::MockEmbeddedWorkerInstanceClient(helper) {}
 
   const std::vector<Message>& events() const { return events_; }
 
  protected:
-  void StartWorker(
-      const EmbeddedWorkerStartParams& params,
-      mojom::ServiceWorkerEventDispatcherRequest request) override {
+  void StartWorker(const EmbeddedWorkerStartParams& params,
+                   mojom::ServiceWorkerEventDispatcherRequest request,
+                   mojom::EmbeddedWorkerInstanceHostAssociatedPtrInfo
+                       instance_host) override {
     events_.push_back(Message::StartWorker);
     EmbeddedWorkerTestHelper::MockEmbeddedWorkerInstanceClient::StartWorker(
-        params, std::move(request));
+        params, std::move(request), std::move(instance_host));
   }
 
   void StopWorker(const StopWorkerCallback& callback) override {
     events_.push_back(Message::StopWorker);
     EmbeddedWorkerTestHelper::MockEmbeddedWorkerInstanceClient::StopWorker(
         std::move(callback));
   }
 
   std::vector<Message> events_;
 
@@ skipping 274 matching lines @@
       registration_id4,
       origin3_p1.GetOrigin(),
       base::Bind(&ExpectRegisteredWorkers,
                  SERVICE_WORKER_OK,
                  false /* expect_waiting */,
                  true /* expect_active */));
 
   base::RunLoop().RunUntilIdle();
 
   ASSERT_EQ(6u, notifications_.size());
+  // TODO(leonhsl) in this CL: Why the 4 REGISTRATION_STORED notifications are
+  // in inverse order with the 4 RegisterServiceWorker calls.

[leonhsl(Using Gerrit), 2017/04/04 07:14:04]

I'm still investigating,, it would be very helpful if you have some insights
about this ;-)

[shimazu, 2017/04/06 05:01:36]

What was the cause of this?

[leonhsl(Using Gerrit), 2017/04/06 09:58:55]

The root cause is that I was using base::MessageLoop::ScopedNestableTaskAllower
in EmbeddedWorkerTestHelper::SimulateWorker*() functions to try to execute all
EWInstanceHost interface functions synchronously.

Then, for this test case, the following scenario comes out:
  EmbeddedWorkerTestHelper::OnStartWorker(0)
  --> SimulateWorkerReadyForInspection(0) here will try to execute all tasks in
current message loop, because EmbeddedWorkerTestHelper::OnStartWorker({1, 2, 3})
for other 3 registrations have been already queued there, they will get
executed, inside this SimulateWorkerReadyForInspection(0).
      --> EmbeddedWorkerTestHelper::OnStartWorker(1)
          --> EmbeddedWorkerTestHelper::OnStartWorker(2)
              --> EmbeddedWorkerTestHelper::OnStartWorker(3)
                  --> OnStarted(3)
              --> OnStarted(2)
          --> OnStarted(1)
      --> OnStarted(0)

[shimazu, 2017/04/10 05:10:10]

Ah, thanks, I got it.

+  EXPECT_EQ(REGISTRATION_STORED, notifications_[3].type);
+  EXPECT_EQ(registration_id1, notifications_[3].registration_id);
+  EXPECT_EQ(origin1_p1, notifications_[3].pattern);
+  EXPECT_EQ(REGISTRATION_STORED, notifications_[2].type);
+  EXPECT_EQ(origin1_p2, notifications_[2].pattern);
+  EXPECT_EQ(registration_id2, notifications_[2].registration_id);
+  EXPECT_EQ(REGISTRATION_STORED, notifications_[1].type);
+  EXPECT_EQ(origin2_p1, notifications_[1].pattern);
+  EXPECT_EQ(registration_id3, notifications_[1].registration_id);
   EXPECT_EQ(REGISTRATION_STORED, notifications_[0].type);
-  EXPECT_EQ(registration_id1, notifications_[0].registration_id);
-  EXPECT_EQ(origin1_p1, notifications_[0].pattern);
-  EXPECT_EQ(REGISTRATION_STORED, notifications_[1].type);
-  EXPECT_EQ(origin1_p2, notifications_[1].pattern);
-  EXPECT_EQ(registration_id2, notifications_[1].registration_id);
-  EXPECT_EQ(REGISTRATION_STORED, notifications_[2].type);
-  EXPECT_EQ(origin2_p1, notifications_[2].pattern);
-  EXPECT_EQ(registration_id3, notifications_[2].registration_id);
-  EXPECT_EQ(REGISTRATION_STORED, notifications_[3].type);
-  EXPECT_EQ(origin3_p1, notifications_[3].pattern);
-  EXPECT_EQ(registration_id4, notifications_[3].registration_id);
+  EXPECT_EQ(origin3_p1, notifications_[0].pattern);
+  EXPECT_EQ(registration_id4, notifications_[0].registration_id);
+
   EXPECT_EQ(REGISTRATION_DELETED, notifications_[4].type);
   EXPECT_EQ(origin1_p2, notifications_[4].pattern);
   EXPECT_EQ(registration_id2, notifications_[4].registration_id);
   EXPECT_EQ(REGISTRATION_DELETED, notifications_[5].type);
   EXPECT_EQ(origin1_p1, notifications_[5].pattern);
   EXPECT_EQ(registration_id1, notifications_[5].registration_id);
 }
 
 // Make sure registering a new script shares an existing registration.
 TEST_F(ServiceWorkerContextTest, RegisterNewScript) {
@@ skipping 260 matching lines @@
   EXPECT_EQ(REGISTRATION_STORED, notifications_[2].type);
   EXPECT_EQ(pattern, notifications_[2].pattern);
   EXPECT_EQ(registration_id, notifications_[2].registration_id);
 }
 
 INSTANTIATE_TEST_CASE_P(ServiceWorkerContextRecoveryTest,
                         ServiceWorkerContextRecoveryTest,
                         testing::Bool());
 
 }  // namespace content