Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(228)

Side by Side Diff: components/metrics/metrics_service_unittest.cc

Issue 1859213002: Move the thread hop for UMA user actions from content:: to base::. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Initializing in content. Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/metrics/metrics_service.h" 5 #include "components/metrics/metrics_service.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <string> 9 #include <string>
10 10
11 #include "base/bind.h" 11 #include "base/bind.h"
12 #include "base/macros.h" 12 #include "base/macros.h"
13 #include "base/memory/scoped_ptr.h" 13 #include "base/memory/scoped_ptr.h"
14 #include "base/message_loop/message_loop.h" 14 #include "base/message_loop/message_loop.h"
15 #include "base/metrics/metrics_hashes.h" 15 #include "base/metrics/metrics_hashes.h"
16 #include "base/metrics/statistics_recorder.h" 16 #include "base/metrics/statistics_recorder.h"
17 #include "base/metrics/user_metrics.h"
17 #include "base/threading/platform_thread.h" 18 #include "base/threading/platform_thread.h"
18 #include "components/metrics/client_info.h" 19 #include "components/metrics/client_info.h"
19 #include "components/metrics/metrics_log.h" 20 #include "components/metrics/metrics_log.h"
20 #include "components/metrics/metrics_pref_names.h" 21 #include "components/metrics/metrics_pref_names.h"
21 #include "components/metrics/metrics_state_manager.h" 22 #include "components/metrics/metrics_state_manager.h"
22 #include "components/metrics/test_metrics_provider.h" 23 #include "components/metrics/test_metrics_provider.h"
23 #include "components/metrics/test_metrics_service_client.h" 24 #include "components/metrics/test_metrics_service_client.h"
24 #include "components/prefs/testing_pref_service.h" 25 #include "components/prefs/testing_pref_service.h"
25 #include "components/variations/metrics_util.h" 26 #include "components/variations/metrics_util.h"
26 #include "testing/gtest/include/gtest/gtest.h" 27 #include "testing/gtest/include/gtest/gtest.h"
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
65 66
66 ~TestMetricsLog() override {} 67 ~TestMetricsLog() override {}
67 68
68 private: 69 private:
69 DISALLOW_COPY_AND_ASSIGN(TestMetricsLog); 70 DISALLOW_COPY_AND_ASSIGN(TestMetricsLog);
70 }; 71 };
71 72
72 class MetricsServiceTest : public testing::Test { 73 class MetricsServiceTest : public testing::Test {
73 public: 74 public:
74 MetricsServiceTest() : is_metrics_reporting_enabled_(false) { 75 MetricsServiceTest() : is_metrics_reporting_enabled_(false) {
76 base::SetRecordActionTaskRunner(message_loop.task_runner());
75 MetricsService::RegisterPrefs(testing_local_state_.registry()); 77 MetricsService::RegisterPrefs(testing_local_state_.registry());
76 metrics_state_manager_ = MetricsStateManager::Create( 78 metrics_state_manager_ = MetricsStateManager::Create(
77 GetLocalState(), 79 GetLocalState(),
78 base::Bind(&MetricsServiceTest::is_metrics_reporting_enabled, 80 base::Bind(&MetricsServiceTest::is_metrics_reporting_enabled,
79 base::Unretained(this)), 81 base::Unretained(this)),
80 base::Bind(&StoreNoClientInfoBackup), 82 base::Bind(&StoreNoClientInfoBackup),
81 base::Bind(&ReturnNoBackup)); 83 base::Bind(&ReturnNoBackup));
82 } 84 }
83 85
84 ~MetricsServiceTest() override { 86 ~MetricsServiceTest() override {
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
151 153
152 private: 154 private:
153 bool is_metrics_reporting_enabled() const { 155 bool is_metrics_reporting_enabled() const {
154 return is_metrics_reporting_enabled_; 156 return is_metrics_reporting_enabled_;
155 } 157 }
156 158
157 bool is_metrics_reporting_enabled_; 159 bool is_metrics_reporting_enabled_;
158 TestingPrefServiceSimple testing_local_state_; 160 TestingPrefServiceSimple testing_local_state_;
159 scoped_ptr<MetricsStateManager> metrics_state_manager_; 161 scoped_ptr<MetricsStateManager> metrics_state_manager_;
160 base::MessageLoop message_loop; 162 base::MessageLoop message_loop;
163 scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
161 164
162 DISALLOW_COPY_AND_ASSIGN(MetricsServiceTest); 165 DISALLOW_COPY_AND_ASSIGN(MetricsServiceTest);
163 }; 166 };
164 167
165 } // namespace 168 } // namespace
166 169
167 TEST_F(MetricsServiceTest, InitialStabilityLogAfterCleanShutDown) { 170 TEST_F(MetricsServiceTest, InitialStabilityLogAfterCleanShutDown) {
168 EnableMetricsReporting(); 171 EnableMetricsReporting();
169 GetLocalState()->SetBoolean(prefs::kStabilityExitedCleanly, true); 172 GetLocalState()->SetBoolean(prefs::kStabilityExitedCleanly, true);
170 173
(...skipping 233 matching lines...) Expand 10 before | Expand all | Expand 10 after
404 407
405 TestMetricsProvider* test_provider = new TestMetricsProvider(); 408 TestMetricsProvider* test_provider = new TestMetricsProvider();
406 service.RegisterMetricsProvider(scoped_ptr<MetricsProvider>(test_provider)); 409 service.RegisterMetricsProvider(scoped_ptr<MetricsProvider>(test_provider));
407 410
408 service.InitializeMetricsRecordingState(); 411 service.InitializeMetricsRecordingState();
409 412
410 EXPECT_TRUE(test_provider->init_called()); 413 EXPECT_TRUE(test_provider->init_called());
411 } 414 }
412 415
413 } // namespace metrics 416 } // namespace metrics
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698