[MD] Remove arrow from location bar bubbles when icon is already showing

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

Index: chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.cc
diff --git a/chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.cc b/chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.cc
index 38783b088c9ba3740ee5a237316a48abeb60833b..3580171aba28ff74158ef8e75ea2cf6b57e2df11 100644
--- a/chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.cc
+++ b/chrome/browser/ui/views/location_bar/location_bar_bubble_delegate_view.cc
@@ -9,6 +9,7 @@
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/exclusive_access/fullscreen_controller.h"
 #include "content/public/browser/notification_source.h"
+#include "ui/base/material_design/material_design_controller.h"
 #include "ui/gfx/geometry/rect.h"
 
 LocationBarBubbleDelegateView::LocationBarBubbleDelegateView(
@@ -31,8 +32,8 @@ LocationBarBubbleDelegateView::~LocationBarBubbleDelegateView() {}
 
 void LocationBarBubbleDelegateView::ShowForReason(DisplayReason reason) {
   if (reason == USER_GESTURE) {
-    // TODO(estade): re-enable this when crbug.com/518941 is fixed.
-    // SetArrowPaintType(views::BubbleBorder::PAINT_TRANSPARENT);
+    if (ui::MaterialDesignController::IsModeMaterial())

[Peter Kasting, 2016/03/03 20:43:28]

Nit: I'd add a comment here, e.g. "Gesture-triggered bubbles will have a hover
state visible on the target icon, so we don't need (or want) to show the arrow."

[Evan Stade, 2016/03/03 21:07:59]

Done.

+      SetArrowPaintType(views::BubbleBorder::PAINT_TRANSPARENT);
     GetWidget()->Show();
   } else {
     GetWidget()->ShowInactive();