Cleanup of ViewManager

mojo/services/view_manager/view_manager_connection.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 "base/stl_util.h"
 #include "mojo/public/cpp/bindings/allocation_scope.h"
 #include "mojo/services/view_manager/node.h"
 #include "mojo/services/view_manager/root_node_manager.h"
 #include "mojo/services/view_manager/type_converters.h"
 #include "mojo/services/view_manager/view.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/aura/window.h"
 #include "ui/gfx/codec/png_codec.h"
 
 namespace mojo {
 namespace view_manager {
 namespace service {
 namespace {
 
 // Places |node| in |nodes| and recurses through the children.
-void GetDescendants(Node* node, std::vector<Node*>* nodes) {
+void GetDescendants(const Node* node, std::vector<const Node*>* nodes) {
   if (!node)
     return;
 
   nodes->push_back(node);
 
-  std::vector<Node*> children(node->GetChildren());
+  std::vector<const Node*> children(node->GetChildren());
   for (size_t i = 0 ; i < children.size(); ++i)
     GetDescendants(children[i], nodes);
 }
 
 }  // namespace
 
 ViewManagerConnection::ViewManagerConnection(RootNodeManager* root_node_manager)
     : root_node_manager_(root_node_manager),
       id_(0) {
 }
@@ skipping 14 matching lines @@
   }
 
   STLDeleteContainerPairSecondPointers(node_map_.begin(), node_map_.end());
   root_node_manager_->RemoveConnection(this);
 }
 
 void ViewManagerConnection::OnConnectionEstablished() {
   DCHECK_EQ(0, id_);  // Should only get OnConnectionEstablished() once.
   id_ = root_node_manager_->GetAndAdvanceNextConnectionId();
   root_node_manager_->AddConnection(this);
-  std::vector<Node*> to_send;
+  std::vector<const Node*> to_send;
   GetUnknownNodesFrom(root_node_manager_->root(), &to_send);
   AllocationScope allocation_scope;
   client()->OnConnectionEstablished(
       id_,
       root_node_manager_->next_server_change_id(),
       Array<INode>::From(to_send));
 }
 
 void ViewManagerConnection::OnConnectionError() {}
 
 Node* ViewManagerConnection::GetNode(const NodeId& id) {
   if (id_ == id.connection_id) {
     NodeMap::iterator i = node_map_.find(id.node_id);
     return i == node_map_.end() ? NULL : i->second;
   }
   return root_node_manager_->GetNode(id);
 }
 
 View* ViewManagerConnection::GetView(const ViewId& id) {
   if (id_ == id.connection_id) {
     ViewMap::const_iterator i = view_map_.find(id.view_id);
     return i == view_map_.end() ? NULL : i->second;
   }
   return root_node_manager_->GetView(id);
 }
 
 void ViewManagerConnection::ProcessNodeHierarchyChanged(
-    const NodeId& node_id,
-    const NodeId& new_parent_id,
-    const NodeId& old_parent_id,
+    const Node* node,
+    const Node* new_parent,
+    const Node* old_parent,
     TransportChangeId server_change_id,
     bool originated_change) {
   if (originated_change || root_node_manager_->is_processing_delete_node())
     return;
-  std::vector<Node*> to_send;
-  if (!ShouldNotifyOnHierarchyChange(node_id, new_parent_id, old_parent_id,
-                                     &to_send)) {
+  std::vector<const Node*> to_send;
+  if (!ShouldNotifyOnHierarchyChange(node, new_parent, old_parent, &to_send)) {
     if (root_node_manager_->IsProcessingChange()) {
       client()->OnServerChangeIdAdvanced(
           root_node_manager_->next_server_change_id() + 1);
     }
     return;
   }
   AllocationScope allocation_scope;
-  client()->OnNodeHierarchyChanged(NodeIdToTransportId(node_id),
+  const NodeId new_parent_id(new_parent ? new_parent->id() : NodeId());
+  const NodeId old_parent_id(old_parent ? old_parent->id() : NodeId());
+  client()->OnNodeHierarchyChanged(NodeIdToTransportId(node->id()),
                                    NodeIdToTransportId(new_parent_id),
                                    NodeIdToTransportId(old_parent_id),
                                    server_change_id,
                                    Array<INode>::From(to_send));
 }
 
 void ViewManagerConnection::ProcessNodeViewReplaced(
-    const NodeId& node,
-    const ViewId& new_view_id,
-    const ViewId& old_view_id,
+    const Node* node,
+    const View* new_view,
+    const View* old_view,
     bool originated_change) {
   if (originated_change)
     return;
-  client()->OnNodeViewReplaced(NodeIdToTransportId(node),
-                               ViewIdToTransportId(new_view_id),
-                               ViewIdToTransportId(old_view_id));
+  const TransportViewId new_view_id = new_view ?
+      ViewIdToTransportId(new_view->id()) : 0;
+  const TransportViewId old_view_id = old_view ?
+      ViewIdToTransportId(old_view->id()) : 0;
+  client()->OnNodeViewReplaced(NodeIdToTransportId(node->id()),
+                               new_view_id, old_view_id);
 }
 
 void ViewManagerConnection::ProcessNodeDeleted(
     const NodeId& node,
     TransportChangeId server_change_id,
     bool originated_change) {
   const bool in_known = known_nodes_.erase(NodeIdToTransportId(node)) > 0;
 
   if (originated_change)
     return;
@@ skipping 64 matching lines @@
   if (!view && view_id != ViewId())
     return false;
   RootNodeManager::ScopedChange change(
       this, root_node_manager_,
       RootNodeManager::CHANGE_TYPE_DONT_ADVANCE_SERVER_CHANGE_ID, false);
   node->SetView(view);
   return true;
 }
 
 void ViewManagerConnection::GetUnknownNodesFrom(
-    Node* node,
-    std::vector<Node*>* nodes) {
+    const Node* node,
+    std::vector<const Node*>* nodes) {
   const TransportNodeId transport_id = NodeIdToTransportId(node->id());
   if (known_nodes_.count(transport_id) == 1)
     return;
   nodes->push_back(node);
   known_nodes_.insert(transport_id);
-  std::vector<Node*> children(node->GetChildren());
+  std::vector<const Node*> children(node->GetChildren());
   for (size_t i = 0 ; i < children.size(); ++i)
     GetUnknownNodesFrom(children[i], nodes);
 }
 
 bool ViewManagerConnection::ShouldNotifyOnHierarchyChange(
-    const NodeId& node_id,
-    const NodeId& new_parent_id,
-    const NodeId& old_parent_id,
-    std::vector<Node*>* to_send) {
-  Node* new_parent = GetNode(new_parent_id);
+    const Node* node,
+    const Node* new_parent,
+    const Node* old_parent,
+    std::vector<const Node*>* to_send) {
   if (new_parent) {
     // On getting a new parent we may need to communicate new nodes to the
     // client. We do that in the following cases:
     // . New parent is a descendant of the root. In this case the client already
     //   knows all ancestors, so we only have to communicate descendants of node
     //   the client doesn't know about.
     // . If the client knew about the parent, we have to do the same.
     // . If the client knows about the node and is added to a tree the client
     //   doesn't know about we have to communicate from the root down (the
     //   client is learning about a new root).
     if (root_node_manager_->root()->Contains(new_parent) ||
-        known_nodes_.count(NodeIdToTransportId(new_parent_id))) {
-      GetUnknownNodesFrom(GetNode(node_id), to_send);
+        known_nodes_.count(NodeIdToTransportId(new_parent->id()))) {
+      GetUnknownNodesFrom(node, to_send);
       return true;
     }
     // If parent wasn't known we have to communicate from the root down.
-    if (known_nodes_.count(NodeIdToTransportId(node_id))) {
+    if (known_nodes_.count(NodeIdToTransportId(node->id()))) {
       GetUnknownNodesFrom(new_parent->GetRoot(), to_send);
       return true;
     }
   }
   // Otherwise only communicate the change if the node was known. We shouldn't
   // need to communicate any nodes on a remove.
-  return known_nodes_.count(NodeIdToTransportId(node_id)) > 0;
+  return known_nodes_.count(NodeIdToTransportId(node->id())) > 0;
 }
 
 void ViewManagerConnection::CreateNode(
     TransportNodeId transport_node_id,
     const Callback<void(bool)>& callback) {
   const NodeId node_id(NodeIdFromTransportId(transport_node_id));
   if (node_id.connection_id != id_ ||
       node_map_.find(node_id.node_id) != node_map_.end()) {
     callback.Run(false);
     return;
@@ skipping 50 matching lines @@
     }
   }
   callback.Run(success);
 }
 
 void ViewManagerConnection::GetNodeTree(
     TransportNodeId node_id,
     const Callback<void(Array<INode>)>& callback) {
   AllocationScope allocation_scope;
   Node* node = GetNode(NodeIdFromTransportId(node_id));
-  std::vector<Node*> nodes;
+  std::vector<const Node*> nodes;
   GetDescendants(node, &nodes);
   for (size_t i = 0; i < nodes.size(); ++i)
     known_nodes_.insert(NodeIdToTransportId(nodes[i]->id()));
   callback.Run(Array<INode>::From(nodes));
 }
 
 void ViewManagerConnection::CreateView(
     TransportViewId transport_view_id,
     const Callback<void(bool)>& callback) {
   const ViewId view_id(ViewIdFromTransportId(transport_view_id));
@@ skipping 34 matching lines @@
                 MOJO_MAP_BUFFER_FLAG_NONE) != MOJO_RESULT_OK) {
     return;
   }
   SkBitmap bitmap;
   gfx::PNGCodec::Decode(static_cast<const unsigned char*>(handle_data),
                         buffer_size, &bitmap);
   view->SetBitmap(bitmap);
   UnmapBuffer(handle_data);
 }
 
-void ViewManagerConnection::OnNodeHierarchyChanged(const NodeId& node,
-                                                   const NodeId& new_parent,
-                                                   const NodeId& old_parent) {
+void ViewManagerConnection::OnNodeHierarchyChanged(const Node* node,
+                                                   const Node* new_parent,
+                                                   const Node* old_parent) {
   root_node_manager_->ProcessNodeHierarchyChanged(node, new_parent, old_parent);
 }
 
-void ViewManagerConnection::OnNodeViewReplaced(const NodeId& node,
-                                               const ViewId& new_view_id,
-                                               const ViewId& old_view_id) {
-  root_node_manager_->ProcessNodeViewReplaced(node, new_view_id, old_view_id);
+void ViewManagerConnection::OnNodeViewReplaced(const Node* node,
+                                               const View* new_view,
+                                               const View* old_view) {
+  root_node_manager_->ProcessNodeViewReplaced(node, new_view, old_view);
 }
 
 }  // namespace service
 }  // namespace view_manager
 }  // namespace mojo