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

chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.mm

Index: chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.mm
diff --git a/chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.mm b/chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.mm
index 59cf17271d727ca9236b9adaf72f1696109ce0b1..0a732e579dea347a7bc54f3310792564d019dba6 100644
--- a/chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.mm
+++ b/chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.mm
@@ -1273,9 +1273,14 @@ void PageInfoUIBridge::Show(gfx::NativeWindow parent,
                             const GURL& virtual_url,
                             const security_state::SecurityInfo& security_info) {
   if (ui::MaterialDesignController::IsSecondaryUiMaterial()) {
+    BrowserWindowController* controller =
+        [BrowserWindowController browserWindowControllerForWindow:parent];
+    LocationBarViewMac* locationBar = [controller locationBarBridge];

[tapted, 2017/05/18 03:40:39]

nit: locationBar -> location_bar.

and-hm - this duplicates some logic in AnchorPointForWindow, but there are other
call sites. You could pass in a LocationBarDecoration** to AnchorPointForWindow,
but that might not look better. Up to you. This is fine.

[spqchan, 2017/05/19 01:47:28]

Done.

+    LocationBarDecoration* decoration =
+        locationBar ? locationBar->GetPageInfoDecoration() : nullptr;
     chrome::ShowPageInfoBubbleViewsAtPoint(
         gfx::ScreenPointFromNSPoint(AnchorPointForWindow(parent)), profile,
-        web_contents, virtual_url, security_info);
+        web_contents, virtual_url, security_info, decoration);
     return;
   }