views: Update event-related API to use PointF/RectF instead of Point/Rect.

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 794 matching lines @@
 }
 
 bool TabStrip::IsTabStripCloseable() const {
   return !IsDragSessionActive();
 }
 
 void TabStrip::UpdateLoadingAnimations() {
   controller_->UpdateLoadingAnimations();
 }
 
-bool TabStrip::IsPositionInWindowCaption(const gfx::Point& point) {
-  return IsRectInWindowCaption(gfx::Rect(point, gfx::Size(1, 1)));
+bool TabStrip::IsPositionInWindowCaption(const gfx::PointF& point) {
+  return IsRectInWindowCaption(gfx::RectF(point, gfx::Size(1, 1)));
 }
 
-bool TabStrip::IsRectInWindowCaption(const gfx::Rect& rect) {
+bool TabStrip::IsRectInWindowCaption(const gfx::RectF& rect) {
   views::View* v = GetEventHandlerForRect(rect);
 
   // If there is no control at this location, claim the hit was in the title
   // bar to get a move action.
   if (v == this)
     return true;
 
   // Check to see if the rect intersects the non-button parts of the new tab
   // button. The button has a non-rectangular shape, so if it's not in the
   // visual portions of the button we treat it as a click to the caption.
-  gfx::RectF rect_in_newtab_coords_f(rect);
-  View::ConvertRectToTarget(this, newtab_button_, &rect_in_newtab_coords_f);
-  gfx::Rect rect_in_newtab_coords = gfx::ToEnclosingRect(
-      rect_in_newtab_coords_f);
-  if (newtab_button_->GetLocalBounds().Intersects(rect_in_newtab_coords) &&
+  gfx::RectF rect_in_newtab_coords(rect);
+  View::ConvertRectToTarget(this, newtab_button_, &rect_in_newtab_coords);
+  gfx::RectF newtab_bounds(newtab_button_->GetLocalBounds());
+  if (newtab_bounds.Intersects(rect_in_newtab_coords) &&
       !newtab_button_->HitTestRect(rect_in_newtab_coords))
     return true;
 
   // All other regions, including the new Tab button, should be considered part
   // of the containing Window's client area so that regular events can be
   // processed for them.
   return false;
 }
 
 void TabStrip::SetBackgroundOffset(const gfx::Point& offset) {
@@ skipping 485 matching lines @@
 
   controller()->PerformDrop(drop_before, drop_index, url);
 
   return GetDropEffect(event);
 }
 
 void TabStrip::GetAccessibleState(ui::AXViewState* state) {
   state->role = ui::AX_ROLE_TAB_LIST;
 }
 
-views::View* TabStrip::GetEventHandlerForRect(const gfx::Rect& rect) {
+views::View* TabStrip::GetEventHandlerForRect(const gfx::RectF& rect) {
   if (!views::UsePointBasedTargeting(rect))
     return View::GetEventHandlerForRect(rect);
-  const gfx::Point point(rect.CenterPoint());
+  const gfx::Point point(gfx::ToFlooredPoint(rect.CenterPoint()));
 
   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::GetEventHandlerForRect(rect);
     if (v && v != this && strcmp(v->GetClassName(), Tab::kViewClassName))
       return v;
 
     views::View* tab = FindTabHitByPoint(point);
     if (tab)
@@ skipping 1289 matching lines @@
   else
     RemoveMessageLoopObserver();
 }
 
 bool TabStrip::GetAdjustLayout() const {
   if (!adjust_layout_)
     return false;
   return chrome::GetHostDesktopTypeForNativeView(
       GetWidget()->GetNativeView()) == chrome::HOST_DESKTOP_TYPE_ASH;
 }