Site Settings Desktop: Support adding exceptions for incognito mode.

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 071a24e705a6fa114f303c38f559900c879306e5..503ea0ba4ffc29e7cb2276e9b0916e3e28f7dfc9 100644
--- a/chrome/browser/ui/webui/options/content_settings_handler.cc
+++ b/chrome/browser/ui/webui/options/content_settings_handler.cc
@@ -948,7 +948,7 @@ void ContentSettingsHandler::CompareMediaExceptionsWithFlash(
 
   base::ListValue exceptions;
   site_settings::GetExceptionsFromHostContentSettingsMap(settings_map, type,
-                                                         web_ui(), &exceptions);
+      web_ui(), /*incognito=*/ false, &exceptions);

[stevenjb, 2016/09/02 15:54:01]

nit: We generally do: false /* incognito */,

[dschuyler, 2016/09/02 22:49:38]

Interesting, I got a different impression. Rather than 
discuss it here, I shared a doc with my understanding 
of the usage at:

https://docs.google.com/a/chromium.org/document/d/15rL7FMf24YO7hdHDkVXve8v7pr...

[stevenjb, 2016/09/02 23:23:03]

Caveat: This is entirely a nit, for discussion prurposes only:

Based on prevalence in the existing code. ~1000 examples of "false /*", 300
examples of "*/ false", and 500 examples of "*/false" (not an exact metric by
any means, but consistent with my impression of the code). So, take your pick.

(The one example in the style guide does show /*incognito=*/ false, but it's not
a specific rule, and I generally find prevalence a better argument than
interpretation of an example).

 
   settings.exceptions.clear();
   for (base::ListValue::const_iterator entry = exceptions.begin();
@@ -1093,11 +1093,10 @@ void ContentSettingsHandler::UpdateExceptionsViewFromHostContentSettingsMap(
   HostContentSettingsMap* settings_map =
       HostContentSettingsMapFactory::GetForProfile(GetProfile());
   site_settings::GetExceptionsFromHostContentSettingsMap(settings_map, type,
-                                                         web_ui(), &exceptions);
+      web_ui(), /*incognito=*/ false, &exceptions);

[dschuyler, 2016/09/02 22:49:38]

nit: inside a parameter list (as an exception to
the normal spacing), spaces are not used in or
after the comment. Visually, it helps make the thing 
look like a single unit. This helps readability 
because parameters are set positionally. So:
 
Foo(settings_map, type, web_ui(), /*incognito=*/false, &exceptions);

is easier to see the five parameters.

E.g.

https://cs.chromium.org/search/?q=%5B%5E%5Cs%5D%3D%5C*/%5B%5E%5Cs%5D&sq=packa...

   base::StringValue type_string(
       site_settings::ContentSettingsTypeToGroupName(type));
   web_ui()->CallJavascriptFunctionUnsafe("ContentSettings.setExceptions",
-
                                          type_string, exceptions);
 
   UpdateExceptionsViewFromOTRHostContentSettingsMap(type);
@@ -1128,8 +1127,8 @@ void ContentSettingsHandler::UpdateExceptionsViewFromOTRHostContentSettingsMap(
   if (!otr_settings_map)
     return;
   base::ListValue exceptions;
-  site_settings::GetExceptionsFromHostContentSettingsMap(
-      otr_settings_map, type, web_ui(), &exceptions);
+  site_settings::GetExceptionsFromHostContentSettingsMap(otr_settings_map, type,
+      web_ui(), /*incognito=*/ true, &exceptions);
   base::StringValue type_string(
       site_settings::ContentSettingsTypeToGroupName(type));
   web_ui()->CallJavascriptFunctionUnsafe("ContentSettings.setOTRExceptions",