Add gesture target fuzzing to views

ui/views/widget/root_view.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 "ui/views/widget/root_view.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "base/message_loop.h"
@@ skipping 437 matching lines @@
   // Reset touch_pressed_handler_ to indicate that no processing is occurring.
   touch_pressed_handler_ = NULL;
 
   return status;
 }
 
 ui::GestureStatus RootView::OnGestureEvent(const GestureEvent& event) {
   GestureEvent e(event, this);
   ui::GestureStatus status = ui::GESTURE_STATUS_UNKNOWN;
 
+  if (!gesture_handler_) {

[sadrul, 2012/07/26 18:41:28]

This is not the right place. The event will not bubble up if you do this here.
This code should be after line 509. Instead of using GetEventHandlerForPoint in
the loop, it will use GetEventHandlerForRect

[tdanderson, 2012/07/27 15:27:04]

Done.

+    gfx::Rect bounding_box(event.details().bounding_box());
+    gfx::Point bounding_box_location(bounding_box.x(), bounding_box.y());
+    ConvertPointToScreen(this, &bounding_box_location);
+    gfx::Rect touch_rect(bounding_box_location.x(),
+                         bounding_box_location.y(),
+                         bounding_box.width(),
+                         bounding_box.height());
+    gesture_handler_ = GetEventHandlerForRect(touch_rect);

[sadrul, 2012/07/26 18:41:28]

You could perhaps use bounding_box.set_origin here instead of creating
touch_rect

[tdanderson, 2012/07/27 15:27:04]

Done.

+  }
+
   if (gesture_handler_) {
     // |gesture_handler_| (or |scroll_gesture_handler_|) can be deleted during
     // processing.
     View* handler = event.IsScrollGestureEvent() && scroll_gesture_handler_ ?
         scroll_gesture_handler_ : gesture_handler_;
     GestureEvent handler_event(event, this, handler);
 
     ui::GestureStatus status = handler->ProcessGestureEvent(handler_event);
 
     if (event.type() == ui::ET_GESTURE_END &&
@@ skipping 131 matching lines @@
 }
 
 void RootView::SetMouseLocationAndFlags(const MouseEvent& event) {
   last_mouse_event_flags_ = event.flags();
   last_mouse_event_x_ = event.x();
   last_mouse_event_y_ = event.y();
 }
 
 }  // namespace internal
 }  // namespace views