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

Unified Diff: mojo/shell/dbus_service_loader_linux.cc

Issue 294833002: Mojo: more idiomatic C++ bindings (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more 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/shell/dbus_service_loader_linux.cc
diff --git a/mojo/shell/dbus_service_loader_linux.cc b/mojo/shell/dbus_service_loader_linux.cc
index f2ecbcf26a6c6ce0e0bb69bbced0bb8d4f32fb3d..4bbf96dfc1c52ef2dae0f40a785afedc4633e9dd 100644
--- a/mojo/shell/dbus_service_loader_linux.cc
+++ b/mojo/shell/dbus_service_loader_linux.cc
@@ -19,7 +19,6 @@
#include "mojo/common/channel_init.h"
#include "mojo/dbus/dbus_external_service.h"
#include "mojo/embedder/platform_channel_pair.h"
-#include "mojo/public/cpp/bindings/allocation_scope.h"
#include "mojo/shell/context.h"
#include "mojo/shell/external_service.mojom.h"
#include "mojo/shell/keep_alive.h"
@@ -105,7 +104,6 @@ class DBusServiceLoader::LoadContext {
// Sends a ShellHandle over to the now-connected externally-running service,
// using the Mojo ExternalService API.
void ActivateService(dbus::Response* response) {
- mojo::AllocationScope scope;
external_service_->Activate(
mojo::ScopedMessagePipeHandle(
mojo::MessagePipeHandle(shell_handle_.release().value())));

Powered by Google App Engine
This is Rietveld 408576698