speculative fix for status-bubble crash

Crash data seems to indicate after the call to SetBubbleWidth(). AFAICT
the functions that are called after SetBubbleWidth() can be moved *before*
the call to SetBubbleWidth(), so moving them in hopes of fixing the crash.

BUG=943564
TEST=none

Change-Id: I271fdc0e90a89f8d3ce4f45a848e4dcbeee0c1dd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1540308Reviewed-by: Evan Stade <estade@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#644903}

chrome/browser/ui/views/status_bubble_views.cc

@@ -588,6 +588,8 @@ class StatusBubbleViews::StatusViewExpander : public gfx::LinearAnimation,
   // Widths at expansion start and end.
   int expansion_start_ = 0;
   int expansion_end_ = 0;
+
+  DISALLOW_COPY_AND_ASSIGN(StatusViewExpander);
 };
 
 void StatusBubbleViews::StatusViewExpander::AnimateToState(double state) {
@@ -596,8 +598,10 @@ void StatusBubbleViews::StatusViewExpander::AnimateToState(double state) {
 
 void StatusBubbleViews::StatusViewExpander::AnimationEnded(
     const gfx::Animation* animation) {
-  SetBubbleWidth(expansion_end_);
   status_view_->SetText(expanded_text_, false);
+  SetBubbleWidth(expansion_end_);
+  // WARNING: crash data seems to indicate |this| may be deleted by the time
+  // SetBubbleWidth() returns.
 }
 
 void StatusBubbleViews::StatusViewExpander::StartExpansion(
@@ -621,8 +625,10 @@ int StatusBubbleViews::StatusViewExpander::GetCurrentBubbleWidth() {
 }
 
 void StatusBubbleViews::StatusViewExpander::SetBubbleWidth(int width) {
-  status_bubble_->SetBubbleWidth(width);
   status_view_->SchedulePaint();
+  status_bubble_->SetBubbleWidth(width);
+  // WARNING: crash data seems to indicate |this| may be deleted by the time
+  // SetBubbleWidth() returns.
 }