Views fuzzing for Aura and Windows

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 249 matching lines @@
   virtual void AnimationCanceled(const ui::Animation* animation) {
     tab_->set_dragging(false);
   }
 
  private:
   BaseTab* tab_;
 
   DISALLOW_COPY_AND_ASSIGN(ResetDraggingStateDelegate);
 };
 
-// If |dest| contains the point |point_in_source| the event handler from |dest|
+// If |dest| contains the rect |rect_in_source| the event handler from |dest|
 // is returned. Otherwise NULL is returned.
-views::View* ConvertPointToViewAndGetEventHandler(
+views::View* ConvertRectToViewAndGetEventHandler(
     views::View* source,
     views::View* dest,
-    const gfx::Point& point_in_source) {
-  gfx::Point dest_point(point_in_source);
-  views::View::ConvertPointToTarget(source, dest, &dest_point);
-  return dest->HitTestPoint(dest_point) ?
-      dest->GetEventHandlerForPoint(dest_point) : NULL;
+    const gfx::Rect& rect_in_source,
+    views::View::EventType type) {
+  gfx::Rect dest_rect(rect_in_source);
+  views::View::ConvertRectToTarget(source, dest, &dest_rect);
+  return dest->HitTestRect(dest_rect) ?
+      dest->GetEventHandler(dest_rect, type) : NULL;
 }
 
 }  // 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 {
  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;
   }
 
+  View* GetEventHandler(const gfx::Rect& rect, EventType type) OVERRIDE {
+    return View::GetEventHandler(rect, type);
+  }
+
  protected:
   // Overridden from views::View:
   virtual bool HasHitTestMask() const OVERRIDE;
   virtual void GetHitTestMask(gfx::Path* path) const OVERRIDE;
 #if defined(OS_WIN) && !defined(USE_AURA)
   void OnMouseReleased(const ui::MouseEvent& event) OVERRIDE;
 #endif
   virtual ui::EventResult OnGestureEvent(
       const ui::GestureEvent& event) OVERRIDE;
   void OnPaint(gfx::Canvas* canvas) OVERRIDE;
@@ skipping 1022 matching lines @@
 
   controller()->PerformDrop(drop_before, drop_index, url);
 
   return GetDropEffect(event);
 }
 
 void TabStrip::GetAccessibleState(ui::AccessibleViewState* state) {
   state->role = ui::AccessibilityTypes::ROLE_PAGETABLIST;
 }
 
-views::View* TabStrip::GetEventHandlerForPoint(const gfx::Point& point) {
+double CoverageProportion(const gfx::Rect& rect_one,

[girard, 2012/09/18 16:58:52]

Move this into the anonymous namespace at the top of the file.

+                          const gfx::Rect& rect_two) {
+  gfx::Rect intersection = rect_one.Intersect(rect_two);
+  double rect_one_area = rect_one.size().GetArea();
+  double rect_two_area = rect_two.size().GetArea();
+  double intersection_area = intersection.size().GetArea();
+  if ((rect_one_area <= 0) || (rect_two_area <= 0))
+    return 0.0;
+  if (rect_one_area > rect_two_area)
+    return intersection_area / rect_two_area;
+  else
+    return intersection_area / rect_one_area;
+}
+
+views::View* TabStrip::GetEventHandler(const gfx::Rect& rect,
+                                       EventType type) {
+#if defined(OS_WIN)
+  if (ui::IsMouseEventFromTouch(WM_MOUSEMOVE) && (type == MOUSE)) {
+    // This MOUSE event was actually generated from a touch.
+    // Views doesn't currently enable Windows OS touch.
+    int touch_size = 24;

[tdanderson, 2012/09/18 19:19:02]

I'm not sure where this 24 comes from. Can you please include a comment?

+    return GetEventHandler(gfx::Rect(rect.x()-touch_size, rect.y()-touch_size,
+                           touch_size*2, touch_size*2), TOUCH);
+  }
+#endif
+
+  View* closest_view = NULL;
+  View* closest_untouched_view = NULL;
+  double max_proportion = 0;
+  int closest_distance = INT_MAX;

[tdanderson, 2012/09/18 19:19:02]

Are you planning on using these in the same way as in view.cc?

+
   if (!touch_layout_.get()) {
     // Return any view that isn't a Tab or this TabStrip immediately. We don't
     // want to interfere.
-    views::View* v = View::GetEventHandlerForPoint(point);
+    views::View* v = View::GetEventHandler(rect, type);
     if (v && v != this && v->GetClassName() != Tab::kViewClassName)
       return v;
 
     // The display order doesn't necessarily match the child list order, so we
     // walk the display list hit-testing Tabs. Since the active tab always
     // renders on top of adjacent tabs, it needs to be hit-tested before any
     // left-adjacent Tab, so we look ahead for it as we walk.
     for (int i = 0; i < tab_count(); ++i) {
       Tab* next_tab = i < (tab_count() - 1) ? tab_at(i + 1) : NULL;
-      if (next_tab && next_tab->IsActive() && IsPointInTab(next_tab, point))
+      if (next_tab && next_tab->IsActive() && IsRectInTab(next_tab, rect))
         return next_tab;
-      if (IsPointInTab(tab_at(i), point))
+      if (IsRectInTab(tab_at(i), rect))
         return tab_at(i);
     }
   } else {
+    // Test for new tab button....
     if (newtab_button_->visible()) {
       views::View* view =
-          ConvertPointToViewAndGetEventHandler(this, newtab_button_, point);
+          ConvertRectToViewAndGetEventHandler(
+              this, newtab_button_, rect, type);
       if (view)

[girard, 2012/09/18 16:58:52]

This should only return if MOUSE.  Otherwise continue considering other options.

         return view;
     }
-    Tab* tab = FindTabForEvent(point);
+    // Otherwise, check for other tabs....
+    Tab* tab = FindTabForEvent(rect);

[girard, 2012/09/18 16:58:52]

This logic should be expanded here, so we can consider all the alternatives.

     if (tab)
-      return ConvertPointToViewAndGetEventHandler(this, tab, point);
+      return ConvertRectToViewAndGetEventHandler(this, tab, rect, type);

[girard, 2012/09/18 16:58:52]

Ditto.

   }
   return this;
 }
 
 int TabStrip::GetMiniTabCount() const {
   int mini_count = 0;
   while (mini_count < tab_count() && tab_at(mini_count)->data().mini)
     mini_count++;
   return mini_count;
 }
@@ skipping 963 matching lines @@
       true);
 }
 
 bool TabStrip::IsPointInTab(Tab* tab,
                             const gfx::Point& point_in_tabstrip_coords) {
   gfx::Point point_in_tab_coords(point_in_tabstrip_coords);
   View::ConvertPointToTarget(this, tab, &point_in_tab_coords);
   return tab->HitTestPoint(point_in_tab_coords);
 }
 
+bool TabStrip::IsRectInTab(Tab* tab,
+                           const gfx::Rect& rect_in_tabstrip_coords) {
+  gfx::Rect rect_in_tab_coords(rect_in_tabstrip_coords);
+  View::ConvertRectToTarget(this, tab, &rect_in_tab_coords);
+  return tab->HitTestRect(rect_in_tab_coords);
+}
+
 int TabStrip::GetStartXForNormalTabs() const {
   int mini_tab_count = GetMiniTabCount();
   if (mini_tab_count == 0)
     return 0;
   return mini_tab_count * (Tab::GetMiniWidth() + tab_h_offset()) +
       kMiniToNonMiniGap;
 }
 
+Tab* TabStrip::FindTabForEvent(const gfx::Rect& rect) {
+  DCHECK(touch_layout_.get());
+  int active_tab_index = touch_layout_->active_index();
+  Tab* tab = NULL;
+  if (active_tab_index != -1) {
+    tab = FindTabForEventFrom(rect, active_tab_index, -1);
+    if (!tab)
+      tab = FindTabForEventFrom(rect, active_tab_index + 1, 1);
+  } else if (tab_count()) {
+    tab = FindTabForEventFrom(rect, 0, 1);
+  }
+  return tab;
+}
+
 Tab* TabStrip::FindTabForEvent(const gfx::Point& point) {
   DCHECK(touch_layout_.get());
   int active_tab_index = touch_layout_->active_index();
   Tab* tab = NULL;
   if (active_tab_index != -1) {
     tab = FindTabForEventFrom(point, active_tab_index, -1);
     if (!tab)
       tab = FindTabForEventFrom(point, active_tab_index + 1, 1);
   } else if (tab_count()) {
     tab = FindTabForEventFrom(point, 0, 1);
   }
   return tab;
 }
 
+Tab* TabStrip::FindTabForEventFrom(const gfx::Rect& rect,
+                                   int start,
+                                   int delta) {
+  // |start| equals tab_count() when there are only pinned tabs.
+  if (start == tab_count())
+    start += delta;
+  for (int i = start; i >= 0 && i < tab_count(); i += delta) {
+    if (IsRectInTab(tab_at(i), rect))
+      return tab_at(i);
+  }
+  return NULL;
+}
+
 Tab* TabStrip::FindTabForEventFrom(const gfx::Point& point,
                                    int start,
                                    int delta) {
   // |start| equals tab_count() when there are only pinned tabs.
   if (start == tab_count())
     start += delta;
   for (int i = start; i >= 0 && i < tab_count(); i += delta) {
     if (IsPointInTab(tab_at(i), point))
       return tab_at(i);
   }
@@ skipping 71 matching lines @@
   if (!adjust_layout_)
     return false;
 
 #if !defined(OS_CHROMEOS)
   if (ui::GetDisplayLayout() != ui::LAYOUT_TOUCH)
     return false;
 #endif
 
   return true;
 }