Avoid overriding Bubble's GetAnchorRect() where possible.

chrome/browser/ui/views/network_profile_bubble_view.cc

 // Copyright (c) 2012 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/network_profile_bubble_view.h"
 
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/network_profile_bubble.h"
@@ skipping 42 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkProfileBubbleView, public:
 
 NetworkProfileBubbleView::NetworkProfileBubbleView(
     views::View* anchor,
     content::PageNavigator* navigator,
     Profile* profile)
     : BubbleDelegateView(anchor, views::BubbleBorder::TOP_RIGHT),
       navigator_(navigator),
       profile_(profile) {
+  // Compensate for built-in vertical padding in the anchor view's image.
+  set_anchor_insets(
+      gfx::Insets(kAnchorVerticalInset, 0, kAnchorVerticalInset, 0));
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // NetworkProfileBubbleView, private:
 
 NetworkProfileBubbleView::~NetworkProfileBubbleView() {
 }
 
 void NetworkProfileBubbleView::Init() {
   views::GridLayout* layout = views::GridLayout::CreatePanel(this);
@@ skipping 27 matching lines @@
       new views::Link(l10n_util::GetStringUTF16(IDS_LEARN_MORE));
   learn_more->set_listener(this);
   layout->AddView(learn_more);
 
   views::NativeTextButton* ok_button = new views::NativeTextButton(
       this, l10n_util::GetStringUTF16(IDS_OK));
   ok_button->SetIsDefault(true);
   layout->AddView(ok_button);
 }
 
-gfx::Rect NetworkProfileBubbleView::GetAnchorRect() {
-  // Compensate for padding in anchor.
-  gfx::Rect rect(BubbleDelegateView::GetAnchorRect());
-  rect.Inset(0, anchor_view() ? kAnchorVerticalInset : 0);
-  return rect;
-}
-
 void NetworkProfileBubbleView::ButtonPressed(views::Button* sender,
                                              const ui::Event& event) {
   NetworkProfileBubble::RecordUmaEvent(
       NetworkProfileBubble::METRIC_ACKNOWLEDGED);
 
   GetWidget()->Close();
 }
 
 void NetworkProfileBubbleView::LinkClicked(views::Link* source,
                                            int event_flags) {
   NetworkProfileBubble::RecordUmaEvent(
       NetworkProfileBubble::METRIC_LEARN_MORE_CLICKED);
   WindowOpenDisposition disposition =
       chrome::DispositionFromEventFlags(event_flags);
   content::OpenURLParams params(
       GURL("https://sites.google.com/a/chromium.org/dev/administrators/"
             "common-problems-and-solutions#network_profile"),
       content::Referrer(),
       disposition == CURRENT_TAB ? NEW_FOREGROUND_TAB : disposition,
       content::PAGE_TRANSITION_LINK, false);
   navigator_->OpenURL(params);
 
   // If the user interacted with the bubble we don't reduce the number of
   // warnings left.
   PrefService* prefs = profile_->GetPrefs();
   int left_warnings = prefs->GetInteger(prefs::kNetworkProfileWarningsLeft);
   prefs->SetInteger(prefs::kNetworkProfileWarningsLeft, ++left_warnings);
   GetWidget()->Close();
 }