Add a Rappor option in the settings page.

chrome/browser/policy/configuration_policy_handler_list_factory.cc

 // Copyright 2013 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/policy/configuration_policy_handler_list_factory.h"
 
 #include "base/basictypes.h"
 #include "base/bind.h"
 #include "base/memory/scoped_vector.h"
 #include "base/values.h"
 #include "chrome/browser/net/proxy_policy_handler.h"
 #include "chrome/browser/policy/managed_bookmarks_policy_handler.h"
 #include "chrome/browser/profiles/incognito_mode_policy_handler.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "components/password_manager/core/common/password_manager_pref_names.h"
 #include "components/policy/core/browser/autofill_policy_handler.h"
 #include "components/policy/core/browser/configuration_policy_handler.h"
 #include "components/policy/core/browser/configuration_policy_handler_list.h"
 #include "components/policy/core/browser/configuration_policy_handler_parameters.h"
 #include "components/policy/core/browser/url_blacklist_policy_handler.h"
 #include "components/policy/core/common/policy_details.h"
 #include "components/policy/core/common/policy_map.h"
 #include "components/policy/core/common/policy_pref_names.h"
 #include "components/policy/core/common/schema.h"
+#include "components/rappor/rappor_pref_names.h"
 #include "components/search_engines/default_search_policy_handler.h"
 #include "components/translate/core/common/translate_pref_names.h"
 #include "policy/policy_constants.h"
 
 #if defined(OS_ANDROID)
 #include "chrome/browser/search/contextual_search_policy_handler_android.h"
 #endif
 
 #if !defined(OS_IOS)
 #include "chrome/browser/net/disk_cache_dir_policy_handler.h"
@@ skipping 76 matching lines @@
     base::Value::TYPE_BOOLEAN },
   { key::kPrintingEnabled,
     prefs::kPrintingEnabled,
     base::Value::TYPE_BOOLEAN },
   { key::kDisablePrintPreview,
     prefs::kPrintPreviewDisabled,
     base::Value::TYPE_BOOLEAN },
   { key::kMetricsReportingEnabled,
     prefs::kMetricsReportingEnabled,
     base::Value::TYPE_BOOLEAN },
+  { key::kMetricsReportingEnabled,
+    rappor::prefs::kRapporEnabled,
+    base::Value::TYPE_BOOLEAN },

[Mattias Nissler (ping if slow), 2015/02/10 08:38:15]

We need to be careful here w.r.t. installations that already have
key::kMetricsReportingEnabled. Given that we have two separate options in
chrome://settings, does it make sense to subsume them both under the same policy
knob?

I assume rappor::prefs::kRapporEnabled toggles RAPPOR entirely? Can you provide
some background on what led to the decision to have to separate pref settings
for metrics reporting and RAPPOR for the consumer experience to help decide and
justify what the enterprise policy behavior should be?

Side note: If we do decide to subsume RAPPOR in kMetricsReportingEnabled, then
you should also update the policy description in
components/policy/resources/policy_templates.json to mention RAPPOR.

   { key::kApplicationLocaleValue,
     prefs::kApplicationLocale,
     base::Value::TYPE_STRING },
   { key::kDisabledPlugins,
     prefs::kPluginsDisabledPlugins,
     base::Value::TYPE_LIST },
   { key::kDisabledPluginsExceptions,
     prefs::kPluginsDisabledPluginsExceptions,
     base::Value::TYPE_LIST },
   { key::kEnabledPlugins,
@@ skipping 660 matching lines @@
           chrome_schema,
           SCHEMA_STRICT,
           SimpleSchemaValidatingPolicyHandler::RECOMMENDED_ALLOWED,
           SimpleSchemaValidatingPolicyHandler::MANDATORY_PROHIBITED)));
 #endif  // defined(OS_CHROMEOS)
 
   return handlers.Pass();
 }
 
 }  // namespace policy