Mojo: more idiomatic C++ bindings

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 <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/scoped_vector.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
 #include "mojo/common/common_type_converters.h"
 #include "mojo/public/cpp/application/application.h"
 #include "mojo/public/cpp/application/connect.h"
-#include "mojo/public/cpp/bindings/allocation_scope.h"
 #include "mojo/public/cpp/environment/environment.h"
 #include "mojo/service_manager/service_manager.h"
 #include "mojo/services/public/cpp/geometry/geometry_type_converters.h"
 #include "mojo/services/public/cpp/view_manager/util.h"
 #include "mojo/services/public/cpp/view_manager/view_manager_types.h"
 #include "mojo/services/public/interfaces/view_manager/view_manager.mojom.h"
 #include "mojo/services/view_manager/test_change_tracker.h"
 #include "mojo/shell/shell_test_helper.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gfx/geometry/rect.h"
 
 namespace mojo {
-
-// TODO(sky): remove this when Darin is done with cleanup.
-template <typename T>
-class MOJO_COMMON_EXPORT TypeConverter<T, T> {
- public:
-  static T ConvertFrom(T input, Buffer* buf) {
-    return input;
-  }
-  static T ConvertTo(T input) {
-    return input;
-  }
-
-  MOJO_ALLOW_IMPLICIT_TYPE_CONVERSION();
-};
-
 namespace view_manager {
 namespace service {
 
 namespace {
 
 base::RunLoop* current_run_loop = NULL;
 
 const char kTestServiceURL[] = "mojo:test_url";
 
-void INodesToTestNodes(const Array<INode>& data,
+void INodesToTestNodes(const Array<INodePtr>& data,
                        std::vector<TestNode>* test_nodes) {
   for (size_t i = 0; i < data.size(); ++i) {
     TestNode node;
-    node.parent_id = data[i].parent_id();
-    node.node_id = data[i].node_id();
-    node.view_id = data[i].view_id();
+    node.parent_id = data[i]->parent_id;
+    node.node_id = data[i]->node_id;
+    node.view_id = data[i]->view_id;
     test_nodes->push_back(node);
   }
 }
 
 // BackgroundConnection is used when a child ViewManagerConnection is created by
 // way of Connect(). BackgroundConnection is created on a background thread (the
 // thread where the shell lives). All public methods are to be invoked on the
 // main thread. They wait for a result (by spinning a nested message loop),
 // calling through to background thread, then the underlying IViewManager* and
 // respond back to the calling thread to return control to the test.
@@ skipping 141 matching lines @@
   // Callbacks from the various IViewManager functions.
   void GotResultOnBackgroundThread(bool* result_cache, bool result) {
     *result_cache = result;
     main_loop_->PostTask(
         FROM_HERE,
         base::Bind(&BackgroundConnection::QuitOnMainThread,
                    base::Unretained(this)));
   }
 
   void GotNodeTreeOnBackgroundThread(std::vector<TestNode>* nodes,
-                                     const Array<INode>& results) {
+                                     Array<INodePtr> results) {
     INodesToTestNodes(results, nodes);
     main_loop_->PostTask(
         FROM_HERE,
         base::Bind(&BackgroundConnection::QuitOnMainThread,
                    base::Unretained(this)));
   }
 
   void QuitOnMainThread() {
     DCHECK(main_run_loop_);
     main_run_loop_->Quit();
@@ skipping 86 matching lines @@
 
   // InterfaceImp:
   virtual void OnConnectionEstablished() OVERRIDE {
     connection_.set_view_manager(client());
   }
 
   // IViewMangerClient:
   virtual void OnViewManagerConnectionEstablished(
       TransportConnectionId connection_id,
       TransportChangeId next_server_change_id,
-      const Array<INode>& nodes) OVERRIDE {
+      Array<INodePtr> nodes) OVERRIDE {
     tracker_.OnViewManagerConnectionEstablished(
-        connection_id, next_server_change_id, nodes);
+        connection_id, next_server_change_id, nodes.Pass());
   }
   virtual void OnServerChangeIdAdvanced(
       TransportChangeId next_server_change_id) OVERRIDE {
     tracker_.OnServerChangeIdAdvanced(next_server_change_id);
   }
   virtual void OnNodeBoundsChanged(TransportNodeId node_id,
-                                   const Rect& old_bounds,
-                                   const Rect& new_bounds) OVERRIDE {
-    tracker_.OnNodeBoundsChanged(node_id, old_bounds, new_bounds);
+                                   RectPtr old_bounds,
+                                   RectPtr new_bounds) OVERRIDE {
+    tracker_.OnNodeBoundsChanged(node_id, old_bounds.Pass(), new_bounds.Pass());
   }
   virtual void OnNodeHierarchyChanged(
       TransportNodeId node,
       TransportNodeId new_parent,
       TransportNodeId old_parent,
       TransportChangeId server_change_id,
-      const Array<INode>& nodes) OVERRIDE {
+      Array<INodePtr> nodes) OVERRIDE {
     tracker_.OnNodeHierarchyChanged(node, new_parent, old_parent,
-                                     server_change_id, nodes);
+                                     server_change_id, nodes.Pass());
   }
   virtual void OnNodeDeleted(TransportNodeId node,
                              TransportChangeId server_change_id) OVERRIDE {
     tracker_.OnNodeDeleted(node, server_change_id);
   }
   virtual void OnViewDeleted(TransportViewId view) OVERRIDE {
     tracker_.OnViewDeleted(view);
   }
   virtual void OnNodeViewReplaced(TransportNodeId node,
                                   TransportViewId new_view_id,
@@ skipping 50 matching lines @@
 // Boolean callback. Sets |result_cache| to the value of |result| and quits
 // the run loop.
 void BooleanCallback(bool* result_cache, bool result) {
   *result_cache = result;
   current_run_loop->Quit();
 }
 
 // Callback that results in a vector of INodes. The INodes are converted to
 // TestNodes.
 void INodesCallback(std::vector<TestNode>* test_nodes,
-                    const Array<INode>& data) {
+                    Array<INodePtr> data) {
   INodesToTestNodes(data, test_nodes);
   current_run_loop->Quit();
 }
 
 // Creates an id used for transport from the specified parameters.
 TransportNodeId CreateNodeId(TransportConnectionId connection_id,
                              TransportConnectionSpecificNodeId node_id) {
   return (connection_id << 16) | node_id;
 }
 
@@ skipping 28 matching lines @@
   bool result = false;
   view_manager->DeleteView(view_id, base::Bind(&BooleanCallback, &result));
   DoRunLoop();
   return result;
 }
 
 bool SetNodeBounds(IViewManager* view_manager,
                    TransportNodeId node_id,
                    const gfx::Rect& bounds) {
   bool result = false;
-  view_manager->SetNodeBounds(node_id, bounds,
+  view_manager->SetNodeBounds(node_id, Rect::From(bounds),
                               base::Bind(&BooleanCallback, &result));
   DoRunLoop();
   return result;
 }
 
 // Adds a node, blocking until done.
 bool AddNode(IViewManager* view_manager,
              TransportNodeId parent,
              TransportNodeId child,
              TransportChangeId server_change_id) {
@@ skipping 43 matching lines @@
   view_manager->SetView(node_id, view_id,
                         base::Bind(&BooleanCallback, &result));
   DoRunLoop();
   return result;
 }
 
 bool Connect(IViewManager* view_manager,
              const std::string& url,
              TransportNodeId id,
              TransportNodeId id2) {
-  AllocationScope scope;
   bool result = false;
   std::vector<TransportNodeId> node_ids;
   node_ids.push_back(id);
   if (id2 != 0)
     node_ids.push_back(id2);
   view_manager->Connect(url, Array<uint32_t>::From(node_ids),
                         base::Bind(&BooleanCallback, &result));
   DoRunLoop();
   return result;
 }
@@ skipping 66 matching lines @@
 
   void DoRunLoopUntilChangesCount(size_t count) {
     main_loop_tracker_delegate_.DoRunLoopUntilChangesCount(count);
   }
 
  private:
   // IViewManagerClient overrides:
   virtual void OnViewManagerConnectionEstablished(
       TransportConnectionId connection_id,
       TransportChangeId next_server_change_id,
-      const Array<INode>& nodes) OVERRIDE {
+      mojo::Array<INodePtr> nodes) OVERRIDE {
     id_ = connection_id;
     next_server_change_id_ = next_server_change_id;
     initial_nodes_.clear();
     INodesToTestNodes(nodes, &initial_nodes_);
     tracker_.OnViewManagerConnectionEstablished(
-        connection_id, next_server_change_id, nodes);
+        connection_id, next_server_change_id, nodes.Pass());
   }
   virtual void OnServerChangeIdAdvanced(
       TransportChangeId next_server_change_id) OVERRIDE {
     tracker_.OnServerChangeIdAdvanced(next_server_change_id);
   }
   virtual void OnNodeBoundsChanged(TransportNodeId node_id,
-                                   const Rect& old_bounds,
-                                   const Rect& new_bounds) OVERRIDE {
-    tracker_.OnNodeBoundsChanged(node_id, old_bounds, new_bounds);
+                                   RectPtr old_bounds,
+                                   RectPtr new_bounds) OVERRIDE {
+    tracker_.OnNodeBoundsChanged(node_id, old_bounds.Pass(), new_bounds.Pass());
   }
   virtual void OnNodeHierarchyChanged(
       TransportNodeId node,
       TransportNodeId new_parent,
       TransportNodeId old_parent,
       TransportChangeId server_change_id,
-      const Array<INode>& nodes) OVERRIDE {
-    tracker_.OnNodeHierarchyChanged(node, new_parent, old_parent,
-                                    server_change_id, nodes);
+      mojo::Array<INodePtr> nodes) OVERRIDE {
     hierarchy_changed_nodes_.clear();
     INodesToTestNodes(nodes, &hierarchy_changed_nodes_);
+    tracker_.OnNodeHierarchyChanged(node, new_parent, old_parent,
+                                    server_change_id, nodes.Pass());
   }
   virtual void OnNodeDeleted(TransportNodeId node,
                              TransportChangeId server_change_id) OVERRIDE {
     tracker_.OnNodeDeleted(node, server_change_id);
   }
   virtual void OnViewDeleted(TransportViewId view) OVERRIDE {
     tracker_.OnViewDeleted(view);
   }
   virtual void OnNodeViewReplaced(TransportNodeId node,
                                   TransportViewId new_view_id,
@@ skipping 123 matching lines @@
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 1));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 2));
 
   EXPECT_TRUE(client_.GetAndClearChanges().empty());
 
   EstablishSecondConnection();
   EXPECT_TRUE(client2_.GetAndClearChanges().empty());
 
   // Make 2 a child of 1.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 1),
                         CreateNodeId(client_.id(), 2),
                         1));
     Changes changes(client_.GetAndClearChanges());
     ASSERT_TRUE(changes.empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     changes = client2_.GetAndClearChanges();
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ("ServerChangeIdAdvanced 2", changes[0]);
   }
 
   // Remove 2 from its parent.
   {
-    AllocationScope scope;
     ASSERT_TRUE(RemoveNodeFromParent(view_manager_.get(),
                                      CreateNodeId(client_.id(), 2),
                                      2));
     Changes changes(client_.GetAndClearChanges());
     ASSERT_TRUE(changes.empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     changes = client2_.GetAndClearChanges();
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ("ServerChangeIdAdvanced 3", changes[0]);
   }
 }
 
 // Verifies AddNode fails when node is already in position.
 TEST_F(ViewManagerConnectionTest, AddNodeWithNoChange) {
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 1));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 2));
 
   EXPECT_TRUE(client_.GetAndClearChanges().empty());
 
   EstablishSecondConnection();
   EXPECT_TRUE(client2_.GetAndClearChanges().empty());
 
   // Make 2 a child of 1.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 1),
                         CreateNodeId(client_.id(), 2),
                         1));
     Changes changes(client_.GetAndClearChanges());
     ASSERT_TRUE(changes.empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     changes = client2_.GetAndClearChanges();
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ("ServerChangeIdAdvanced 2", changes[0]);
   }
 
   // Try again, this should fail.
   {
-    AllocationScope scope;
     EXPECT_FALSE(AddNode(view_manager_.get(),
                          CreateNodeId(client_.id(), 1),
                          CreateNodeId(client_.id(), 2),
                          2));
     Changes changes(client_.GetAndClearChanges());
     EXPECT_TRUE(changes.empty());
   }
 }
 
 // Verifies AddNode fails when node is already in position.
 TEST_F(ViewManagerConnectionTest, AddAncestorFails) {
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 1));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 2));
 
   EXPECT_TRUE(client_.GetAndClearChanges().empty());
 
   EstablishSecondConnection();
   EXPECT_TRUE(client2_.GetAndClearChanges().empty());
 
   // Make 2 a child of 1.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 1),
                         CreateNodeId(client_.id(), 2),
                         1));
     Changes changes(client_.GetAndClearChanges());
     ASSERT_TRUE(changes.empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     changes = client2_.GetAndClearChanges();
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ("ServerChangeIdAdvanced 2", changes[0]);
   }
 
   // Try to make 1 a child of 2, this should fail since 1 is an ancestor of 2.
   {
-    AllocationScope scope;
     EXPECT_FALSE(AddNode(view_manager_.get(),
                          CreateNodeId(client_.id(), 2),
                          CreateNodeId(client_.id(), 1),
                          2));
     Changes changes(client_.GetAndClearChanges());
     EXPECT_TRUE(changes.empty());
   }
 }
 
 // Verifies adding with an invalid id fails.
 TEST_F(ViewManagerConnectionTest, AddWithInvalidServerId) {
   // Create two nodes.
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 1));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 2));
 
   // Make 2 a child of 1. Supply an invalid change id, which should fail.
   {
-    AllocationScope scope;
     ASSERT_FALSE(AddNode(view_manager_.get(),
                          CreateNodeId(client_.id(), 1),
                          CreateNodeId(client_.id(), 2),
                          0));
     Changes changes(client_.GetAndClearChanges());
     EXPECT_TRUE(changes.empty());
   }
 }
 
 // Verifies adding to root sends right notifications.
 TEST_F(ViewManagerConnectionTest, AddToRoot) {
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 21));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 3));
   EXPECT_TRUE(client_.GetAndClearChanges().empty());
 
   EstablishSecondConnection();
   EXPECT_TRUE(client2_.GetAndClearChanges().empty());
 
   // Make 3 a child of 21.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 21),
                         CreateNodeId(client_.id(), 3),
                         1));
     Changes changes(client_.GetAndClearChanges());
     ASSERT_TRUE(changes.empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     changes = client2_.GetAndClearChanges();
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ("ServerChangeIdAdvanced 2", changes[0]);
   }
 
   // Make 21 a child of the root.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(0, 1),
                         CreateNodeId(client_.id(), 21),
                         2));
     Changes changes(client_.GetAndClearChanges());
     ASSERT_TRUE(changes.empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     changes = client2_.GetAndClearChanges();
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ(
         "HierarchyChanged change_id=2 node=1,21 new_parent=0,1 old_parent=null",
         changes[0]);
   }
 }
 
 // Verifies adding to root sends right notifications.
 TEST_F(ViewManagerConnectionTest, NodeHierarchyChangedNodes) {
   // Create nodes 1 and 11 with 1 parented to the root and 11 a child of 1.
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 1));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 11));
 
   // Make 11 a child of 1.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 1),
                         CreateNodeId(client_.id(), 11),
                         1));
     ASSERT_TRUE(client_.GetAndClearChanges().empty());
   }
 
   EstablishSecondConnection();
 
   // Make 1 a child of the root.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(0, 1),
                         CreateNodeId(client_.id(), 1),
                         2));
     ASSERT_TRUE(client_.GetAndClearChanges().empty());
 
     // Client 2 should get a hierarchy change that includes the new nodes as it
     // has not yet seen them.
     client2_.DoRunLoopUntilChangesCount(1);
     Changes changes(client2_.GetAndClearChanges());
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ(
         "HierarchyChanged change_id=2 node=1,1 new_parent=0,1 old_parent=null",
         changes[0]);
     const std::vector<TestNode>& nodes(client2_.hierarchy_changed_nodes());
     ASSERT_EQ(2u, nodes.size());
     EXPECT_EQ("node=1,1 parent=0,1 view=null", nodes[0].ToString());
     EXPECT_EQ("node=1,11 parent=1,1 view=null", nodes[1].ToString());
   }
 
   // Remove 1 from the root.
   {
-    AllocationScope scope;
     ASSERT_TRUE(RemoveNodeFromParent(view_manager_.get(),
                                      CreateNodeId(client_.id(), 1),
                                      3));
     ASSERT_TRUE(client_.GetAndClearChanges().empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     Changes changes(client2_.GetAndClearChanges());
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ(
         "HierarchyChanged change_id=3 node=1,1 new_parent=null old_parent=0,1",
         changes[0]);
   }
 
   // Create another node, 111, parent it to 11.
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 111));
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 11),
                         CreateNodeId(client_.id(), 111),
                         4));
     ASSERT_TRUE(client_.GetAndClearChanges().empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     Changes changes(client2_.GetAndClearChanges());
     ASSERT_EQ(1u, changes.size());
     // Even though 11 isn't attached to the root client 2 is still notified of
@@ skipping 27 matching lines @@
 TEST_F(ViewManagerConnectionTest, NodeHierarchyChangedAddingKnownToUnknown) {
   // Create the following structure: root -> 1 -> 11 and 2->21 (2 has no
   // parent).
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 1));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 11));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 2));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 21));
 
   // Set up the hierarchy.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(0, 1),
                         CreateNodeId(client_.id(), 1),
                         1));
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 1),
                         CreateNodeId(client_.id(), 11),
                         2));
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 2),
                         CreateNodeId(client_.id(), 21),
                         3));
   }
 
   EstablishSecondConnection();
 
   // Remove 11.
   {
-    AllocationScope scope;
     ASSERT_TRUE(RemoveNodeFromParent(view_manager_.get(),
                                      CreateNodeId(client_.id(), 11),
                                      4));
     ASSERT_TRUE(client_.GetAndClearChanges().empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     Changes changes(client2_.GetAndClearChanges());
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ(
         "HierarchyChanged change_id=4 node=1,11 new_parent=null old_parent=1,1",
         changes[0]);
     EXPECT_TRUE(client2_.hierarchy_changed_nodes().empty());
   }
 
   // Add 11 to 21. As client2 knows about 11 it should receive the new
   // hierarchy.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 21),
                         CreateNodeId(client_.id(), 11),
                         5));
     ASSERT_TRUE(client_.GetAndClearChanges().empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     Changes changes(client2_.GetAndClearChanges());
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ(
         "HierarchyChanged change_id=5 node=1,11 new_parent=1,21 "
         "old_parent=null", changes[0]);
     const std::vector<TestNode>& nodes(client2_.hierarchy_changed_nodes());
     ASSERT_EQ(2u, nodes.size());
     EXPECT_EQ("node=1,2 parent=null view=null", nodes[0].ToString());
     EXPECT_EQ("node=1,21 parent=1,2 view=null", nodes[1].ToString());
   }
 }
 
 // Verifies connection on told descendants of the root when connecting.
 TEST_F(ViewManagerConnectionTest, GetInitialNodesOnInit) {
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 21));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 3));
   EXPECT_TRUE(client_.GetAndClearChanges().empty());
 
   // Make 3 a child of 21.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 21),
                         CreateNodeId(client_.id(), 3),
                         1));
     ASSERT_TRUE(client_.GetAndClearChanges().empty());
   }
 
   // Make 21 a child of the root.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(0, 1),
                         CreateNodeId(client_.id(), 21),
                         2));
     ASSERT_TRUE(client_.GetAndClearChanges().empty());
   }
 
   EstablishSecondConnection();
   // Should get notification of children of the root.
   const std::vector<TestNode>& nodes(client2_.initial_nodes());
   ASSERT_EQ(3u, nodes.size());
   EXPECT_EQ("node=0,1 parent=null view=null", nodes[0].ToString());
   EXPECT_EQ("node=1,21 parent=0,1 view=null", nodes[1].ToString());
   EXPECT_EQ("node=1,3 parent=1,21 view=null", nodes[2].ToString());
 }
 
 // Verifies DeleteNode works.
 TEST_F(ViewManagerConnectionTest, DeleteNode) {
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 1));
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 2));
   EXPECT_TRUE(client_.GetAndClearChanges().empty());
 
   EstablishSecondConnection();
   EXPECT_TRUE(client2_.GetAndClearChanges().empty());
 
   // Make 2 a child of 1.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(client_.id(), 1),
                         CreateNodeId(client_.id(), 2),
                         1));
     Changes changes(client_.GetAndClearChanges());
     ASSERT_TRUE(changes.empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     changes = client2_.GetAndClearChanges();
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ("ServerChangeIdAdvanced 2", changes[0]);
   }
 
   // Add 1 to the root
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(0, 1),
                         CreateNodeId(client_.id(), 1),
                         2));
     Changes changes(client_.GetAndClearChanges());
     ASSERT_TRUE(changes.empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     changes = client2_.GetAndClearChanges();
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ(
         "HierarchyChanged change_id=2 node=1,1 new_parent=0,1 old_parent=null",
         changes[0]);
   }
 
   // Delete 1.
   {
-    AllocationScope scope;
     ASSERT_TRUE(DeleteNode(view_manager_.get(), CreateNodeId(client_.id(), 1)));
     Changes changes(client_.GetAndClearChanges());
     ASSERT_TRUE(changes.empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     changes = client2_.GetAndClearChanges();
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ("NodeDeleted change_id=3 node=1,1", changes[0]);
   }
 }
@@ skipping 15 matching lines @@
 // Verifies if a node was deleted and then reused that other clients are
 // properly notified.
 TEST_F(ViewManagerConnectionTest, ReusedDeletedId) {
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 1));
   EXPECT_TRUE(client_.GetAndClearChanges().empty());
 
   EstablishSecondConnection();
 
   // Make 1 a child of the root.
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(0, 1),
                         CreateNodeId(client_.id(), 1),
                         1));
     EXPECT_TRUE(client_.GetAndClearChanges().empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     Changes changes = client2_.GetAndClearChanges();
     EXPECT_EQ(
         "HierarchyChanged change_id=1 node=1,1 new_parent=0,1 old_parent=null",
         changes[0]);
     const std::vector<TestNode>& nodes(client2_.hierarchy_changed_nodes());
     ASSERT_EQ(1u, nodes.size());
     EXPECT_EQ("node=1,1 parent=0,1 view=null", nodes[0].ToString());
   }
 
   // Delete 1.
   {
-    AllocationScope scope;
     ASSERT_TRUE(DeleteNode(view_manager_.get(), CreateNodeId(client_.id(), 1)));
     EXPECT_TRUE(client_.GetAndClearChanges().empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     Changes changes = client2_.GetAndClearChanges();
     ASSERT_EQ(1u, changes.size());
     EXPECT_EQ("NodeDeleted change_id=2 node=1,1", changes[0]);
   }
 
   // Create 1 again, and add it back to the root. Should get the same
   // notification.
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 1));
   {
-    AllocationScope scope;
     ASSERT_TRUE(AddNode(view_manager_.get(),
                         CreateNodeId(0, 1),
                         CreateNodeId(client_.id(), 1),
                         3));
     EXPECT_TRUE(client_.GetAndClearChanges().empty());
 
     client2_.DoRunLoopUntilChangesCount(1);
     Changes changes = client2_.GetAndClearChanges();
     EXPECT_EQ(
         "HierarchyChanged change_id=3 node=1,1 new_parent=0,1 old_parent=null",
@@ skipping 181 matching lines @@
                       4));
 
   // Attach view to node 11 in the first connection.
   ASSERT_TRUE(CreateView(view_manager_.get(), 1, 51));
   ASSERT_TRUE(SetView(view_manager_.get(),
                       CreateNodeId(client_.id(), 11),
                       CreateViewId(client_.id(), 51)));
 
   // Verifies GetNodeTree() on the root.
   {
-    AllocationScope scope;
     std::vector<TestNode> nodes;
     GetNodeTree(view_manager2_.get(), CreateNodeId(0, 1), &nodes);
     ASSERT_EQ(5u, nodes.size());
     EXPECT_EQ("node=0,1 parent=null view=null", nodes[0].ToString());
     EXPECT_EQ("node=1,1 parent=0,1 view=null", nodes[1].ToString());
     EXPECT_EQ("node=1,11 parent=1,1 view=1,51", nodes[2].ToString());
     EXPECT_EQ("node=2,2 parent=0,1 view=null", nodes[3].ToString());
     EXPECT_EQ("node=2,3 parent=0,1 view=null", nodes[4].ToString());
   }
 
   // Verifies GetNodeTree() on the node 1,1.
   {
-    AllocationScope scope;
     std::vector<TestNode> nodes;
     GetNodeTree(view_manager2_.get(), CreateNodeId(1, 1), &nodes);
     ASSERT_EQ(2u, nodes.size());
     EXPECT_EQ("node=1,1 parent=0,1 view=null", nodes[0].ToString());
     EXPECT_EQ("node=1,11 parent=1,1 view=1,51", nodes[1].ToString());
   }
 }
 
 TEST_F(ViewManagerConnectionTest, SetNodeBounds) {
   ASSERT_TRUE(CreateNode(view_manager_.get(), 1, 1));
   ASSERT_TRUE(AddNode(view_manager_.get(),
                       CreateNodeId(0, 1),
                       CreateNodeId(1, 1),
                       1));
   EstablishSecondConnection();
 
-  AllocationScope scope;
   ASSERT_TRUE(SetNodeBounds(view_manager_.get(),
                             CreateNodeId(1, 1),
                             gfx::Rect(0, 0, 100, 100)));
 
   client2_.DoRunLoopUntilChangesCount(1);
   Changes changes(client2_.GetAndClearChanges());
   ASSERT_EQ(1u, changes.size());
   EXPECT_EQ("BoundsChanged node=1,1 old_bounds=0,0 0x0 new_bounds=0,0 100x100",
             changes[0]);
 
@@ skipping 135 matching lines @@
 
   // Parent 11 to 10.
   ASSERT_TRUE(background_connection_->AddNode(CreateNodeId(2, 10),
                                               CreateNodeId(2, 11), 3));
   // Remove 11 from 10.
   ASSERT_TRUE(background_connection_->RemoveNodeFromParent(
                   CreateNodeId(2, 11), 4));
 
   // Verify nothing was actually removed.
   {
-    AllocationScope scope;
     std::vector<TestNode> nodes;
     GetNodeTree(view_manager_.get(), CreateNodeId(0, 1), &nodes);
     ASSERT_EQ(3u, nodes.size());
     EXPECT_EQ("node=0,1 parent=null view=null", nodes[0].ToString());
     EXPECT_EQ("node=1,1 parent=0,1 view=null", nodes[1].ToString());
     EXPECT_EQ("node=1,2 parent=0,1 view=null", nodes[2].ToString());
   }
 }
 
 // Verify SetView fails for nodes that are not descendants of the roots.
@@ skipping 34 matching lines @@
                       CreateNodeId(0, 1),
                       CreateNodeId(client_.id(), 1),
                       1));
   ASSERT_TRUE(AddNode(view_manager_.get(),
                       CreateNodeId(0, 1),
                       CreateNodeId(client_.id(), 2),
                       2));
 
   ASSERT_NO_FATAL_FAILURE(EstablishBackgroundConnectionWithRoot1());
 
-  AllocationScope scope;
   std::vector<TestNode> nodes;
 
   // Should get nothing for the root.
   background_connection_->GetNodeTree(CreateNodeId(0, 1), &nodes);
   ASSERT_TRUE(nodes.empty());
 
   // Should get nothing for node 2.
   background_connection_->GetNodeTree(CreateNodeId(1, 2), &nodes);
   ASSERT_TRUE(nodes.empty());
 
@@ skipping 15 matching lines @@
   ASSERT_EQ(1u, changes.size());
   EXPECT_EQ("OnConnectionEstablished", changes[0]);
 }
 
 // TODO(sky): add coverage of test that destroys connections and ensures other
 // connections get deletion notification (or advanced server id).
 
 }  // namespace service
 }  // namespace view_manager
 }  // namespace mojo