Change mojo geometry structs from using type converters to StructTraits.

services/navigation/view_impl.cc

 // Copyright 2016 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 "services/navigation/view_impl.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "components/mus/public/cpp/window_tree_connection.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/web_contents.h"
-#include "ui/gfx/geometry/mojo/geometry_type_converters.h"
 #include "ui/views/controls/webview/webview.h"
 #include "ui/views/mus/native_widget_mus.h"
 #include "ui/views/widget/widget.h"
 #include "url/gurl.h"
 
 namespace navigation {
 
 ViewImpl::ViewImpl(shell::Connector* connector,
                    content::BrowserContext* browser_context,
                    mojom::ViewClientPtr client,
@@ skipping 42 matching lines @@
 void ViewImpl::AddNewContents(content::WebContents* source,
                               content::WebContents* new_contents,
                               WindowOpenDisposition disposition,
                               const gfx::Rect& initial_rect,
                               bool user_gesture,
                               bool* was_blocked) {
   mojom::ViewClientPtr client;
   mojom::ViewPtr view;
   mojom::ViewRequest view_request = GetProxy(&view);
   client_->ViewCreated(std::move(view), GetProxy(&client),
-                       disposition == NEW_POPUP, mojo::Rect::From(initial_rect),
-                       user_gesture);
+                       disposition == NEW_POPUP, initial_rect, user_gesture);
   ViewImpl* impl =
       new ViewImpl(connector_, new_contents->GetBrowserContext(),
                    std::move(client), std::move(view_request), ref_->Clone());
   // TODO(beng): This is a bit crappy. should be able to create the ViewImpl
   //             with |new_contents| instead.
   impl->web_view_->SetWebContents(new_contents);
   impl->web_view_->GetWebContents()->SetDelegate(impl);
 
   // TODO(beng): this reply is currently synchronous, figure out a fix.
   if (was_blocked)
@@ skipping 44 matching lines @@
 
 views::Widget* ViewImpl::GetWidget() {
   return web_view_->GetWidget();
 }
 
 const views::Widget* ViewImpl::GetWidget() const {
   return web_view_->GetWidget();
 }
 
 }  // navigation