Commit 89874793 authored by Roger McFarlane's avatar Roger McFarlane Committed by Commit Bot

Change DemographicMetricsReporting to FEATURE_ENABLED_BY_DEFAULT for desktop.

Bug: 978849
Change-Id: Ide816851b13c00fb5068174f86d2d21a674a6494
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2007971Reviewed-by: default avatarAlexei Svitkine <asvitkine@chromium.org>
Commit-Queue: Roger McFarlane <rogerm@chromium.org>
Cr-Commit-Position: refs/heads/master@{#733017}
parent 353199ab
......@@ -7,15 +7,25 @@
#include "base/feature_list.h"
#include "base/metrics/histogram_functions.h"
#include "base/optional.h"
#include "build/build_config.h"
#include "components/sync/base/sync_prefs.h"
#include "components/sync/driver/sync_service.h"
#include "third_party/metrics_proto/ukm/report.pb.h"
namespace metrics {
// Enable UMA Demogrpahics by default for desktop platforms but NOT mobile.
namespace {
#if defined(OS_IOS) || defined(OS_ANDROID)
constexpr auto default_feature_state = base::FEATURE_DISABLED_BY_DEFAULT;
#else
constexpr auto default_feature_state = base::FEATURE_ENABLED_BY_DEFAULT;
#endif
} // namespace
// static
const base::Feature DemographicMetricsProvider::kDemographicMetricsReporting = {
"DemographicMetricsReporting", base::FEATURE_DISABLED_BY_DEFAULT};
"DemographicMetricsReporting", default_feature_state};
DemographicMetricsProvider::DemographicMetricsProvider(
std::unique_ptr<ProfileClient> profile_client,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment