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

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

Issue 11269022: Add Vector2d classes that represent offsets, instead of using Point. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: RenderText fixup Created 8 years, 2 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/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index c04501bfe0bb3da3aeb890c47b41c0c829dc812c..59eda6790940c599f8e81e82eab4568c10bea1b7 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -459,7 +459,8 @@ gfx::Point BrowserView::OffsetPointForToolbarBackgroundImage(
// The background image starts tiling horizontally at the window left edge and
// vertically at the top edge of the horizontal tab strip (or where it would
// be). We expect our parent's origin to be the window origin.
- gfx::Point window_point(point.Add(GetMirroredPosition()));
+ gfx::Point window_point(
+ point.Add(GetMirroredPosition().OffsetFromOrigin()));
window_point.Offset(frame_->GetThemeBackgroundXInset(),
-frame_->GetTabStripInsets(false).top);
return window_point;

Powered by Google App Engine
This is Rietveld 408576698