Use AddedToWidget() instead of ViewHierarchyChanged()

Bug: 245409
Change-Id: I6a96fcf3ab975146631137322f338d0841da3ee3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1993678
Commit-Queue: Allen Bauer <kylixrd@chromium.org>
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#729964}

chrome/browser/ui/views/confirm_bubble_views.cc

@@ -119,13 +119,10 @@ void ConfirmBubbleViews::ButtonPressed(views::Button* sender,
   }
 }
 
-void ConfirmBubbleViews::ViewHierarchyChanged(
-    const views::ViewHierarchyChangedDetails& details) {
-  if (details.is_add && details.child == this && GetWidget()) {
+void ConfirmBubbleViews::AddedToWidget() {
   GetWidget()->GetRootView()->GetViewAccessibility().OverrideDescribedBy(
       label_);
-  }
-  DialogDelegateView::ViewHierarchyChanged(details);
+  DialogDelegateView::AddedToWidget();
 }
 
 namespace chrome {

chrome/browser/ui/views/confirm_bubble_views.h

@@ -49,8 +49,7 @@ class ConfirmBubbleViews : public views::DialogDelegateView,
   void ButtonPressed(views::Button* sender, const ui::Event& event) override;
 
   // views::View implementation.
-  void ViewHierarchyChanged(
-      const views::ViewHierarchyChangedDetails& details) override;
+  void AddedToWidget() override;
 
  private:
   // The model to customize this bubble view.