Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1079)

Unified Diff: mojo/services/view_manager/view_manager_connection.cc

Issue 265793015: Mojo: Replace RemotePtr with InterfacePtr and InterfaceImpl (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: mojo/services/view_manager/view_manager_connection.cc
diff --git a/mojo/services/view_manager/view_manager_connection.cc b/mojo/services/view_manager/view_manager_connection.cc
index 37e5f059fa375bb531e60851cc9f593651b047a1..a4decb6035facda224c788179344500d7eb6ddd7 100644
--- a/mojo/services/view_manager/view_manager_connection.cc
+++ b/mojo/services/view_manager/view_manager_connection.cc
@@ -56,7 +56,7 @@ void NodeToINode(Node* node,
} // namespace
-ViewManagerConnection::ViewManagerConnection() : id_(0) {
+ViewManagerConnection::ViewManagerConnection() : client_(NULL), id_(0) {
}
ViewManagerConnection::~ViewManagerConnection() {
@@ -78,15 +78,11 @@ ViewManagerConnection::~ViewManagerConnection() {
context()->RemoveConnection(this);
}
-void ViewManagerConnection::Initialize(
- ServiceConnector<ViewManagerConnection, RootNodeManager>* service_factory,
- ScopedMessagePipeHandle client_handle) {
+void ViewManagerConnection::Initialize() {
DCHECK_EQ(0, id_); // Should only get Initialize() once.
- ServiceConnection<IViewManager, ViewManagerConnection, RootNodeManager>::
- Initialize(service_factory, client_handle.Pass());
id_ = context()->GetAndAdvanceNextConnectionId();
context()->AddConnection(this);
- client()->OnConnectionEstablished(id_);
+ client_->OnConnectionEstablished(id_);
}
Node* ViewManagerConnection::GetNode(const NodeId& id) {
@@ -110,7 +106,7 @@ void ViewManagerConnection::NotifyNodeHierarchyChanged(
const NodeId& new_parent,
const NodeId& old_parent,
TransportChangeId change_id) {
- client()->OnNodeHierarchyChanged(NodeIdToTransportId(node),
+ client_->OnNodeHierarchyChanged(NodeIdToTransportId(node),
NodeIdToTransportId(new_parent),
NodeIdToTransportId(old_parent),
change_id);
@@ -121,7 +117,7 @@ void ViewManagerConnection::NotifyNodeViewReplaced(
const ViewId& new_view_id,
const ViewId& old_view_id,
TransportChangeId change_id) {
- client()->OnNodeViewReplaced(NodeIdToTransportId(node),
+ client_->OnNodeViewReplaced(NodeIdToTransportId(node),
ViewIdToTransportId(new_view_id),
ViewIdToTransportId(old_view_id),
change_id);
@@ -175,6 +171,10 @@ bool ViewManagerConnection::SetViewImpl(const NodeId& node_id,
return true;
}
+void ViewManagerConnection::SetClient(IViewManagerClient* client) {
+ client_ = client;
+}
+
void ViewManagerConnection::CreateNode(
TransportConnectionSpecificNodeId node_id,
const Callback<void(bool)>& callback) {

Powered by Google App Engine
This is Rietveld 408576698