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

Side by Side Diff: chrome/browser/metrics/chrome_browser_main_extra_parts_metrics_mac.mm

Issue 2299993004: Migrate chrome/browser files to histogram_macros.h includes. (Closed)
Patch Set: Created 4 years, 3 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
« no previous file with comments | « chrome/browser/memory_details.cc ('k') | chrome/browser/mod_pagespeed/mod_pagespeed_metrics.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "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 "base/mac/sdk_forward_declarations.h" 7 #include "base/mac/sdk_forward_declarations.h"
8 #include "base/metrics/histogram.h" 8 #include "base/metrics/histogram_macros.h"
9 9
10 namespace { 10 namespace {
11 11
12 // The possible values of the setting "Screens Have Separate Spaces". 12 // The possible values of the setting "Screens Have Separate Spaces".
13 enum ScreenSpacesConfiguration { 13 enum ScreenSpacesConfiguration {
14 SCREENS_CANNOT_HAVE_SEPARATE_SPACES = 0, 14 SCREENS_CANNOT_HAVE_SEPARATE_SPACES = 0,
15 SCREENS_HAVE_SEPARATE_SPACES = 1, 15 SCREENS_HAVE_SEPARATE_SPACES = 1,
16 SCREENS_HAVE_SHARED_SPACES = 2, 16 SCREENS_HAVE_SHARED_SPACES = 2,
17 SCREEN_SPACES_CONFIGURATION_COUNT = 3 17 SCREEN_SPACES_CONFIGURATION_COUNT = 3
18 }; 18 };
19 19
20 } // namespace 20 } // namespace
21 21
22 void ChromeBrowserMainExtraPartsMetrics::RecordMacMetrics() { 22 void ChromeBrowserMainExtraPartsMetrics::RecordMacMetrics() {
23 ScreenSpacesConfiguration separate_spaces = 23 ScreenSpacesConfiguration separate_spaces =
24 SCREENS_CANNOT_HAVE_SEPARATE_SPACES; 24 SCREENS_CANNOT_HAVE_SEPARATE_SPACES;
25 25
26 if ([NSScreen respondsToSelector:@selector(screensHaveSeparateSpaces)]) { 26 if ([NSScreen respondsToSelector:@selector(screensHaveSeparateSpaces)]) {
27 BOOL screens_have_separate_spaces = [NSScreen screensHaveSeparateSpaces]; 27 BOOL screens_have_separate_spaces = [NSScreen screensHaveSeparateSpaces];
28 separate_spaces = screens_have_separate_spaces 28 separate_spaces = screens_have_separate_spaces
29 ? SCREENS_HAVE_SEPARATE_SPACES 29 ? SCREENS_HAVE_SEPARATE_SPACES
30 : SCREENS_HAVE_SHARED_SPACES; 30 : SCREENS_HAVE_SHARED_SPACES;
31 } 31 }
32 UMA_HISTOGRAM_ENUMERATION("OSX.Settings.ScreensHaveSeparateSpaces", 32 UMA_HISTOGRAM_ENUMERATION("OSX.Settings.ScreensHaveSeparateSpaces",
33 separate_spaces, 33 separate_spaces,
34 SCREEN_SPACES_CONFIGURATION_COUNT); 34 SCREEN_SPACES_CONFIGURATION_COUNT);
35 } 35 }
OLDNEW
« no previous file with comments | « chrome/browser/memory_details.cc ('k') | chrome/browser/mod_pagespeed/mod_pagespeed_metrics.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698