Mojo: more idiomatic C++ bindings

mojo/service_manager/service_manager_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/cpp/bindings/allocation_scope.h"
 #include "mojo/public/cpp/environment/environment.h"
 #include "mojo/public/cpp/shell/application.h"
 #include "mojo/public/interfaces/shell/shell.mojom.h"
 #include "mojo/service_manager/service_loader.h"
 #include "mojo/service_manager/service_manager.h"
 #include "mojo/service_manager/test.mojom.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace mojo {
 namespace {
@@ skipping 15 matching lines @@
 
   virtual ~TestServiceImpl() {
     --context_->num_impls;
   }
 
   // InterfaceImpl<TestService> implementation.
   virtual void OnConnectionError() OVERRIDE {
   }
 
   // TestService implementation:
-  virtual void Test(const mojo::String& test_string) OVERRIDE {
+  virtual void Test(mojo::String test_string) OVERRIDE {
     context_->last_test_string = test_string.To<std::string>();
     client()->AckTest();
   }
 
  private:
   TestContext* context_;
 };
 
 class TestClientImpl : public TestClient {
  public:
   explicit TestClientImpl(TestServicePtr service)
       : service_(service.Pass()),
         quit_after_ack_(false) {
     service_->SetClient(this);
   }
 
   virtual ~TestClientImpl() {}
 
   virtual void AckTest() OVERRIDE {
     if (quit_after_ack_)
       base::MessageLoop::current()->Quit();
   }
 
   void Test(std::string test_string) {
-    AllocationScope scope;
     quit_after_ack_ = true;
-    service_->Test(test_string);
+    service_->Test(String::From(test_string));
   }
 
  private:
   TestServicePtr service_;
   bool quit_after_ack_;
   DISALLOW_COPY_AND_ASSIGN(TestClientImpl);
 };
 
 class TestServiceLoader : public ServiceLoader {
  public:
@@ skipping 194 matching lines @@
 
   std::string url("test:test3");
   MessagePipe pipe1;
   sm.Connect(GURL(url), pipe1.handle0.Pass());
   EXPECT_EQ(1, interceptor.call_count());
   EXPECT_EQ(url, interceptor.url_spec());
   EXPECT_EQ(1, default_loader->num_loads());
 }
 
 }  // namespace mojo