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

Unified Diff: chrome/browser/ui/views/content_setting_bubble_contents.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/content_setting_bubble_contents.cc
diff --git a/chrome/browser/ui/views/content_setting_bubble_contents.cc b/chrome/browser/ui/views/content_setting_bubble_contents.cc
index 993cb292bf48907cf1ad01d4889d8a633fac49f4..dd4bcd397a1c8253722e672d43b3f416de838dcf 100644
--- a/chrome/browser/ui/views/content_setting_bubble_contents.cc
+++ b/chrome/browser/ui/views/content_setting_bubble_contents.cc
@@ -133,8 +133,10 @@ gfx::Size ContentSettingBubbleContents::GetPreferredSize() {
return preferred_size;
}
-gfx::Point ContentSettingBubbleContents::GetAnchorPoint() {
- return BubbleDelegateView::GetAnchorPoint().Subtract(gfx::Point(0, 5));
+gfx::Rect ContentSettingBubbleContents::GetAnchorRect() {
+ gfx::Rect rect(BubbleDelegateView::GetAnchorRect());
+ rect.Offset(0, -5);
msw 2011/12/08 00:40:14 Please change this to rect.Inset(5, 5);? Thanks!
+ return rect;
}
void ContentSettingBubbleContents::Init() {

Powered by Google App Engine
This is Rietveld 408576698