Convert ConnectToApplication to take a params class

mandoline/ui/desktop_ui/browser_window.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "mandoline/ui/desktop_ui/browser_window.h"
 
 #include "base/command_line.h"
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/time/time.h"
@@ skipping 123 matching lines @@
 void BrowserWindow::Close() {
   if (root_)
     mus::ScopedWindowPtr::DeleteWindowOrWindowManager(root_);
   else
     delete this;
 }
 
 void BrowserWindow::ShowOmnibox() {
   TRACE_EVENT0("desktop_ui", "BrowserWindow::ShowOmnibox");
   if (!omnibox_.get()) {
-    mojo::URLRequestPtr request(mojo::URLRequest::New());
-    request->url = mojo::String::From("mojo:omnibox");
-    omnibox_connection_ = app_->ConnectToApplication(request.Pass());
+    omnibox_connection_ = app_->ConnectToApplication("mojo:omnibox");
     omnibox_connection_->AddService<ViewEmbedder>(this);
     omnibox_connection_->ConnectToService(&omnibox_);
     omnibox_connection_->SetRemoteServiceProviderConnectionErrorHandler(
       [this]() {
         // This will cause the connection to be re-established the next time
         // we come through this codepath.
         omnibox_.reset();
     });
   }
   omnibox_->ShowForURL(mojo::String::From(current_url_.spec()));
@@ skipping 78 matching lines @@
 
   // Record the time spent opening initial tabs, used for performance testing.
   const base::TimeDelta open_tabs_delta =
       base::TimeTicks::Now() - display_ticks;
 
   // Record the browser startup time metrics, used for performance testing.
   static bool recorded_browser_startup_metrics = false;
   if (!recorded_browser_startup_metrics &&
       base::CommandLine::ForCurrentProcess()->HasSwitch(
           tracing::kEnableStatsCollectionBindings)) {
-    mojo::URLRequestPtr request(mojo::URLRequest::New());
-    request->url = mojo::String::From("mojo:tracing");
     tracing::StartupPerformanceDataCollectorPtr collector;
-    app_->ConnectToService(request.Pass(), &collector);
+    app_->ConnectToService("mojo:tracing", &collector);
     collector->SetBrowserWindowDisplayTicks(display_ticks.ToInternalValue());
     collector->SetBrowserOpenTabsTimeDelta(open_tabs_delta.ToInternalValue());
     collector->SetBrowserMessageLoopStartTicks(
         manager_->startup_ticks().ToInternalValue());
     recorded_browser_startup_metrics = true;
   }
 }
 
 void BrowserWindow::OnConnectionLost(mus::WindowTreeConnection* connection) {
   root_ = nullptr;
@@ skipping 186 matching lines @@
   content_bounds.set_width(DIPSToPixels(progress_bar_bounds.width()));
   content_bounds.set_height(host->bounds().height() - content_bounds.y() -
                             DIPSToPixels(10));
   content_->SetBounds(content_bounds);
 
   // The omnibox view bounds are in physical pixels.
   omnibox_view_->SetBounds(bounds_in_physical_pixels);
 }
 
 }  // namespace mandoline