[Mac] Hover/Active Omnibox Icon States for Secondary UI MD

chrome/browser/ui/cocoa/location_bar/content_setting_decoration.mm

Index: chrome/browser/ui/cocoa/location_bar/content_setting_decoration.mm
diff --git a/chrome/browser/ui/cocoa/location_bar/content_setting_decoration.mm b/chrome/browser/ui/cocoa/location_bar/content_setting_decoration.mm
index 3fb4f0293b77fa6c57d824f38fd72502dfad0e10..9b6aacdddd2607177bf7cd864e327b712a6521ff 100644
--- a/chrome/browser/ui/cocoa/location_bar/content_setting_decoration.mm
+++ b/chrome/browser/ui/cocoa/location_bar/content_setting_decoration.mm
@@ -302,6 +302,7 @@ bool ContentSettingDecoration::OnMousePressed(NSRect frame, NSPoint location) {
           browser->content_setting_bubble_model_delegate(),
           web_contents,
           profile_);
+  model->set_decoration(this);

[tapted, 2017/05/16 07:39:37]

this doesn't look right - can we just pass |this| through to
ContentSettingBubbleViewsBridge::Show(..) below, and avoid changing the model?

[spqchan, 2017/05/17 01:39:40]

Good point. I implemented a method in LocationBarViewMac that can return the
content setting decoration based on the anchor point.

 
   if (ui::MaterialDesignController::IsSecondaryUiMaterial()) {
     gfx::Point origin = gfx::ScreenPointFromNSPoint(anchor);