Make View::SetBounds take a const gfx::Rect& instead of a const CRect&

Make View::DidChangeBounds call Layout by default, eliminating this function from most places.

http://crbug.com/2186

Review URL: http://codereview.chromium.org/7429

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@3471 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 5 insertions, 6 deletions

diff --git a/chrome/browser/automation/automation_provider.cc b/chrome/browser/automation/automation_provider.cc
index e4b4bf6..716e3e1 100644
--- a/chrome/browser/automation/automation_provider.cc
+++ b/chrome/browser/automation/automation_provider.cc
@@ -1136,8 +1136,7 @@ void AutomationProvider::WindowGetViewBounds(const IPC::Message& message,
                                              int view_id,
                                              bool screen_coordinates) {
   bool succeeded = false;
-  CRect bounds;
-  bounds.SetRect(0, 0, 0, 0);
+  gfx::Rect bounds;
 
   void* iter = NULL;
   if (window_tracker_->ContainsHandle(handle)) {
@@ -1153,14 +1152,14 @@ void AutomationProvider::WindowGetViewBounds(const IPC::Message& message,
           ChromeViews::View::ConvertPointToScreen(view, &point);
         else
           ChromeViews::View::ConvertPointToView(view, root_view, &point);
-        view->GetLocalBounds(&bounds, false);
-        bounds.MoveToXY(point.x(), point.y());
+        bounds = view->GetLocalBounds(false);
+        bounds.set_origin(point);
       }
     }
   }
 
-  Send(new AutomationMsg_WindowViewBoundsResponse(
-           message.routing_id(), succeeded, gfx::Rect(bounds)));
+  Send(new AutomationMsg_WindowViewBoundsResponse(message.routing_id(),
+                                                  succeeded, bounds));
 }
 
 // This task enqueues a mouse event on the event loop, so that the view