Changing string id for CONTENT_SETTINGS_TYPE_COOKIES for block and allow case.

chrome/browser/ui/content_settings/content_setting_bubble_model.cc

Index: chrome/browser/ui/content_settings/content_setting_bubble_model.cc
diff --git a/chrome/browser/ui/content_settings/content_setting_bubble_model.cc b/chrome/browser/ui/content_settings/content_setting_bubble_model.cc
index bdc7df799563be57ae400c71306f41395bd38395..38c1f6b7c50b38602b4828bc404dab56a63667b1 100644
--- a/chrome/browser/ui/content_settings/content_setting_bubble_model.cc
+++ b/chrome/browser/ui/content_settings/content_setting_bubble_model.cc
@@ -304,8 +304,7 @@ void ContentSettingSingleRadioGroup::SetRadioGroup() {
   };
   // Fields as for kBlockedAllowIDs, above.
   static const ContentSettingsTypeIdEntry kAllowedAllowIDs[] = {
-    // TODO(bauerb): The string shouldn't be "unblock" (they weren't blocked).
-    {CONTENT_SETTINGS_TYPE_COOKIES, IDS_BLOCKED_COOKIES_UNBLOCK},
+    {CONTENT_SETTINGS_TYPE_COOKIES, IDS_BLOCKED_COOKIES_ALLOW},

[Bernhard Bauer, 2015/06/04 08:08:02]

This doesn't really change anything if the string value stays the same. If you
look at the code that uses this, this is the string for the case where cookies
are already allowed, and should stay allowed. This means that the right string
value would be "Continue allowing cookies" (cf. "Continue allowing unsandboxed
plugins" for PPAPI_BROKER below), and the string ID
IDS_ALLOWED_COOKIES_NO_ACTION.

[Deepak, 2015/06/04 10:09:10]

Done.

     {CONTENT_SETTINGS_TYPE_PPAPI_BROKER, IDS_ALLOWED_PPAPI_BROKER_NO_ACTION},
     {CONTENT_SETTINGS_TYPE_AUTOMATIC_DOWNLOADS, IDS_ALLOWED_DOWNLOAD_NO_ACTION},
   };