Corrects ContentSettingBubbleContents to show bubble with arrow when animation is paused

chrome/browser/ui/views/location_bar/content_setting_image_view.cc

Index: chrome/browser/ui/views/location_bar/content_setting_image_view.cc
diff --git a/chrome/browser/ui/views/location_bar/content_setting_image_view.cc b/chrome/browser/ui/views/location_bar/content_setting_image_view.cc
index e88ab5c3f5717dfaad2fb524f3ac242e1e3be424..82af69084d35c63014dd350faa45139a2934dc37 100644
--- a/chrome/browser/ui/views/location_bar/content_setting_image_view.cc
+++ b/chrome/browser/ui/views/location_bar/content_setting_image_view.cc
@@ -260,8 +260,9 @@ void ContentSettingImageView::OnClick() {
         parent_->delegate()->CreateViewsBubble(bubble_view_);
     bubble_widget->AddObserver(this);
     // This is triggered by an input event, the icon will be in an active state
-    // so the bubble doesn't need an arrow.
-    if (ui::MaterialDesignController::IsModeMaterial())
+    // so the bubble doesn't need an arrow. When animation is paused the anchor
+    // view doesn't show ink drop so the arrow is still shown.

[Peter Kasting, 2016/03/08 03:16:28]

Nit: Grammar.  How about:

This is triggered by an input event.  If the user clicks the icon while it's not
animating, the icon will be placed in an active state, so the bubble doesn't
need an arrow.  If the user clicks during an animation, the animation simply
pauses and no other visible state change occurs, so show the arrow in this case.

(Though, frankly, I'd almost rather just always show the arrows than have this
kind of complicated logic, but whatever.)

+    if (ui::MaterialDesignController::IsModeMaterial() && !pause_animation_)
       bubble_view_->SetArrowPaintType(views::BubbleBorder::PAINT_TRANSPARENT);
     bubble_widget->Show();
   }