Prevent hover cards from showing while any tabstrip bounds animation is running.

Previously if a mid-tabstrip tab in an uncrowded tabstrip was closed the tabs to the left could show a hover card since their bounds were not animating while the tabs to the right couldn't since they were moving. This change provides more consistent behavior across the tab strip.

This change also fixes a bug where a hover card flashed on the first tab after closing a tab due to focus changing during the tab close.

Bug: 965731
Change-Id: Id7161ec54b5f7bfc564df8f8b819fc828e561663
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2374287Reviewed-by: Dana Fried <dfried@chromium.org>
Commit-Queue: Caroline Rising <corising@chromium.org>
Cr-Commit-Position: refs/heads/master@{#802238}

chrome/browser/ui/views/tabs/tab_strip.cc

@@ -1948,8 +1948,8 @@ void TabStrip::OnMouseEventInTab(views::View* source,
 void TabStrip::UpdateHoverCard(Tab* tab) {
   if (!base::FeatureList::IsEnabled(features::kTabHoverCards))
     return;
-  // We don't want to show a hover card for a tab while it is animating.
-  if (tab && bounds_animator_.IsAnimating(tab)) {
+  // We don't want to show a hover card while the tabstrip is animating.
+  if (bounds_animator_.IsAnimating()) {
     return;
   }