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

Unified Diff: chrome/browser/performance_monitor/constants.h

Issue 10907121: Add guards to metric values; erase bad events/metrics from db (Closed) Base URL: http://git.chromium.org/chromium/src.git@dc_use_units
Patch Set: Estade's requests Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/performance_monitor/constants.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/performance_monitor/constants.h
diff --git a/chrome/browser/performance_monitor/constants.h b/chrome/browser/performance_monitor/constants.h
index 84681f9cc1315886a6ce0ecade79fbf05d3c5f26..24009da3772de73f988f664aa16d5ed3a0998655 100644
--- a/chrome/browser/performance_monitor/constants.h
+++ b/chrome/browser/performance_monitor/constants.h
@@ -5,6 +5,8 @@
#ifndef CHROME_BROWSER_PERFORMANCE_MONITOR_CONSTANTS_H_
#define CHROME_BROWSER_PERFORMANCE_MONITOR_CONSTANTS_H_
+#include "base/basictypes.h"
+
namespace performance_monitor {
// Constants which are used by the PerformanceMonitor and its related classes.
@@ -19,6 +21,21 @@ extern const int kDefaultGatherIntervalInSeconds;
extern const char kStateChromeVersion[];
extern const char kStateProfilePrefix[];
+// Unit values (for use in metric, and on the UI side).
+
+// Memory measurements
+extern const int64 kBytesPerKilobyte;
+extern const int64 kBytesPerMegabyte;
+extern const int64 kBytesPerGigabyte;
+extern const int64 kBytesPerTerabyte;
+
+// Time measurements - Most of these are imported from base/time.h
+// These units are used for display (and it's related calculations), not for
+// any mathematical analysis. Thus we can estimate for values without an exact
+// conversion.
+extern const int64 kMicrosecondsPerMonth;
+extern const int64 kMicrosecondsPerYear;
+
} // namespace performance_monitor
#endif // CHROME_BROWSER_PERFORMANCE_MONITOR_CONSTANTS_H_
« no previous file with comments | « no previous file | chrome/browser/performance_monitor/constants.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698