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..ec07fd69145418bef802c1ef9dd9902bc5690494 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
@@ -256,14 +256,8 @@ void ContentSettingImageView::OnClick() {
                     parent_->delegate()->GetContentSettingBubbleModelDelegate(),
                     web_contents, parent_->profile()),
                 web_contents, this, views::BubbleBorder::TOP_RIGHT);
-    views::Widget* bubble_widget =
-        parent_->delegate()->CreateViewsBubble(bubble_view_);
+    views::Widget* bubble_widget = bubble_view_->Show(pause_animation_);

[Evan Stade, 2016/03/07 18:10:37]

seems like there is a behavior change here. The arrow used to never show in MD,
now it depends on pause_animation_?

[varkha, 2016/03/07 18:28:51]

Yes, since there is no ink ripple active state when a label is shown during a
paused animation.

     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())
-      bubble_view_->SetArrowPaintType(views::BubbleBorder::PAINT_TRANSPARENT);
-    bubble_widget->Show();
   }
 }