Mojo: more idiomatic C++ bindings

mojo/shell/app_child_process.cc

Index: mojo/shell/app_child_process.cc
diff --git a/mojo/shell/app_child_process.cc b/mojo/shell/app_child_process.cc
index 099cfb2ee07d289a69d7859fcc6f1d3b6724aecf..5ef7eb1235ee30a22fd4f58bb44e5e51c32dc2ae 100644
--- a/mojo/shell/app_child_process.cc
+++ b/mojo/shell/app_child_process.cc
@@ -193,16 +193,13 @@ class AppChildControllerImpl : public InterfaceImpl<AppChildController> {
   }
 
   // |AppChildController| methods:
-  virtual void StartApp(const String& app_path,
+  virtual void StartApp(String app_path,
                         ScopedMessagePipeHandle service) OVERRIDE {
-    DVLOG(2) << "AppChildControllerImpl::StartApp("
-             << app_path.To<std::string>() << ", ...)";
+    DVLOG(2) << "AppChildControllerImpl::StartApp(" << app_path << ", ...)";
     DCHECK(thread_checker_.CalledOnValidThread());
 
-    // TODO(darin): Add TypeConverter for FilePath <-> mojo::String.
     unblocker_.Unblock(base::Bind(&AppChildControllerImpl::StartAppOnMainThread,
-                                  base::FilePath::FromUTF8Unsafe(
-                                      app_path.To<std::string>()),
+                                  base::FilePath::FromUTF8Unsafe(app_path),
                                   base::Passed(&service)));
   }