Chrome OS device policy for Variations Restrict Parameter.

chrome/browser/chromeos/settings/device_settings_provider.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/chromeos/settings/device_settings_provider.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
 #include "base/command_line.h"
@@ skipping 49 matching lines @@
   kReportDeviceLocation,
   kReportDeviceVersionInfo,
   kScreenSaverExtensionId,
   kScreenSaverTimeout,
   kSettingProxyEverywhere,
   kSignedDataRoamingEnabled,
   kStartUpUrls,
   kStatsReportingPref,
   kSystemTimezonePolicy,
   kStartUpFlags,
+  kVariationsRestrictParameter,
 };
 
 // Legacy policy file location. Used to detect migration from pre v12 ChromeOS.
 const char kLegacyPolicyFile[] = "/var/lib/whitelist/preferences";
 
 bool HasOldMetricsFile() {
   // TODO(pastarmovj): Remove this once migration is not needed anymore.
   // If the value is not set we should try to migrate legacy consent file.
   // Loading consent file state causes us to do blocking IO on UI thread.
   // Temporarily allow it until we fix http://crbug.com/62626
@@ skipping 265 matching lines @@
     //   kIdleLogoutWarningDuration
     //   kReleaseChannelDelegated
     //   kReportDeviceVersionInfo
     //   kReportDeviceActivityTimes
     //   kReportDeviceBootMode
     //   kReportDeviceLocation
     //   kScreenSaverExtensionId
     //   kScreenSaverTimeout
     //   kStartUpUrls
     //   kSystemTimezonePolicy
+    //   kVariationsRestrictParameter
 
     LOG(FATAL) << "Device setting " << prop << " is read-only.";
   }
 
   em::PolicyData data;
   data.set_username(device_settings_service_->GetUsername());
   CHECK(device_settings_.SerializeToString(data.mutable_policy_value()));
 
   // Set the cache to the updated value.
   UpdateValuesCache(data, device_settings_, trusted_status_);
@@ skipping 246 matching lines @@
 
   if (policy.has_allow_redeem_offers()) {
     new_values_cache->SetBoolean(
         kAllowRedeemChromeOsRegistrationOffers,
         policy.allow_redeem_offers().allow_redeem_offers());
   } else {
     new_values_cache->SetBoolean(
         kAllowRedeemChromeOsRegistrationOffers,
         !g_browser_process->browser_policy_connector()->IsEnterpriseManaged());
   }
+
+  if (policy.has_variations_restrict_parameter() &&
+      policy.variations_restrict_parameter().has_parameter()) {
+    new_values_cache->SetString(
+        kVariationsRestrictParameter,
+        policy.variations_restrict_parameter().parameter());
+  } else {
+    new_values_cache->SetString(kVariationsRestrictParameter, "");

[Mattias Nissler (ping if slow), 2013/04/17 17:36:50]

The default value is an empty string, so you can simplify this entire block to:

if (policy.has_variations_restrict_parameter()) {
  new_values_cache->SetString(
      kVariationsRestrictParameter,
      policy.variations_restrict_parameter().parameter());
}

[Mathieu, 2013/04/17 20:04:58]

Done.

+  }
 }
 
 void DeviceSettingsProvider::UpdateValuesCache(
     const em::PolicyData& policy_data,
     const em::ChromeDeviceSettingsProto& settings,
     TrustedStatus trusted_status) {
   PrefValueMap new_values_cache;
 
   if (policy_data.has_username() && !policy_data.has_request_token())
     new_values_cache.SetString(kDeviceOwner, policy_data.username());
@@ skipping 225 matching lines @@
 void DeviceSettingsProvider::AttemptMigration() {
   if (device_settings_service_->HasPrivateOwnerKey()) {
     PrefValueMap::const_iterator i;
     for (i = migration_values_.begin(); i != migration_values_.end(); ++i)
       DoSet(i->first, *i->second);
     migration_values_.Clear();
   }
 }
 
 }  // namespace chromeos