First step at synchronizing client model changes with service.

mojo/services/view_manager/view_manager_connection_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 "mojo/services/view_manager/view_manager_connection.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 42 matching lines @@
   return NodeIdToTransportId(NodeId(connection_id, node_id));
 }
 
 // Creates an id used for transport from the specified parameters.
 uint32_t CreateViewId(uint16_t connection_id, uint16_t view_id) {
   return ViewIdToTransportId(ViewId(connection_id, view_id));
 }
 
 // Creates a node with the specified id. Returns true on success. Blocks until
 // we get back result from server.
-bool CreateNode(ViewManager* view_manager, uint16_t id) {
+bool CreateNode(IViewManager* view_manager, uint16_t id) {
   bool result = false;
   view_manager->CreateNode(id, base::Bind(&BooleanCallback, &result));
   DoRunLoop();
   return result;
 }
 
 // TODO(sky): make a macro for these functions, they are all the same.
 
 // Deletes a node, blocking until done.
-bool DeleteNode(ViewManager* view_manager,
+bool DeleteNode(IViewManager* view_manager,
                 uint32_t node_id,
                 int32_t change_id) {
   bool result = false;
   view_manager->DeleteNode(node_id, change_id,
                            base::Bind(&BooleanCallback, &result));
   DoRunLoop();
   return result;
 }
 
 // Adds a node, blocking until done.
-bool AddNode(ViewManager* view_manager,
+bool AddNode(IViewManager* view_manager,
              uint32_t parent,
              uint32_t child,
              int32_t change_id) {
   bool result = false;
   view_manager->AddNode(parent, child, change_id,
                         base::Bind(&BooleanCallback, &result));
   DoRunLoop();
   return result;
 }
 
 // Removes a node, blocking until done.
-bool RemoveNodeFromParent(ViewManager* view_manager,
+bool RemoveNodeFromParent(IViewManager* view_manager,
                           uint32_t node_id,
                           int32_t change_id) {
   bool result = false;
   view_manager->RemoveNodeFromParent(node_id, change_id,
                                      base::Bind(&BooleanCallback, &result));
   DoRunLoop();
   return result;
 }
 
 // Creates a view with the specified id. Returns true on success. Blocks until
 // we get back result from server.
-bool CreateView(ViewManager* view_manager, uint16_t id) {
+bool CreateView(IViewManager* view_manager, uint16_t id) {
   bool result = false;
   view_manager->CreateView(id, base::Bind(&BooleanCallback, &result));
   DoRunLoop();
   return result;
 }
 
 // Sets a view on the specified node. Returns true on success. Blocks until we
 // get back result from server.
-bool SetView(ViewManager* view_manager,
+bool SetView(IViewManager* view_manager,
              uint32_t node_id,
              uint32_t view_id,
              int32_t change_id) {
   bool result = false;
   view_manager->SetView(node_id, view_id, change_id,
                         base::Bind(&BooleanCallback, &result));
   DoRunLoop();
   return result;
 }
 
 }  // namespace
 
 typedef std::vector<std::string> Changes;
 
-class ViewManagerClientImpl : public ViewManagerClient {
+class ViewManagerClientImpl : public IViewManagerClient {
  public:
   ViewManagerClientImpl() : id_(0), quit_count_(0) {}
 
   void set_quit_count(int count) { quit_count_ = count; }
 
   uint16_t id() const { return id_; }
 
   Changes GetAndClearChanges() {
     Changes changes;
     changes.swap(changes_);
     return changes;
   }
 
  private:
-  // ViewManagerClient overrides:
+  // IViewManagerClient overrides:
   virtual void OnConnectionEstablished(uint16_t connection_id) OVERRIDE {
     id_ = connection_id;
     current_run_loop->Quit();
   }
   virtual void OnNodeHierarchyChanged(uint32_t node,
                                       uint32_t new_parent,
                                       uint32_t old_parent,
                                       int32_t change_id) OVERRIDE {
     changes_.push_back(
         base::StringPrintf(
@@ skipping 29 matching lines @@
   Changes changes_;
 
   DISALLOW_COPY_AND_ASSIGN(ViewManagerClientImpl);
 };
 
 class ViewManagerConnectionTest : public testing::Test {
  public:
   ViewManagerConnectionTest() : service_factory_(&root_node_manager_) {}
 
   virtual void SetUp() OVERRIDE {
-    InterfacePipe<ViewManagerClient, ViewManager> pipe;
+    InterfacePipe<IViewManagerClient, IViewManager> pipe;
     view_manager_.reset(pipe.handle_to_peer.Pass(), &client_);
     connection_.Initialize(
         &service_factory_,
         ScopedMessagePipeHandle::From(pipe.handle_to_self.Pass()));
     // Wait for the id.
     DoRunLoop();
   }
 
  protected:
   // Creates a second connection to the viewmanager.
   void EstablishSecondConnection() {
     connection2_.reset(new ViewManagerConnection);
-    InterfacePipe<ViewManagerClient, ViewManager> pipe;
+    InterfacePipe<IViewManagerClient, IViewManager> pipe;
     view_manager2_.reset(pipe.handle_to_peer.Pass(), &client2_);
     connection2_->Initialize(
         &service_factory_,
         ScopedMessagePipeHandle::From(pipe.handle_to_self.Pass()));
     // Wait for the id.
     DoRunLoop();
   }
 
   void DestroySecondConnection() {
     connection2_.reset();
     view_manager2_.reset();
   }
 
   Environment env_;
   base::MessageLoop loop_;
   RootNodeManager root_node_manager_;
   ServiceConnector<ViewManagerConnection, RootNodeManager> service_factory_;
   ViewManagerConnection connection_;
   ViewManagerClientImpl client_;
-  RemotePtr<ViewManager> view_manager_;
+  RemotePtr<IViewManager> view_manager_;
 
   ViewManagerClientImpl client2_;
-  RemotePtr<ViewManager> view_manager2_;
+  RemotePtr<IViewManager> view_manager2_;
   scoped_ptr<ViewManagerConnection> connection2_;
 
   DISALLOW_COPY_AND_ASSIGN(ViewManagerConnectionTest);
 };
 
 // Verifies client gets a valid id.
 TEST_F(ViewManagerConnectionTest, ValidId) {
   // All these tests assume 1 for the client id. The only real assertion here is
   // the client id is not zero, but adding this as rest of code here assumes 1.
   EXPECT_EQ(1, client_.id());
@@ skipping 273 matching lines @@
 
     Changes changes(client_.GetAndClearChanges());
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ("change_id=0 node=1,1 new_view=null old_view=2,51", changes[0]);
   }
 }
 
 }  // namespace view_manager
 }  // namespace services
 }  // namespace mojo