When the browser actions container's animation ended, we should set...

When the browser actions container's animation ended, we should set 'suppress_chevron_' to false then re-layout it.

If we keep 'suppress_chevron_' being true and notify to re-layout the toolbar, then finish the animation, because there is no chance to layout toolbar again, some browser action views would not be showed, neither chevron. That makes people puzzled.

TBR=sky
R=finnur@chromium.org
BUG=328307

Review URL: https://codereview.chromium.org/188353002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@255371 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/ui/views/toolbar/browser_actions_container.cc

@@ -450,8 +450,8 @@ void BrowserActionsContainer::AnimationEnded(const gfx::Animation* animation) {
   container_width_ = animation_target_size_;
   animation_target_size_ = 0;
   resize_amount_ = 0;
-  OnBrowserActionVisibilityChanged();
   suppress_chevron_ = false;
+  OnBrowserActionVisibilityChanged();
 }
 
 void BrowserActionsContainer::NotifyMenuDeleted(