Add mojo::Binding<Interface> for more flexible pipe<->impl binding

mojo/services/view_manager/view_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 <string>
 #include <vector>
 
 #include "base/at_exit.h"
 #include "base/auto_reset.h"
 #include "base/bind.h"
@@ skipping 333 matching lines @@
     : public InterfaceImpl<ViewManagerClient> {
  public:
   TestViewManagerClientConnection() : proxy_(&tracker_) {
     tracker_.set_delegate(&proxy_);
   }
 
   TestChangeTracker* tracker() { return &tracker_; }
 
   ViewManagerProxy* proxy() { return &proxy_; }
 
-  // InterfaceImpl:
-  void OnConnectionEstablished() override {
-    proxy()->set_router(internal_state()->router());
-    proxy()->set_view_manager(client());
+  void OnConnectionEstablished() {
+    proxy_.set_router(internal_router());
+    proxy_.set_view_manager(client());
   }
 
   // ViewManagerClient:
   void OnEmbed(ConnectionSpecificId connection_id,
                const String& creator_url,
                ViewDataPtr root,
                InterfaceRequest<ServiceProvider> services,
                ScopedMessagePipeHandle window_manager_pipe) override {
     tracker()->OnEmbed(connection_id, creator_url, root.Pass());
   }
@@ skipping 68 matching lines @@
 
   // ApplicationDelegate implementation:
   bool ConfigureIncomingConnection(ApplicationConnection* connection) override {
     connection->AddService<ViewManagerClient>(this);
     return true;
   }
 
   // InterfaceFactory<ViewManagerClient> implementation:
   void Create(ApplicationConnection* connection,
               InterfaceRequest<ViewManagerClient> request) override {
-    BindToRequest(new TestViewManagerClientConnection(), &request);
+    auto client_connection = new TestViewManagerClientConnection;
+    BindToRequest(client_connection, &request);
+    client_connection->OnConnectionEstablished();
   }
 
  private:
   ScopedVector<ApplicationImpl> apps_;
 
   DISALLOW_COPY_AND_ASSIGN(EmbedApplicationLoader);
 };
 
 class TestWindowManagerImpl : public InterfaceImpl<WindowManager> {
  public:
@@ skipping 11 matching lines @@
         window_manager_client_.get());
     view_manager_client_->proxy()->set_view_manager(view_manager_.get());
   }
 
   virtual ~TestWindowManagerImpl() {}
 
   TestViewManagerClientConnection* view_manager_client() {
     return view_manager_client_;
   }
 
-  // InterfaceImpl:
-  virtual void OnConnectionEstablished() override {}
-
   // WindowManager:
   void Embed(const String& url,
              InterfaceRequest<ServiceProvider> service_provider) override {
     if (!got_initial_embed_) {
       got_initial_embed_ = true;
       return;
     }
     view_manager_client_->tracker()->DelegateEmbed(url);
   }
   void SetCapture(Id view, const Callback<void(bool)>& callback) override {
@@ skipping 1111 matching lines @@
     EXPECT_EQ("PropertyChanged view=1,1 key=one value=NULL",
               ChangesToDescription1(connection2_->changes())[0]);
   }
 }
 
 // WARNING: this class is deprecated and will be replaced with
 // view_manager_server_apptest soonish. Add new tests there.
 
 }  // namespace service
 }  // namespace mojo