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

Unified Diff: chrome/browser/mac/mac_startup_profiler.cc

Issue 1425263003: Use TimeTicks as much as possible in startup_metric_utils. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: ifdef thread priority dance out on Mac per lack of support in base Created 5 years, 1 month 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
Index: chrome/browser/mac/mac_startup_profiler.cc
diff --git a/chrome/browser/mac/mac_startup_profiler.cc b/chrome/browser/mac/mac_startup_profiler.cc
index 95aa7d8da005bf710baa1282b5c995a3bc0d6ba7..91a8719f67560f4e13a95aecbab014ae2b6bfc07 100644
--- a/chrome/browser/mac/mac_startup_profiler.cc
+++ b/chrome/browser/mac/mac_startup_profiler.cc
@@ -20,24 +20,19 @@ MacStartupProfiler::~MacStartupProfiler() {
}
void MacStartupProfiler::Profile(Location location) {
- profiled_times_[location] = base::Time::Now();
+ profiled_ticks_[location] = base::TimeTicks::Now();
}
void MacStartupProfiler::RecordMetrics() {
- const base::Time main_entry_time = startup_metric_utils::MainEntryPointTime();
- DCHECK(!main_entry_time.is_null());
+ const base::TimeTicks main_entry_ticks =
+ startup_metric_utils::MainEntryPointTicks();
+ DCHECK(!main_entry_ticks.is_null());
DCHECK(!recorded_metrics_);
recorded_metrics_ = true;
- for (std::map<Location, base::Time>::const_iterator it =
- profiled_times_.begin();
- it != profiled_times_.end();
- ++it) {
- const base::Time& location_time = it->second;
- base::TimeDelta delta = location_time - main_entry_time;
- RecordHistogram(it->first, delta);
- }
+ for (const std::pair<Location, base::TimeTicks>& entry : profiled_ticks_)
+ RecordHistogram(entry.first, entry.second - main_entry_ticks);
}
const std::string MacStartupProfiler::HistogramName(Location location) {

Powered by Google App Engine
This is Rietveld 408576698