Modify service_connector_unitttest to use Service<>

mojo/shell/service_connector_unittest.cc

 // Copyright 2014 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 "base/message_loop/message_loop.h"
 #include "mojo/public/bindings/allocation_scope.h"
 #include "mojo/public/bindings/remote_ptr.h"
+#include "mojo/public/shell/service.h"
 #include "mojo/shell/service_connector.h"
 #include "mojom/shell.h"
 #include "mojom/test.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace mojo {
 namespace shell {
 namespace {
 
-class TestApp : public ShellClient {
+struct Context {
+  std::string last_test_string;
+};
+
+class TestServiceImpl : public Service<TestService, TestServiceImpl, Context> {
  public:
-  TestApp(ScopedMessagePipeHandle shell_handle)
-      : shell_(shell_handle.Pass(), this) {
+  virtual void Test(const mojo::String& test_string) OVERRIDE {
+    context()->last_test_string = test_string.To<std::string>();
+    client()->AckTest();
   }
-  virtual ~TestApp() {
-  }
-  virtual void AcceptConnection(ScopedMessagePipeHandle client_handle)
-      MOJO_OVERRIDE {
-    service_.reset(new TestServiceImpl(this, client_handle.Pass()));
-  }
-  std::string GetLastTestString() {
-    return service_->last_test_string_;
-  }
-
- private:
-  class TestServiceImpl : public TestService {
-   public:
-    TestServiceImpl(TestApp* service, ScopedMessagePipeHandle client_handle)
-        : service_(service),
-          client_(client_handle.Pass(), this) {
-    }
-    virtual ~TestServiceImpl() {
-    }
-    virtual void Test(const mojo::String& test_string) OVERRIDE {
-      last_test_string_ = test_string.To<std::string>();
-      client_->AckTest();
-    }
-    TestApp* service_;
-    RemotePtr<TestClient> client_;
-    std::string last_test_string_;
-  };
-  RemotePtr<Shell> shell_;
-  scoped_ptr<TestServiceImpl> service_;
 };
 
 class TestClientImpl : public TestClient {
  public:
   explicit TestClientImpl(ScopedMessagePipeHandle service_handle)
       : service_(service_handle.Pass(), this),
         quit_after_ack_(false) {
   }
   virtual ~TestClientImpl() {
   }
@@ skipping 29 matching lines @@
   }
 
   virtual void TearDown() OVERRIDE {
     test_client_.reset(NULL);
     test_app_.reset(NULL);
     service_connector_.reset(NULL);
   }
 
   virtual void Load(const GURL& url,
                     ScopedMessagePipeHandle shell_handle) OVERRIDE {
-    test_app_.reset(new TestApp(shell_handle.Pass()));
+    test_app_.reset(new ServiceFactory<TestServiceImpl, Context>(
+        shell_handle.Pass(), &context_));
   }
 
  protected:
   base::MessageLoop loop_;
-  scoped_ptr<TestApp> test_app_;
+  Context context_;
+  scoped_ptr<ServiceFactory<TestServiceImpl, Context>> test_app_;

[darin (slow to review), 2014/02/03 21:02:16]

nit: I think some compilers will complain about the ">>" here.

   scoped_ptr<TestClientImpl> test_client_;
   scoped_ptr<ServiceConnector> service_connector_;
   DISALLOW_COPY_AND_ASSIGN(ServiceConnectorTest);
 };
 
 TEST_F(ServiceConnectorTest, Basic) {
   test_client_->Test("test");
   loop_.Run();
-  EXPECT_EQ(std::string("test"), test_app_->GetLastTestString());
+  EXPECT_EQ(std::string("test"), context_.last_test_string);
 }
 
 }  // namespace
 }  // namespace shell
 }  // namespace mojo