Removed Layout() override on HatsBubbleView in favor of setting the margins in OnBoundsChanged().

Bug: 1005568
Change-Id: If12927896ad1daa745517c03a6951ff65b3de2f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1823257
Auto-Submit: Allen Bauer <kylixrd@chromium.org>
Commit-Queue: Wei Li <weili@chromium.org>
Reviewed-by: Wei Li <weili@chromium.org>
Cr-Commit-Position: refs/heads/master@{#699891}

chrome/browser/ui/views/hats/hats_bubble_view.cc

@@ -172,15 +172,17 @@ void HatsBubbleView::OnWidgetDestroying(views::Widget* widget) {
   instance_ = nullptr;
 }
 
-void HatsBubbleView::Layout() {
+void HatsBubbleView::OnBoundsChanged(const gfx::Rect& previous_bounds) {
   auto* frame_view = GetBubbleFrameView();
   if (frame_view && frame_view->title()) {
     // Align bubble content to the beginning of the title text.
     gfx::Point point(frame_view->title()->x(), 0);
     views::View::ConvertPointToTarget(frame_view, GetWidget()->client_view(),
                                       &point);
-    SetX(point.x());
+    auto dialog_margins = margins();
+    dialog_margins.set_left(point.x());
+    set_margins(dialog_margins);
   }
 
-  views::BubbleDialogDelegateView::Layout();
+  views::BubbleDialogDelegateView::OnBoundsChanged(previous_bounds);
 }

chrome/browser/ui/views/hats/hats_bubble_view.h

@@ -62,7 +62,7 @@ class HatsBubbleView : public views::BubbleDialogDelegateView {
   void OnWidgetDestroying(views::Widget* widget) override;
 
   // views::View:
-  void Layout() override;
+  void OnBoundsChanged(const gfx::Rect& previous_bounds) override;
 
  private:
   HatsBubbleView(Browser* browser,