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, 4 insertions, 4 deletions

diff --git a/chrome/browser/history_view.cc b/chrome/browser/history_view.cc
index ef439eb..7ff4225 100644
--- a/chrome/browser/history_view.cc
+++ b/chrome/browser/history_view.cc
@@ -840,7 +840,8 @@ bool HistoryView::IsVisible() {
   return vc && vc->IsVisible();
 }
 
-void HistoryView::DidChangeBounds(const CRect& previous, const CRect& current) {
+void HistoryView::DidChangeBounds(const gfx::Rect& previous,
+                                  const gfx::Rect& current) {
   SchedulePaint();
 }
 
@@ -851,14 +852,13 @@ void HistoryView::Layout() {
   if (!parent)
     return;
 
-  CRect bounds;
-  parent->GetLocalBounds(&bounds, true);
+  gfx::Rect bounds = parent->GetLocalBounds(true);
 
   // If not visible, have zero size so we don't compute anything.
   int width = 0;
   int height = 0;
   if (IsVisible()) {
-    width = bounds.Width();
+    width = bounds.width();
     height = std::max(GetLastEntryMaxY(),
                       kEntryPadding + kNoResultTextHeight);
   }