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, 0 insertions, 12 deletions

diff --git a/chrome/browser/views/download_tab_view.cc b/chrome/browser/views/download_tab_view.cc
index 61f5164..fb65c1f 100644
--- a/chrome/browser/views/download_tab_view.cc
+++ b/chrome/browser/views/download_tab_view.cc
@@ -670,11 +670,6 @@ void DownloadItemTabView::PaintBackground(ChromeCanvas* canvas) {
   }
 }
 
-void DownloadItemTabView::DidChangeBounds(const CRect& previous,
-                                          const CRect& current) {
-  Layout();
-}
-
 bool DownloadItemTabView::OnMousePressed(const ChromeViews::MouseEvent& event) {
   gfx::Point point(event.location());
 
@@ -819,13 +814,7 @@ void DownloadTabView::UpdateDownloadProgress() {
   SchedulePaint();
 }
 
-void DownloadTabView::DidChangeBounds(const CRect& previous,
-                                      const CRect& current) {
-  Layout();
-}
-
 void DownloadTabView::Layout() {
-  CRect r;
   DetachAllFloatingViews();
   // Dangerous downloads items use NativeButtons, so they need to be attached
   // as NativeControls are not supported yet in floating views.
@@ -844,7 +833,6 @@ void DownloadTabView::Layout() {
   }
   View* v = GetParent();
   if (v) {
-    v->GetLocalBounds(&r, true);
     int h = static_cast<int>(downloads_.size()) *
             (download_util::kBigProgressIconSize + kSpacer) + kSpacer;
     SetBounds(x(), y(), v->width(), h);