Rename MetricsServiceHelper to ChromeMetricsServiceAccessor.

chrome/browser/metrics/chrome_metrics_service_accessor_unittest.cc

Index: chrome/browser/metrics/chrome_metrics_service_accessor_unittest.cc
diff --git a/chrome/browser/metrics/chrome_metrics_service_accessor_unittest.cc b/chrome/browser/metrics/chrome_metrics_service_accessor_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..938dd8da11cef7dfca350e0bce0b90930905220c
--- /dev/null
+++ b/chrome/browser/metrics/chrome_metrics_service_accessor_unittest.cc
@@ -0,0 +1,68 @@
+// Copyright (c) 2014 The Chromium Authors. All rights reserved.

[Ilya Sherman, 2014/05/20 15:18:11]

Ultra nit: Please omit the "(c)" (in all the new files).

[motek., 2014/05/20 15:54:39]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/metrics/chrome_metrics_service_accessor.h"
+
+#include "chrome/common/pref_names.h"
+#include "chrome/test/base/scoped_testing_local_state.h"
+#include "chrome/test/base/testing_browser_process.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace {
+
+class ChromeMetricsServiceAccessorTest : public testing::Test {
+ public:
+  ChromeMetricsServiceAccessorTest()
+      : testing_local_state_(TestingBrowserProcess::GetGlobal()) {
+  }
+
+  PrefService* GetLocalState() {
+    return testing_local_state_.Get();
+  }
+
+ private:
+  ScopedTestingLocalState testing_local_state_;
+
+  DISALLOW_COPY_AND_ASSIGN(ChromeMetricsServiceAccessorTest);
+};
+
+}  // namespace

[Ilya Sherman, 2014/05/20 15:18:11]

nit: The actual test harness classes shouldn't be tucked into the anonymous
namespace.  These class names should be unique across the entire project. 
Otherwise, you can get confusing test output results, especially thanks to C++
name mangling.

[motek., 2014/05/20 15:54:39]

Done.

+
+
+TEST_F(ChromeMetricsServiceAccessorTest, MetricsReportingEnabled) {
+#if !defined(OS_CHROMEOS)
+  GetLocalState()->SetBoolean(prefs::kMetricsReportingEnabled, false);
+  EXPECT_FALSE(ChromeMetricsServiceAccessor::IsMetricsReportingEnabled());
+  GetLocalState()->SetBoolean(prefs::kMetricsReportingEnabled, true);
+  EXPECT_TRUE(ChromeMetricsServiceAccessor::IsMetricsReportingEnabled());
+  GetLocalState()->ClearPref(prefs::kMetricsReportingEnabled);
+  EXPECT_FALSE(ChromeMetricsServiceAccessor::IsMetricsReportingEnabled());
+#else
+  // ChromeOS does not register prefs::kMetricsReportingEnabled and uses
+  // device settings for metrics reporting.
+  EXPECT_FALSE(ChromeMetricsServiceAccessor::IsMetricsReportingEnabled());
+#endif
+}
+
+TEST_F(ChromeMetricsServiceAccessorTest, CrashReportingEnabled) {
+#if defined(GOOGLE_CHROME_BUILD)
+// ChromeOS has different device settings for crash reporting.
+#if !defined(OS_CHROMEOS)
+#if defined(OS_ANDROID)
+  const char* crash_pref = prefs::kCrashReportingEnabled;
+#else
+  const char* crash_pref = prefs::kMetricsReportingEnabled;
+#endif
+  GetLocalState()->SetBoolean(crash_pref, false);
+  EXPECT_FALSE(ChromeMetricsServiceAccessor::IsCrashReportingEnabled());
+  GetLocalState()->SetBoolean(crash_pref, true);
+  EXPECT_TRUE(ChromeMetricsServiceAccessor::IsCrashReportingEnabled());
+  GetLocalState()->ClearPref(crash_pref);
+  EXPECT_FALSE(ChromeMetricsServiceAccessor::IsCrashReportingEnabled());
+#endif  // !defined(OS_CHROMEOS)
+#else  // defined(GOOGLE_CHROME_BUILD)
+  // Chromium branded browsers never have crash reporting enabled.
+  EXPECT_FALSE(ChromeMetricsServiceAccessor::IsCrashReportingEnabled());
+#endif  // defined(GOOGLE_CHROME_BUILD)
+}