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

Side by Side Diff: chrome/browser/metrics/chrome_browser_main_extra_parts_metrics.h

Issue 641453003: Add metrics for the number of attached displays. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments from isherman. 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 (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 #ifndef CHROME_BROWSER_METRICS_CHROME_BROWSER_MAIN_EXTRA_PARTS_METRICS_H_ 5 #ifndef CHROME_BROWSER_METRICS_CHROME_BROWSER_MAIN_EXTRA_PARTS_METRICS_H_
6 #define CHROME_BROWSER_METRICS_CHROME_BROWSER_MAIN_EXTRA_PARTS_METRICS_H_ 6 #define CHROME_BROWSER_METRICS_CHROME_BROWSER_MAIN_EXTRA_PARTS_METRICS_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
11 #include "chrome/browser/chrome_browser_main_extra_parts.h" 11 #include "chrome/browser/chrome_browser_main_extra_parts.h"
12 #include "ui/gfx/display_observer.h"
12 13
13 class ChromeBrowserMainParts; 14 class ChromeBrowserMainParts;
14 15
15 namespace chrome { 16 namespace chrome {
16 void AddMetricsExtraParts(ChromeBrowserMainParts* main_parts); 17 void AddMetricsExtraParts(ChromeBrowserMainParts* main_parts);
17 } 18 }
18 19
19 class ChromeBrowserMainExtraPartsMetrics : public ChromeBrowserMainExtraParts { 20 class ChromeBrowserMainExtraPartsMetrics : public ChromeBrowserMainExtraParts,
21 public gfx::DisplayObserver {
20 public: 22 public:
21 ChromeBrowserMainExtraPartsMetrics(); 23 ChromeBrowserMainExtraPartsMetrics();
22 virtual ~ChromeBrowserMainExtraPartsMetrics(); 24 virtual ~ChromeBrowserMainExtraPartsMetrics();
23 25
24 // Overridden from ChromeBrowserMainExtraParts: 26 // Overridden from ChromeBrowserMainExtraParts:
25 virtual void PreProfileInit() override; 27 virtual void PreProfileInit() override;
26 virtual void PreBrowserStart() override; 28 virtual void PreBrowserStart() override;
27 virtual void PostBrowserStart() override; 29 virtual void PostBrowserStart() override;
28 30
29 private: 31 private:
32 // DisplayObserver overrides.
33 virtual void OnDisplayAdded(const gfx::Display& new_display) override;
34 virtual void OnDisplayRemoved(const gfx::Display& old_display) override;
35 virtual void OnDisplayMetricsChanged(const gfx::Display& display,
36 uint32_t changed_metrics) override;
37
38 // If the number of displays has changed, emit a UMA metric.
39 void EmitDisplaysChangedMetric();
40
41 // A cached value for the number of displays.
42 int display_count_;
43
30 DISALLOW_COPY_AND_ASSIGN(ChromeBrowserMainExtraPartsMetrics); 44 DISALLOW_COPY_AND_ASSIGN(ChromeBrowserMainExtraPartsMetrics);
31 }; 45 };
32 46
33 #endif // CHROME_BROWSER_METRICS_CHROME_BROWSER_MAIN_EXTRA_PARTS_METRICS_H_ 47 #endif // CHROME_BROWSER_METRICS_CHROME_BROWSER_MAIN_EXTRA_PARTS_METRICS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698