NewTabButton should implement MaskedTargeterDelegate

chrome/browser/ui/views/tabs/tab_strip.cc

 // Copyright (c) 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/tabs/tab_strip.h"
 
 #if defined(OS_WIN)
 #include <windowsx.h>
 #endif
 
@@ skipping 28 matching lines @@
 #include "ui/gfx/animation/animation_container.h"
 #include "ui/gfx/animation/throb_animation.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/display.h"
 #include "ui/gfx/image/image_skia.h"
 #include "ui/gfx/image/image_skia_operations.h"
 #include "ui/gfx/path.h"
 #include "ui/gfx/rect_conversions.h"
 #include "ui/gfx/screen.h"
 #include "ui/gfx/size.h"
+#include "ui/gfx/skia_util.h"
 #include "ui/views/controls/image_view.h"
+#include "ui/views/masked_targeter_delegate.h"
 #include "ui/views/mouse_watcher_view_host.h"
 #include "ui/views/rect_based_targeting_utils.h"
 #include "ui/views/view_model_utils.h"
 #include "ui/views/widget/root_view.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/window/non_client_view.h"
 
 #if defined(OS_WIN)
 #include "ui/gfx/win/hwnd_util.h"
 #include "ui/views/widget/monitor_win.h"
@@ skipping 162 matching lines @@
 }
 
 }  // namespace
 
 ///////////////////////////////////////////////////////////////////////////////
 // NewTabButton
 //
 //  A subclass of button that hit-tests to the shape of the new tab button and
 //  does custom drawing.
 
-class NewTabButton : public views::ImageButton {
+class NewTabButton : public views::ImageButton,
+                     public views::MaskedTargeterDelegate {
  public:
   NewTabButton(TabStrip* tab_strip, views::ButtonListener* listener);
   virtual ~NewTabButton();
 
   // Set the background offset used to match the background image to the frame
   // image.
   void set_background_offset(const gfx::Point& offset) {
     background_offset_ = offset;
   }
 
  protected:
-  // Overridden from views::View:
-  virtual bool HasHitTestMask() const OVERRIDE;
-  virtual void GetHitTestMaskDeprecated(HitTestSource source,
-                                        gfx::Path* path) const OVERRIDE;
+  // views::View:
 #if defined(OS_WIN)
   virtual void OnMouseReleased(const ui::MouseEvent& event) OVERRIDE;
 #endif
   virtual void OnPaint(gfx::Canvas* canvas) OVERRIDE;
 
-  // Overridden from ui::EventHandler:
+  // ui::EventHandler:
   virtual void OnGestureEvent(ui::GestureEvent* event) OVERRIDE;
 
  private:
+  // views::MaskedTargeterDelegate:
+  virtual bool GetHitTestMask(gfx::Path* mask) const OVERRIDE;
+
   bool ShouldWindowContentsBeTransparent() const;
   gfx::ImageSkia GetBackgroundImage(views::CustomButton::ButtonState state,
                                     float scale) const;
   gfx::ImageSkia GetImageForState(views::CustomButton::ButtonState state,
                                   float scale) const;
   gfx::ImageSkia GetImageForScale(float scale) const;
 
   // Tab strip that contains this button.
   TabStrip* tab_strip_;
 
@@ skipping 14 matching lines @@
   set_triggerable_event_flags(triggerable_event_flags() |
                               ui::EF_MIDDLE_MOUSE_BUTTON);
 #endif
 }
 
 NewTabButton::~NewTabButton() {
   if (destroyed_)
     *destroyed_ = true;
 }
 
-bool NewTabButton::HasHitTestMask() const {
-  // When the button is sized to the top of the tab strip we want the user to
-  // be able to click on complete bounds, and so don't return a custom hit
-  // mask.
-  return !tab_strip_->SizeTabButtonToTopOfTabStrip();
-}
-
-void NewTabButton::GetHitTestMaskDeprecated(HitTestSource source,
-                                            gfx::Path* path) const {
-  DCHECK(path);
-
-  SkScalar w = SkIntToScalar(width());
-  SkScalar v_offset = SkIntToScalar(TabStrip::kNewTabButtonVerticalOffset);
-
-  // These values are defined by the shape of the new tab image. Should that
-  // image ever change, these values will need to be updated. They're so
-  // custom it's not really worth defining constants for.
-  // These values are correct for regular and USE_ASH versions of the image.
-  path->moveTo(0, v_offset + 1);
-  path->lineTo(w - 7, v_offset + 1);
-  path->lineTo(w - 4, v_offset + 4);
-  path->lineTo(w, v_offset + 16);
-  path->lineTo(w - 1, v_offset + 17);
-  path->lineTo(7, v_offset + 17);
-  path->lineTo(4, v_offset + 13);
-  path->lineTo(0, v_offset + 1);
-  path->close();
-}
-
 #if defined(OS_WIN)
 void NewTabButton::OnMouseReleased(const ui::MouseEvent& event) {
   if (event.IsOnlyRightMouseButton()) {
     gfx::Point point = event.location();
     views::View::ConvertPointToScreen(this, &point);
     bool destroyed = false;
     destroyed_ = &destroyed;
     gfx::ShowSystemMenuAtPoint(views::HWNDForView(this), point);
     if (destroyed)
       return;
@@ skipping 11 matching lines @@
   canvas->DrawImageInt(image, 0, height() - image.height());
 }
 
 void NewTabButton::OnGestureEvent(ui::GestureEvent* event) {
   // Consume all gesture events here so that the parent (Tab) does not
   // start consuming gestures.
   views::ImageButton::OnGestureEvent(event);
   event->SetHandled();
 }
 
+bool NewTabButton::GetHitTestMask(gfx::Path* mask) const {
+  DCHECK(mask);
+
+  // When the button is sized to the top of the tab strip, we want the hit
+  // test mask to be defined as the complete (rectangular) bounds of the
+  // button.
+  if (tab_strip_->SizeTabButtonToTopOfTabStrip()) {
+    gfx::Rect button_bounds(GetContentsBounds());
+    button_bounds.set_x(GetMirroredXForRect(button_bounds));
+    mask->addRect(RectToSkRect(button_bounds));
+    return true;
+  }
+
+  SkScalar w = SkIntToScalar(width());
+  SkScalar v_offset = SkIntToScalar(TabStrip::kNewTabButtonVerticalOffset);
+
+  // These values are defined by the shape of the new tab image. Should that
+  // image ever change, these values will need to be updated. They're so
+  // custom it's not really worth defining constants for.
+  // These values are correct for regular and USE_ASH versions of the image.
+  mask->moveTo(0, v_offset + 1);
+  mask->lineTo(w - 7, v_offset + 1);
+  mask->lineTo(w - 4, v_offset + 4);
+  mask->lineTo(w, v_offset + 16);
+  mask->lineTo(w - 1, v_offset + 17);
+  mask->lineTo(7, v_offset + 17);
+  mask->lineTo(4, v_offset + 13);
+  mask->lineTo(0, v_offset + 1);
+  mask->close();
+
+  return true;
+}
+
 bool NewTabButton::ShouldWindowContentsBeTransparent() const {
   return GetWidget() &&
          GetWidget()->GetTopLevelWidget()->ShouldWindowContentsBeTransparent();
 }
 
 gfx::ImageSkia NewTabButton::GetBackgroundImage(
     views::CustomButton::ButtonState state,
     float scale) const {
   int background_id = 0;
   if (ShouldWindowContentsBeTransparent()) {
@@ skipping 1102 matching lines @@
                                 kNewTabButtonAssetWidth,
                                 kNewTabButtonAssetHeight +
                                     kNewTabButtonVerticalOffset);
   newtab_button_ = new NewTabButton(this, this);
   newtab_button_->SetTooltipText(
       l10n_util::GetStringUTF16(IDS_TOOLTIP_NEW_TAB));
   newtab_button_->SetAccessibleName(
       l10n_util::GetStringUTF16(IDS_ACCNAME_NEWTAB));
   newtab_button_->SetImageAlignment(views::ImageButton::ALIGN_LEFT,
                                     views::ImageButton::ALIGN_BOTTOM);
+  newtab_button_->SetEventTargeter(
+      scoped_ptr<views::ViewTargeter>(new views::ViewTargeter(newtab_button_)));
   AddChildView(newtab_button_);
+
   if (drop_indicator_width == 0) {
     // Direction doesn't matter, both images are the same size.
     gfx::ImageSkia* drop_image = GetDropArrowImage(true);
     drop_indicator_width = drop_image->width();
     drop_indicator_height = drop_image->height();
   }
 }
 
 Tab* TabStrip::CreateTab() {
   Tab* tab = new Tab(this);
@@ skipping 1203 matching lines @@
         action = TouchUMA::GESTURE_TABSWITCH_TAP;
       TouchUMA::RecordGestureAction(action);
       break;
     }
 
     default:
       break;
   }
   event->SetHandled();
 }