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: chrome/browser/metrics/chrome_browser_main_extra_parts_metrics_unittest.cc

Issue 2902853002: Add UMA metrics for the linux distro. (Closed)
Patch Set: Fix unit tests. Created 3 years, 6 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.h" 5 #include "chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/macros.h" 9 #include "base/macros.h"
10 #include "base/message_loop/message_loop.h" 10 #include "base/message_loop/message_loop.h"
11 #include "base/test/histogram_tester.h" 11 #include "base/test/histogram_tester.h"
12 #include "base/test/scoped_task_environment.h"
12 #include "testing/gtest/include/gtest/gtest.h" 13 #include "testing/gtest/include/gtest/gtest.h"
13 #include "ui/display/screen.h" 14 #include "ui/display/screen.h"
14 #include "ui/display/test/test_screen.h" 15 #include "ui/display/test/test_screen.h"
15 #include "ui/events/test/device_data_manager_test_api.h" 16 #include "ui/events/test/device_data_manager_test_api.h"
16 #include "ui/gfx/geometry/size.h" 17 #include "ui/gfx/geometry/size.h"
17 18
18 namespace { 19 namespace {
19 20
20 const char kTouchEventFeatureDetectionEnabledHistogramName[] = 21 const char kTouchEventFeatureDetectionEnabledHistogramName[] =
21 "Touchscreen.TouchEventsEnabled"; 22 "Touchscreen.TouchEventsEnabled";
22 23
23 } // namespace 24 } // namespace
24 25
25 class ChromeBrowserMainExtraPartsMetricsTest : public testing::Test { 26 class ChromeBrowserMainExtraPartsMetricsTest : public testing::Test {
26 public: 27 public:
27 ChromeBrowserMainExtraPartsMetricsTest(); 28 ChromeBrowserMainExtraPartsMetricsTest();
28 ~ChromeBrowserMainExtraPartsMetricsTest() override; 29 ~ChromeBrowserMainExtraPartsMetricsTest() override;
29 30
30 protected: 31 protected:
31 // Test API wrapping |device_data_manager_|. 32 // Test API wrapping |device_data_manager_|.
32 ui::test::DeviceDataManagerTestAPI device_data_manager_test_api_; 33 ui::test::DeviceDataManagerTestAPI device_data_manager_test_api_;
33 34
34 private: 35 private:
35 // Required by a ChromeBrowserMainExtraPartsMetrics test target. 36 // Provides a message loop and allows the use of the task scheduler
36 base::MessageLoop message_loop_; 37 base::test::ScopedTaskEnvironment scoped_task_environment_;
37 38
38 // Dummy screen required by a ChromeBrowserMainExtraPartsMetrics test target. 39 // Dummy screen required by a ChromeBrowserMainExtraPartsMetrics test target.
39 display::test::TestScreen test_screen_; 40 display::test::TestScreen test_screen_;
40 41
41 DISALLOW_COPY_AND_ASSIGN(ChromeBrowserMainExtraPartsMetricsTest); 42 DISALLOW_COPY_AND_ASSIGN(ChromeBrowserMainExtraPartsMetricsTest);
42 }; 43 };
43 44
44 ChromeBrowserMainExtraPartsMetricsTest::ChromeBrowserMainExtraPartsMetricsTest() 45 ChromeBrowserMainExtraPartsMetricsTest::ChromeBrowserMainExtraPartsMetricsTest()
45 : device_data_manager_test_api_() { 46 : device_data_manager_test_api_() {
46 display::Screen::SetScreenInstance(&test_screen_); 47 display::Screen::SetScreenInstance(&test_screen_);
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 VerifyTouchEventsEnabledIsRecordedAfterPostBrowserStart) { 125 VerifyTouchEventsEnabledIsRecordedAfterPostBrowserStart) {
125 base::HistogramTester histogram_tester; 126 base::HistogramTester histogram_tester;
126 ChromeBrowserMainExtraPartsMetrics test_target; 127 ChromeBrowserMainExtraPartsMetrics test_target;
127 128
128 test_target.PostBrowserStart(); 129 test_target.PostBrowserStart();
129 histogram_tester.ExpectTotalCount( 130 histogram_tester.ExpectTotalCount(
130 kTouchEventFeatureDetectionEnabledHistogramName, 1); 131 kTouchEventFeatureDetectionEnabledHistogramName, 1);
131 } 132 }
132 133
133 #endif // defined(USE_OZONE) || defined(USE_X11) 134 #endif // defined(USE_OZONE) || defined(USE_X11)
OLDNEW
« no previous file with comments | « chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.cc ('k') | tools/metrics/histograms/enums.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698