Fix alignment of avatar bubbles in the NTP

chrome/browser/ui/views/page_info_bubble_view.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/views/page_info_bubble_view.h"
 
 #include <algorithm>
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/certificate_viewer.h"
@@ skipping 280 matching lines @@
   // animation.
   // TODO(derat): Remove this once we're not using a toplevel X window for the
   // bubble.
   SizeToContents();
 #else
   resize_animation_.SetSlideDuration(kPageInfoSlideDuration);
   resize_animation_.Show();
 #endif
 }
 
-gfx::Point PageInfoBubbleView::GetAnchorPoint() {
+gfx::Rect PageInfoBubbleView::GetAnchorRect() {
   // Compensate for some built-in padding in the icon.
-  gfx::Point anchor(BubbleDelegateView::GetAnchorPoint());
-  return anchor_view() ? anchor.Subtract(gfx::Point(0, 5)) : anchor;
+  gfx::Rect anchor(BubbleDelegateView::GetAnchorRect());
+  if (anchor_view())
+    anchor.Offset(0, -5);

[msw, 2011/12/08 00:40:14]

Please change this to rect.Inset(5, 5); Thanks!

+  return anchor;
 }
 
 void PageInfoBubbleView::LinkClicked(views::Link* source, int event_flags) {
   GURL url = google_util::AppendGoogleLocaleParam(
       GURL(chrome::kPageInfoHelpCenterURL));
   Browser* browser = BrowserList::GetLastActive();
   browser->OpenURL(
       url, GURL(), NEW_FOREGROUND_TAB, content::PAGE_TRANSITION_LINK);
   // NOTE: The bubble closes automatically on deactivation as the link opens.
 }
@@ skipping 161 matching lines @@
                         const GURL& url,
                         const NavigationEntry::SSLStatus& ssl,
                         bool show_history) {
   PageInfoBubbleView* page_info_bubble =
       new PageInfoBubbleView(anchor_view, profile, url, ssl, show_history);
   browser::CreateViewsBubble(page_info_bubble);
   page_info_bubble->Show();
 }
 
 }