Add support for rect-based event targeting in 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/message_loop.h"
@@ skipping 260 matching lines @@
   // events are not dispatched to any views.
   switch (event->type()) {
     case ui::ET_GESTURE_SCROLL_UPDATE:
     case ui::ET_GESTURE_SCROLL_END:
     case ui::ET_SCROLL_FLING_START:
       return;
     default:
       break;
   }
 
+  // Terry: A runtime flag will specify whether or not fuzzing should be

[tdanderson, 2013/10/07 21:35:21]

I still have to define a --use-rect-based-views-targeting flag and replace
|use_fuzzing| with a check for the existence of this flag.

Since there is no *_switches.{h,cc} files in ui/views/widget/ or any of its
parent directories, is it appropriate to create the new files
ui/views/views_switches.{h,cc} to define the new flag? Or would you prefer
another way?

+  // used here (and will possibly also specify an overlap percentage).
+  bool use_fuzzing = true;
+
+  View* gesture_handler = NULL;
+  if (use_fuzzing) {
+    // Terry: account for any possible transformations on rect?
+    gfx::Rect touch_rect(event->details().bounding_box());
+    gfx::Point origin(touch_rect.origin());
+    View::ConvertPointFromScreen(this, &origin);
+    touch_rect.set_origin(origin);
+    gesture_handler = GetEventHandlerForRect(touch_rect);
+  } else {
+    gesture_handler = GetEventHandlerForPoint(event->location());
+  }
+
   // Walk up the tree until we find a view that wants the gesture event.
-  for (gesture_handler_ = GetEventHandlerForPoint(event->location());
-      gesture_handler_ && (gesture_handler_ != this);
-      gesture_handler_ = gesture_handler_->parent()) {
+  for (gesture_handler_ = gesture_handler;
+       gesture_handler_ && (gesture_handler_ != this);
+       gesture_handler_ = gesture_handler_->parent()) {
     if (!gesture_handler_->enabled()) {
       // Disabled views eat events but are treated as not handled.
       return;
     }
 
     // See if this view wants to handle the Gesture.
     ui::GestureEvent gesture_event(*event, static_cast<View*>(this),
                                    gesture_handler_);
     DispatchEventToTarget(gesture_handler_, &gesture_event);
 
@@ skipping 419 matching lines @@
       return;
   }
 }
 
 bool RootView::CanDispatchToTarget(ui::EventTarget* target) {
   return event_dispatch_target_ == target;
 }
 
 }  // namespace internal
 }  // namespace views