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

Unified Diff: chrome/browser/ui/cocoa/location_bar/content_setting_decoration.mm

Issue 2882533003: [Mac] Hover/Active Omnibox Icon States for Secondary UI MD (Closed)
Patch Set: Fixes for tapted 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/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..3d424a4048f5b9a36f67a98cbd9ef231cab86230 100644
--- a/chrome/browser/ui/cocoa/location_bar/content_setting_decoration.mm
+++ b/chrome/browser/ui/cocoa/location_bar/content_setting_decoration.mm
@@ -306,8 +306,8 @@ bool ContentSettingDecoration::OnMousePressed(NSRect frame, NSPoint location) {
if (ui::MaterialDesignController::IsSecondaryUiMaterial()) {
gfx::Point origin = gfx::ScreenPointFromNSPoint(anchor);
chrome::ContentSettingBubbleViewsBridge::Show(
- [web_contents->GetTopLevelNativeWindow() contentView],
- model, web_contents, origin);
+ [web_contents->GetTopLevelNativeWindow() contentView], model,
+ web_contents, origin, this);
} else {
// If the bubble is already opened, close it. Otherwise, open a new bubble.
if (bubbleWindow_ && [bubbleWindow_ isVisible]) {

Powered by Google App Engine
This is Rietveld 408576698