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

Unified Diff: mojo/dbus/dbus_external_service.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/dbus/dbus_external_service.cc
diff --git a/mojo/dbus/dbus_external_service.cc b/mojo/dbus/dbus_external_service.cc
index 3a13cf7b521c336187a151ae4e1d1f6206158967..92cd6f3e1d6498474440bdd00e8d7b6d9c445e61 100644
--- a/mojo/dbus/dbus_external_service.cc
+++ b/mojo/dbus/dbus_external_service.cc
@@ -16,7 +16,6 @@
#include "dbus/object_path.h"
#include "mojo/common/channel_init.h"
#include "mojo/public/cpp/bindings/error_handler.h"
-#include "mojo/public/cpp/bindings/remote_ptr.h"
#include "mojo/public/cpp/shell/application.h"
#include "mojo/public/interfaces/shell/shell.mojom.h"
#include "mojo/shell/external_service.mojom.h"
@@ -37,10 +36,6 @@ void DBusExternalServiceBase::Start() {
DVLOG(1) << "External service started";
}
-void DBusExternalServiceBase::OnError() {
- Disconnect();
-}
-
void DBusExternalServiceBase::ConnectChannel(
dbus::MethodCall* method_call,
dbus::ExportedObject::ResponseSender sender) {
@@ -61,7 +56,7 @@ void DBusExternalServiceBase::ConnectChannel(
base::MessageLoopProxy::current());
CHECK(message_pipe.is_valid());
- Connect(mojo::ScopedExternalServiceHostHandle::From(message_pipe.Pass()));
+ Connect(message_pipe.Pass());
sender.Run(dbus::Response::FromMethodCall(method_call));
}

Powered by Google App Engine
This is Rietveld 408576698