Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6453)

Unified Diff: chrome/browser/ui/cocoa/page_info/page_info_bubble_controller.mm

Issue 2882533003: [Mac] Hover/Active Omnibox Icon States for Secondary UI MD (Closed)
Patch Set: Addressed tapted's comments Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..c9ec731ca6ff6c8abc2787367cdfe42fb1aac06b 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* location_bar = [controller locationBarBridge];
+ LocationBarDecoration* decoration =
+ location_bar ? location_bar->GetPageInfoDecoration() : nullptr;
chrome::ShowPageInfoBubbleViewsAtPoint(
gfx::ScreenPointFromNSPoint(AnchorPointForWindow(parent)), profile,
- web_contents, virtual_url, security_info);
+ web_contents, virtual_url, security_info, decoration);
return;
}

Powered by Google App Engine
This is Rietveld 408576698