Remove unused ResizeCorner code from BrowserView

chrome/browser/ui/views/frame/browser_view.cc

 // Copyright 2012 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 "chrome/browser/ui/views/frame/browser_view.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/command_line.h"
@@ skipping 360 matching lines @@
     DetachableToolbarView::PaintBackgroundAttachedMode(canvas, host_view_,
         browser_view_->OffsetPointForToolbarBackgroundImage(
             gfx::Point(host_view_->GetMirroredX(), host_view_->y())),
         browser_->host_desktop_type());
     if (host_view_->height() >= toolbar_overlap)
       DetachableToolbarView::PaintHorizontalBorderForState(canvas, host_view_);
   }
 }
 
 ///////////////////////////////////////////////////////////////////////////////
-// ResizeCorner, private:
-
-class ResizeCorner : public views::View {
- public:
-  ResizeCorner() {
-    EnableCanvasFlippingForRTLUI(true);
-  }
-
-  virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE {
-    views::Widget* widget = GetWidget();
-    if (!widget || (widget->IsMaximized() || widget->IsFullscreen()))
-      return;
-
-    gfx::ImageSkia* image = ui::ResourceBundle::GetSharedInstance().
-        GetImageSkiaNamed(IDR_TEXTAREA_RESIZER);
-    canvas->DrawImageInt(*image, width() - image->width(),
-                         height() - image->height());
-  }
-
-  static gfx::Size GetSize() {
-    // This is disabled until we find what makes us slower when we let
-    // WebKit know that we have a resizer rect...
-    // int scrollbar_thickness = gfx::scrollbar_size();
-    // return gfx::Size(scrollbar_thickness, scrollbar_thickness);
-    return gfx::Size();
-  }
-
-  virtual gfx::Size GetPreferredSize() OVERRIDE {
-    views::Widget* widget = GetWidget();
-    return (!widget || widget->IsMaximized() || widget->IsFullscreen()) ?
-        gfx::Size() : GetSize();
-  }
-
-  virtual void Layout() OVERRIDE {
-    if (parent()) {
-      gfx::Size ps = GetPreferredSize();
-      // No need to handle Right to left text direction here,
-      // our parent must take care of it for us...
-      SetBounds(parent()->width() - ps.width(),
-                parent()->height() - ps.height(), ps.width(), ps.height());
-    }
-  }
-
- private:
-  DISALLOW_COPY_AND_ASSIGN(ResizeCorner);
-};
-
-///////////////////////////////////////////////////////////////////////////////
 // BrowserView, public:
 
 BrowserView::BrowserView(Browser* browser)
     : views::ClientView(NULL, NULL),
       last_focused_view_storage_id_(
           views::ViewStorage::GetInstance()->CreateStorageID()),
       frame_(NULL),
       browser_(browser),
       top_container_(NULL),
       tabstrip_(NULL),
@@ skipping 734 matching lines @@
 }
 
 bool BrowserView::IsToolbarVisible() const {
   if (immersive_mode_controller_->ShouldHideTopViews())
     return false;
   return browser_->SupportsWindowFeature(Browser::FEATURE_TOOLBAR) ||
          browser_->SupportsWindowFeature(Browser::FEATURE_LOCATIONBAR);
 }
 
 gfx::Rect BrowserView::GetRootWindowResizerRect() const {
-  if (frame_->IsMaximized() || frame_->IsFullscreen())
-    return gfx::Rect();
-
-  // We don't specify a resize corner size if we have a bottom shelf either.
-  // This is because we take care of drawing the resize corner on top of that
-  // shelf, so we don't want others to do it for us in this case.
-  // Currently, the only visible bottom shelf is the download shelf.
-  // Other tests should be added here if we add more bottom shelves.
-  if (IsDownloadShelfVisible())
-    return gfx::Rect();
-
-  gfx::Rect client_rect = contents_split_->bounds();
-  gfx::Size resize_corner_size = ResizeCorner::GetSize();
-  int x = client_rect.width() - resize_corner_size.width();
-  if (base::i18n::IsRTL())
-    x = 0;
-  return gfx::Rect(x, client_rect.height() - resize_corner_size.height(),
-                   resize_corner_size.width(), resize_corner_size.height());
+  // Views does not support resizer rects because they caused page cycler
+  // performance regressions when they were added. See crrev.com/9654
+  return gfx::Rect();
 }
 
 void BrowserView::DisableInactiveFrame() {
 #if defined(OS_WIN) && !defined(USE_AURA)
   frame_->DisableInactiveRendering();
 #endif  // No tricks are needed to get the right behavior on Linux.
 }
 
 void BrowserView::ConfirmAddSearchProvider(TemplateURL* template_url,
                                            Profile* profile) {
@@ skipping 653 matching lines @@
 
   // Empty TabStripModel, it's now safe to allow the Window to be closed.
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_WINDOW_CLOSED,
       content::Source<gfx::NativeWindow>(frame_->GetNativeWindow()),
       content::NotificationService::NoDetails());
   return true;
 }
 
 int BrowserView::NonClientHitTest(const gfx::Point& point) {
-#if defined(OS_WIN) && !defined(USE_AURA)
-  // The following code is not in the LayoutManager because it's
-  // independent of layout and also depends on the ResizeCorner which
-  // is private.
-  if (!frame_->IsMaximized() && !frame_->IsFullscreen()) {
-    CRect client_rect;
-    ::GetClientRect(frame_->GetNativeWindow(), &client_rect);
-    gfx::Size resize_corner_size = ResizeCorner::GetSize();
-    gfx::Rect resize_corner_rect(client_rect.right - resize_corner_size.width(),
-        client_rect.bottom - resize_corner_size.height(),
-        resize_corner_size.width(), resize_corner_size.height());
-    bool rtl_dir = base::i18n::IsRTL();
-    if (rtl_dir)
-      resize_corner_rect.set_x(0);
-    if (resize_corner_rect.Contains(point)) {
-      if (rtl_dir)
-        return HTBOTTOMLEFT;
-      return HTBOTTOMRIGHT;
-    }
-  }
-#endif
-
   return GetBrowserViewLayout()->NonClientHitTest(point);
 }
 
 gfx::Size BrowserView::GetMinimumSize() {
   return GetBrowserViewLayout()->GetMinimumSize();
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // BrowserView, views::View overrides:
 
@@ skipping 679 matching lines @@
       else if (key_code == ui::VKEY_K)
         content::RecordAction(UserMetricsAction("Accel_FocusSearch_K"));
       break;
     default:
       // Do nothing.
       break;
   }
 #endif
 }
 
-gfx::Size BrowserView::GetResizeCornerSize() const {
-  return ResizeCorner::GetSize();
-}
-
 void BrowserView::CreateLauncherIcon() {
 #if defined(USE_ASH)
   if (chrome::IsNativeWindowInAsh(GetNativeWindow()) &&
       !launcher_item_controller_.get()) {
     launcher_item_controller_.reset(
         BrowserLauncherItemController::Create(browser_.get()));
   }
 #endif  // defined(USE_ASH)
 }
 
@@ skipping 118 matching lines @@
 
   Browser* modal_browser =
       chrome::FindBrowserWithWebContents(active_dialog->web_contents());
   if (modal_browser && (browser_ != modal_browser)) {
     modal_browser->window()->FlashFrame(true);
     modal_browser->window()->Activate();
   }
 
   AppModalDialogQueue::GetInstance()->ActivateModalDialog();
 }