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

Unified Diff: chrome/browser/ui/cocoa/content_settings/content_setting_bubble_cocoa.mm

Issue 10826164: Add infobar for PPAPI broker usage. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync Created 8 years, 3 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/content_settings/content_setting_bubble_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/content_settings/content_setting_bubble_cocoa.mm b/chrome/browser/ui/cocoa/content_settings/content_setting_bubble_cocoa.mm
index 730c2f7308ba7df6eb1f2cdc90656149ac059c47..51d7fb23b3b0227d2092f41a9c8b6bfadd94cadb 100644
--- a/chrome/browser/ui/cocoa/content_settings/content_setting_bubble_cocoa.mm
+++ b/chrome/browser/ui/cocoa/content_settings/content_setting_bubble_cocoa.mm
@@ -133,7 +133,16 @@ NSTextField* LabelWithFrame(NSString* text, const NSRect& frame) {
nibPath = @"ContentBlockedMixedScript"; break;
case CONTENT_SETTINGS_TYPE_PROTOCOL_HANDLERS:
nibPath = @"ContentProtocolHandlers"; break;
- default:
+ // These content types have no bubble:
+ case CONTENT_SETTINGS_TYPE_DEFAULT:
+ case CONTENT_SETTINGS_TYPE_NOTIFICATIONS:
+ case CONTENT_SETTINGS_TYPE_INTENTS:
+ case CONTENT_SETTINGS_TYPE_AUTO_SELECT_CERTIFICATE:
+ case CONTENT_SETTINGS_TYPE_FULLSCREEN:
+ case CONTENT_SETTINGS_TYPE_MOUSELOCK:
+ case CONTENT_SETTINGS_TYPE_MEDIASTREAM:
+ case CONTENT_SETTINGS_TYPE_PPAPI_BROKER:
+ case CONTENT_SETTINGS_NUM_TYPES:
NOTREACHED();
}
if ((self = [super initWithWindowNibPath:nibPath

Powered by Google App Engine
This is Rietveld 408576698