Implement fullscreen info bubble on Win and Mac

chrome/browser/ui/webui/options/content_settings_handler.cc

Index: chrome/browser/ui/webui/options/content_settings_handler.cc
diff --git a/chrome/browser/ui/webui/options/content_settings_handler.cc b/chrome/browser/ui/webui/options/content_settings_handler.cc
index 5b158c70647a5af6161a6b8248b52184927a42d3..00cc28a83b2d75d54ce3d6f2658529fed801ff32 100644
--- a/chrome/browser/ui/webui/options/content_settings_handler.cc
+++ b/chrome/browser/ui/webui/options/content_settings_handler.cc
@@ -63,6 +63,7 @@ const ContentSettingsTypeNameEntry kContentSettingsTypeGroupNames[] = {
   {CONTENT_SETTINGS_TYPE_NOTIFICATIONS, "notifications"},
   {CONTENT_SETTINGS_TYPE_INTENTS, "intents"},
   {CONTENT_SETTINGS_TYPE_AUTO_SELECT_CERTIFICATE, "auto-select-certificate"},
+  {CONTENT_SETTINGS_TYPE_FULLSCREEN, "fullscreen"},
 };
 COMPILE_ASSERT(arraysize(kContentSettingsTypeGroupNames) ==
                    CONTENT_SETTINGS_NUM_TYPES,
@@ -416,6 +417,9 @@ void ContentSettingsHandler::UpdateAllExceptionsViewsFromModel() {
     // for this content type and we skip it here.
     if (type == CONTENT_SETTINGS_TYPE_AUTO_SELECT_CERTIFICATE)
       continue;
+    // TODO(koz): Implement fullscreen content settings UI.

[Peter Kasting, 2011/10/13 23:48:06]

Nit: Is there a bug for this?  Seems like this should be covered by a bug rather
than a TODO...

[koz (OOO until 15th September), 2011/10/14 01:06:13]

Actually, I don't think there are any plans to do this. This was speculative
comment. Removed.

[Peter Kasting, 2011/10/14 01:23:19]

Um, I don't think that's OK.  This needs content settings UI.

[jeremya, 2011/10/14 02:01:49]

http://crbug.com/100292

I don't think it's urgent to have that UI for M16, so that bug is marked
Mstone-17.

+    if (type == CONTENT_SETTINGS_TYPE_FULLSCREEN)
+      continue;
     UpdateExceptionsViewFromModel(static_cast<ContentSettingsType>(type));
   }
 }