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

Side by Side Diff: chromecast/metrics/cast_metrics_service_client_unittest.cc

Issue 602393004: Chromecast: adds stability metrics provider to track crashes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: break common->metrics dependency Created 6 years, 2 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 "chromecast/metrics/cast_metrics_service_client.h" 5 #include "chromecast/metrics/cast_metrics_service_client.h"
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/prefs/testing_pref_service.h" 10 #include "base/prefs/testing_pref_service.h"
11 #include "components/metrics/metrics_service.h" 11 #include "components/metrics/metrics_service.h"
12 #include "testing/gtest/include/gtest/gtest.h" 12 #include "testing/gtest/include/gtest/gtest.h"
13 13
14 namespace chromecast { 14 namespace chromecast {
15 15
16 class CastMetricsTest : public testing::Test { 16 class CastMetricsTest : public testing::Test {
17 public: 17 public:
18 CastMetricsTest() {} 18 CastMetricsTest() {}
19 virtual ~CastMetricsTest() {} 19 virtual ~CastMetricsTest() {}
20 20
21 protected: 21 protected:
22 virtual void SetUp() OVERRIDE { 22 virtual void SetUp() OVERRIDE {
23 message_loop_.reset(new base::MessageLoop()); 23 message_loop_.reset(new base::MessageLoop());
24 prefs_.reset(new TestingPrefServiceSimple()); 24 prefs_.reset(new TestingPrefServiceSimple());
25 ::metrics::MetricsService::RegisterPrefs(prefs_->registry()); 25 ::metrics::MetricsService::RegisterPrefs(prefs_->registry());
26 } 26 }
27 27
28 base::TaskRunner* task_runner() {
29 return message_loop_->task_runner().get();
30 }
28 TestingPrefServiceSimple* prefs() { return prefs_.get(); } 31 TestingPrefServiceSimple* prefs() { return prefs_.get(); }
29 32
30 private: 33 private:
31 scoped_ptr<base::MessageLoop> message_loop_; 34 scoped_ptr<base::MessageLoop> message_loop_;
32 scoped_ptr<TestingPrefServiceSimple> prefs_; 35 scoped_ptr<TestingPrefServiceSimple> prefs_;
33 36
34 DISALLOW_COPY_AND_ASSIGN(CastMetricsTest); 37 DISALLOW_COPY_AND_ASSIGN(CastMetricsTest);
35 }; 38 };
36 39
37 TEST_F(CastMetricsTest, CreateMetricsServiceClient) { 40 TEST_F(CastMetricsTest, CreateMetricsServiceClient) {
38 // Create and expect this to not crash. 41 // Create and expect this to not crash.
39 metrics::CastMetricsServiceClient::Create(prefs(), NULL); 42 metrics::CastMetricsServiceClient::Create(task_runner(), prefs(), NULL);
40 } 43 }
41 44
42 } // namespace chromecast 45 } // namespace chromecast
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698