ChromeOS: Implement CaptivePortalAuthenticationIgnoresProxy policy.

chrome/browser/policy/configuration_policy_handler_list_factory.cc

Index: chrome/browser/policy/configuration_policy_handler_list_factory.cc
diff --git a/chrome/browser/policy/configuration_policy_handler_list_factory.cc b/chrome/browser/policy/configuration_policy_handler_list_factory.cc
index ba7269a7b3a50298dbb53174278e365f9ab80135..0007aa041986938e2220e0d8547eb2e373e37290 100644
--- a/chrome/browser/policy/configuration_policy_handler_list_factory.cc
+++ b/chrome/browser/policy/configuration_policy_handler_list_factory.cc
@@ -514,6 +514,12 @@ const PolicyToPreferenceMapEntry kSimplePolicyMap[] = {
     prefs::kBrowserAddPersonEnabled,
     base::Value::TYPE_BOOLEAN },
 #endif  // !defined(OS_CHROMEOS) && !defined(OS_ANDROID) && !defined(OS_IOS)
+
+#if defined(OS_CHROMEOS)

[bartfab (slow), 2014/12/19 12:08:54]

There is a block for Chrome OS-specific policies already that ends in line 492.
You can just add your policy in there.

[Alexander Alekseev, 2014/12/22 22:20:11]

Done.

+  { key::kCaptivePortalAuthenticationIgnoresProxy,
+    prefs::kCaptivePortalAuthenticationIgnoresProxy,
+    base::Value::TYPE_INTEGER },

[bartfab (slow), 2014/12/19 12:08:54]

Since this is actually an int-enum, you should use a ConfigurationPolicyHandler
that checks whether the value is in the right range. However, as mentioned in my
comments for policy_templates.json, I think it would be best to turn this into a
boolean policy anyway.

[Alexander Alekseev, 2014/12/22 22:20:11]

I've added IntRangePolicyHandler for the new preference.

Also, please see my comments below.

+#endif  // defined(OS_CHROMEOS)
 };
 
 #if defined(ENABLE_EXTENSIONS)