IPH: do not override GetBubbleBounds

Remove |FeaturePromoBubbleView::GetBubbleBounds| override as it's not
needed anymore.

Change-Id: I0c8ebd04982889842494c7a85b68f902a0832b33
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2622733
Commit-Queue: Phillis Tang <phillis@chromium.org>
Commit-Queue: Collin Baker <collinbaker@chromium.org>
Reviewed-by: Collin Baker <collinbaker@chromium.org>
Cr-Commit-Position: refs/heads/master@{#842269}

chrome/browser/ui/views/user_education/feature_promo_bubble_view.cc

@@ -316,17 +316,6 @@ void FeaturePromoBubbleView::OnMouseExited(const ui::MouseEvent& event) {
     feature_promo_bubble_timeout_->OnMouseExited();
 }
 
-gfx::Rect FeaturePromoBubbleView::GetBubbleBounds() {
-  gfx::Rect bounds = BubbleDialogDelegateView::GetBubbleBounds();
-  if (!focusable_) {
-    if (base::i18n::IsRTL())
-      bounds.Offset(5, 0);
-    else
-      bounds.Offset(-5, 0);
-  }
-  return bounds;
-}
-
 ax::mojom::Role FeaturePromoBubbleView::GetAccessibleWindowRole() {
   // Since we don't have any controls for the user to interact with (we're just
   // an information bubble), override our role to kAlert.

chrome/browser/ui/views/user_education/feature_promo_bubble_view.h

@@ -15,9 +15,6 @@
 #include "ui/gfx/geometry/rect.h"
 #include "ui/views/bubble/bubble_dialog_delegate_view.h"
 
-namespace gfx {
-class Rect;
-}
 
 namespace ui {
 class MouseEvent;
@@ -89,7 +86,6 @@ class FeaturePromoBubbleView : public views::BubbleDialogDelegateView {
   bool OnMousePressed(const ui::MouseEvent& event) override;
   void OnMouseEntered(const ui::MouseEvent& event) override;
   void OnMouseExited(const ui::MouseEvent& event) override;
-  gfx::Rect GetBubbleBounds() override;
   ax::mojom::Role GetAccessibleWindowRole() override;
   base::string16 GetAccessibleWindowTitle() const override;
   void UpdateHighlightedButton(bool highlighted) override {