Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(255)

Unified Diff: chrome/browser/ui/views/tabs/tab.cc

Issue 265713007: views: Update event-related API to use PointF/RectF instead of Point/Rect. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/tabs/tab.cc
diff --git a/chrome/browser/ui/views/tabs/tab.cc b/chrome/browser/ui/views/tabs/tab.cc
index 9a529f08c15c9217c784f4ecd3718d6352d9bd72..a02a49a9de07e7090765f5a7c7fb8f019ac6457e 100644
--- a/chrome/browser/ui/views/tabs/tab.cc
+++ b/chrome/browser/ui/views/tabs/tab.cc
@@ -240,7 +240,7 @@ class Tab::TabCloseButton : public views::ImageButton {
virtual ~TabCloseButton() {}
// Overridden from views::View.
- virtual View* GetEventHandlerForRect(const gfx::Rect& rect) OVERRIDE {
+ virtual View* GetEventHandlerForRect(const gfx::RectF& rect) OVERRIDE {
if (!views::UsePointBasedTargeting(rect))
return View::GetEventHandlerForRect(rect);
@@ -259,7 +259,7 @@ class Tab::TabCloseButton : public views::ImageButton {
// http://crbug.com/145258
#endif
- return contents_bounds.Intersects(rect) ? this : parent();
+ return gfx::RectF(contents_bounds).Intersects(rect) ? this : parent();
}
// Overridden from views::View.

Powered by Google App Engine
This is Rietveld 408576698