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

Unified Diff: mojo/public/cpp/shell/lib/application.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/public/cpp/shell/lib/application.cc
diff --git a/mojo/public/cpp/shell/lib/application.cc b/mojo/public/cpp/shell/lib/application.cc
index 161e4e0e582bba6d710201677739ccb1f158edde..9e93a679e998f276195a8bcbeba5beab91e9d432 100644
--- a/mojo/public/cpp/shell/lib/application.cc
+++ b/mojo/public/cpp/shell/lib/application.cc
@@ -41,13 +41,13 @@ void Application::RemoveServiceConnector(
shell_.reset();
}
-void Application::AcceptConnection(const mojo::String& url,
+void Application::AcceptConnection(mojo::String url,
ScopedMessagePipeHandle client_handle) {
// TODO(davemoore): This method must be overridden by an Application subclass
// to dispatch to the right ServiceConnector. We need to figure out an
// approach to registration to make this better.
assert(1 == service_connectors_.size());
- return service_connectors_.front()->AcceptConnection(url.To<std::string>(),
+ return service_connectors_.front()->AcceptConnection(url,
client_handle.Pass());
}

Powered by Google App Engine
This is Rietveld 408576698