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

Unified Diff: chrome/browser/ui/views/frame/opaque_browser_frame_view.cc

Issue 2897533003: Change default ink drop center points to center of contents bounds (Closed)
Patch Set: fix compile Created 3 years, 7 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/frame/opaque_browser_frame_view.cc
diff --git a/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc b/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc
index c68146b2931e5163674a44b87c5f3adb65cd7543..09b9db83f04b026ecf49f357136c77f4b070f53b 100644
--- a/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc
+++ b/chrome/browser/ui/views/frame/opaque_browser_frame_view.cc
@@ -202,7 +202,7 @@ int OpaqueBrowserFrameView::NonClientHitTest(const gfx::Point& point) {
// of Fitts' Law.
if (layout_->IsTitleBarCondensed())
sysmenu_rect.SetRect(0, 0, sysmenu_rect.right(), sysmenu_rect.bottom());
- sysmenu_rect.set_x(GetMirroredXForRect(sysmenu_rect));
+ sysmenu_rect = GetMirroredRect(sysmenu_rect);
if (sysmenu_rect.Contains(point))
return (frame_component == HTCLIENT) ? HTCLIENT : HTSYSMENU;
@@ -574,8 +574,8 @@ void OpaqueBrowserFrameView::PaintToolbarBackground(gfx::Canvas* canvas) const {
// Top stroke.
gfx::Rect separator_rect(x, y, w, 0);
gfx::ScopedCanvas scoped_canvas(canvas);
- gfx::Rect tabstrip_bounds(GetBoundsForTabStrip(browser_view()->tabstrip()));
- tabstrip_bounds.set_x(GetMirroredXForRect(tabstrip_bounds));
+ gfx::Rect tabstrip_bounds =
+ GetMirroredRect(GetBoundsForTabStrip(browser_view()->tabstrip()));
canvas->sk_canvas()->clipRect(gfx::RectToSkRect(tabstrip_bounds),
SkClipOp::kDifference);
separator_rect.set_y(tabstrip_bounds.bottom());
« no previous file with comments | « chrome/browser/ui/views/frame/glass_browser_frame_view.cc ('k') | chrome/browser/ui/views/status_bubble_views.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698