ui::LocatedEvent location() returns gfx::PointF

ui/views/controls/button/menu_button.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/controls/button/menu_button.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "grit/ui_resources.h"
 #include "grit/ui_strings.h"
 #include "ui/accessibility/ax_view_state.h"
@@ skipping 148 matching lines @@
 const char* MenuButton::GetClassName() const {
   return kViewClassName;
 }
 
 bool MenuButton::OnMousePressed(const ui::MouseEvent& event) {
   RequestFocus();
   if (state() != STATE_DISABLED) {
     // If we're draggable (GetDragOperations returns a non-zero value), then
     // don't pop on press, instead wait for release.
     if (event.IsOnlyLeftMouseButton() &&
-        HitTestPoint(event.location()) &&
-        GetDragOperations(event.location()) == ui::DragDropTypes::DRAG_NONE) {
+        HitTestPoint(gfx::ToFlooredPoint(event.location())) &&
+        GetDragOperations(gfx::ToFlooredPoint(event.location())) ==
+            ui::DragDropTypes::DRAG_NONE) {
       TimeDelta delta = TimeTicks::Now() - menu_closed_time_;
       if (delta.InMilliseconds() > kMinimumMsBetweenButtonClicks)
         return Activate();
     }
   }
   return true;
 }
 
 void MenuButton::OnMouseReleased(const ui::MouseEvent& event) {
   // Explicitly test for left mouse button to show the menu. If we tested for
   // !IsTriggerableEvent it could lead to a situation where we end up showing
   // the menu and context menu (this would happen if the right button is not
   // triggerable and there's a context menu).
-  if (GetDragOperations(event.location()) != ui::DragDropTypes::DRAG_NONE &&
+  if (GetDragOperations(gfx::ToFlooredPoint(event.location())) !=
+          ui::DragDropTypes::DRAG_NONE &&
       state() != STATE_DISABLED && !InDrag() && event.IsOnlyLeftMouseButton() &&
-      HitTestPoint(event.location())) {
+      HitTestPoint(gfx::ToFlooredPoint(event.location()))) {
     Activate();
   } else {
     TextButton::OnMouseReleased(event);
   }
 }
 
 // The reason we override View::OnMouseExited is because we get this event when
 // we display the menu. If we don't override this method then
 // BaseButton::OnMouseExited will get the event and will set the button's state
 // to STATE_NORMAL instead of keeping the state BM_PUSHED. This, in turn, will
@@ skipping 70 matching lines @@
   if (!GetWidget()) {
     NOTREACHED();
     return 0;
   }
 
   gfx::Rect monitor_bounds = GetWidget()->GetWorkAreaBoundsInScreen();
   return monitor_bounds.right() - 1;
 }
 
 }  // namespace views