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

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

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.h"
 
 #import <AppKit/AppKit.h>
 
 #include <cmath>
 
@@ skipping 1255 matching lines @@
     PageInfoBubbleController* controller) {
   bubble_controller_ = controller;
 }
 
 void PageInfoUIBridge::Show(gfx::NativeWindow parent,
                             Profile* profile,
                             content::WebContents* web_contents,
                             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;
   }
 
   // Don't show the bubble if it's already being shown. Since this method is
   // called each time the location icon is clicked, each click toggles the
   // bubble in and out.
   if (g_is_bubble_showing)
     return;
 
   // Create the bridge. This will be owned by the bubble controller.
@@ skipping 35 matching lines @@
 void PageInfoUIBridge::SetCookieInfo(const CookieInfoList& cookie_info_list) {
   [bubble_controller_ setCookieInfo:cookie_info_list];
 }
 
 void PageInfoUIBridge::SetPermissionInfo(
     const PermissionInfoList& permission_info_list,
     ChosenObjectInfoList chosen_object_info_list) {
   [bubble_controller_ setPermissionInfo:permission_info_list
                        andChosenObjects:std::move(chosen_object_info_list)];
 }