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

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

Issue 10825485: Record device scale factor in the metrics log (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: mpearson Created 8 years, 4 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 // This file defines a set of user experience metrics data recorded by 5 // This file defines a set of user experience metrics data recorded by
6 // the MetricsService. This is the unit of data that is sent to the server. 6 // the MetricsService. This is the unit of data that is sent to the server.
7 7
8 #ifndef CHROME_BROWSER_METRICS_METRICS_LOG_H_ 8 #ifndef CHROME_BROWSER_METRICS_METRICS_LOG_H_
9 #define CHROME_BROWSER_METRICS_METRICS_LOG_H_ 9 #define CHROME_BROWSER_METRICS_METRICS_LOG_H_
10 10
(...skipping 108 matching lines...) Expand 10 before | Expand all | Expand 10 after
119 119
120 protected: 120 protected:
121 // Exposed for the sake of mocking in test code. 121 // Exposed for the sake of mocking in test code.
122 122
123 // Returns the PrefService from which to log metrics data. 123 // Returns the PrefService from which to log metrics data.
124 virtual PrefService* GetPrefService(); 124 virtual PrefService* GetPrefService();
125 125
126 // Returns the screen size for the primary monitor. 126 // Returns the screen size for the primary monitor.
127 virtual gfx::Size GetScreenSize() const; 127 virtual gfx::Size GetScreenSize() const;
128 128
129 // Returns the device scale factor for the primary monitor.
130 virtual float GetScreenDeviceScaleFactor() const;
131
129 // Returns the number of monitors the user is using. 132 // Returns the number of monitors the user is using.
130 virtual int GetScreenCount() const; 133 virtual int GetScreenCount() const;
131 134
132 // Fills |field_trial_ids| with the list of initialized field trials name and 135 // Fills |field_trial_ids| with the list of initialized field trials name and
133 // group ids. 136 // group ids.
134 virtual void GetFieldTrialIds( 137 virtual void GetFieldTrialIds(
135 std::vector<chrome_variations::SelectedGroupId>* field_trial_ids) const; 138 std::vector<chrome_variations::SelectedGroupId>* field_trial_ids) const;
136 139
137 private: 140 private:
138 FRIEND_TEST_ALL_PREFIXES(MetricsLogTest, ChromeOSStabilityData); 141 FRIEND_TEST_ALL_PREFIXES(MetricsLogTest, ChromeOSStabilityData);
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
177 const base::DictionaryValue& profile_metrics); 180 const base::DictionaryValue& profile_metrics);
178 181
179 // Writes info about the Google Update install that is managing this client. 182 // Writes info about the Google Update install that is managing this client.
180 // This is a no-op if called on a non-Windows platform. 183 // This is a no-op if called on a non-Windows platform.
181 void WriteGoogleUpdateProto(const GoogleUpdateMetrics& google_update_metrics); 184 void WriteGoogleUpdateProto(const GoogleUpdateMetrics& google_update_metrics);
182 185
183 DISALLOW_COPY_AND_ASSIGN(MetricsLog); 186 DISALLOW_COPY_AND_ASSIGN(MetricsLog);
184 }; 187 };
185 188
186 #endif // CHROME_BROWSER_METRICS_METRICS_LOG_H_ 189 #endif // CHROME_BROWSER_METRICS_METRICS_LOG_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/metrics/metrics_log.cc » ('j') | chrome/browser/metrics/metrics_log_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698