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

Unified Diff: chrome/browser/ui/views/first_run_bubble.cc

Issue 8863009: Fix alignment of avatar bubbles in the NTP (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: use Rect::Ofset() Created 9 years 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/first_run_bubble.cc
diff --git a/chrome/browser/ui/views/first_run_bubble.cc b/chrome/browser/ui/views/first_run_bubble.cc
index 42a0353d25723c71e8cb7b8c62022d9a4d11fea5..1bdb31fb8236d2fc09b7228e6b36f0e88570877a 100644
--- a/chrome/browser/ui/views/first_run_bubble.cc
+++ b/chrome/browser/ui/views/first_run_bubble.cc
@@ -97,10 +97,11 @@ void FirstRunBubble::Init() {
layout->AddView(label2);
}
-gfx::Point FirstRunBubble::GetAnchorPoint() {
+gfx::Rect FirstRunBubble::GetAnchorRect() {
// Compensate for padding in anchor.
- return BubbleDelegateView::GetAnchorPoint().Add(
- gfx::Point(0, anchor_view() ? kAnchorVerticalOffset : 0));
+ gfx::Rect rect(BubbleDelegateView::GetAnchorRect());
+ rect.Offset(0, anchor_view() ? kAnchorVerticalOffset : 0);
msw 2011/12/08 00:40:14 Please change this to: rect.Inset(0, anchor_view()
+ return rect;
}
FirstRunBubble::FirstRunBubble(

Powered by Google App Engine
This is Rietveld 408576698