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

Unified Diff: mojo/services/dbus_echo/dbus_echo_service.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/services/dbus_echo/dbus_echo_service.cc
diff --git a/mojo/services/dbus_echo/dbus_echo_service.cc b/mojo/services/dbus_echo/dbus_echo_service.cc
index 0678339166613774040eff9a06ee747399f4c243..515bb056b8847264163e43835fce4116f3ffae49 100644
--- a/mojo/services/dbus_echo/dbus_echo_service.cc
+++ b/mojo/services/dbus_echo/dbus_echo_service.cc
@@ -25,10 +25,10 @@ class EchoServiceImpl : public mojo::InterfaceImpl<mojo::EchoService> {
protected:
virtual void Echo(
- const mojo::String& in_to_echo,
+ mojo::String in_to_echo,
const mojo::Callback<void(mojo::String)>& callback) OVERRIDE {
- DVLOG(1) << "Asked to echo " << in_to_echo.To<std::string>();
- callback.Run(in_to_echo);
+ DVLOG(1) << "Asked to echo " << in_to_echo;
+ callback.Run(in_to_echo.Pass());
}
};

Powered by Google App Engine
This is Rietveld 408576698