Allow SAML logins to use the webcam

chrome/browser/chromeos/policy/device_policy_decoder_chromeos.cc

Index: chrome/browser/chromeos/policy/device_policy_decoder_chromeos.cc
diff --git a/chrome/browser/chromeos/policy/device_policy_decoder_chromeos.cc b/chrome/browser/chromeos/policy/device_policy_decoder_chromeos.cc
index c5d155f9198a8d26c4390583561f8d13f15e085b..cfe7c3362df7fdd16cf20f6ab475f86085d0f407 100644
--- a/chrome/browser/chromeos/policy/device_policy_decoder_chromeos.cc
+++ b/chrome/browser/chromeos/policy/device_policy_decoder_chromeos.cc
@@ -325,6 +325,19 @@ void DecodeLoginPolicies(const em::ChromeDeviceSettingsProto& policy,
                     nullptr);
     }
   }
+
+  if (policy.has_login_video_capture_allowed_urls()) {
+    const em::LoginVideoCaptureAllowedUrlsProto& container(
+        policy.login_video_capture_allowed_urls());
+    base::ListValue* urls = new base::ListValue();

[bartfab (slow), 2016/05/02 10:30:28]

Nit 1: const pointer.
Nit 2: A unique_ptr would be semantically cleaner.

[Kevin Cernekee, 2016/05/02 20:02:29]

1) Done
2) Don't think that will work here, because PolicyMap::Set() takes ownership of
the |value| argument.  (It crashed when I tried it.)

[bartfab (slow), 2016/05/03 16:14:15]

Sure, you need to relase() when hading over ownership to PolicyMap::Set().

[Kevin Cernekee, 2016/05/03 17:54:54]

Done.

+    RepeatedPtrField<std::string>::const_iterator entry;
+    for (entry = container.urls().begin(); entry != container.urls().end();
+         ++entry) {
+      urls->Append(new base::StringValue(*entry));
+    }
+    policies->Set(key::kLoginVideoCaptureAllowedUrls, POLICY_LEVEL_MANDATORY,
+                  POLICY_SCOPE_MACHINE, POLICY_SOURCE_CLOUD, urls, nullptr);
+  }
 }
 
 void DecodeNetworkPolicies(const em::ChromeDeviceSettingsProto& policy,