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

Unified Diff: components/startup_metric_utils/browser/startup_metric_utils.cc

Issue 2359503002: Let TRACE_EVENT_XXX_WITH_TIMESTAMP accept TimeTicks instead of int64_t (Closed)
Patch Set: Rebase 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/trace_event/trace_event_unittest.cc ('k') | content/browser/devtools/devtools_frame_trace_recorder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/startup_metric_utils/browser/startup_metric_utils.cc
diff --git a/components/startup_metric_utils/browser/startup_metric_utils.cc b/components/startup_metric_utils/browser/startup_metric_utils.cc
index cb3b24a35f5875aa353155d7ddc4c5c45f1046f1..6d9ffcc2da6bda3e1333fba7211fcbb624c6c3ce 100644
--- a/components/startup_metric_utils/browser/startup_metric_utils.cc
+++ b/components/startup_metric_utils/browser/startup_metric_utils.cc
@@ -258,10 +258,10 @@ bool GetHardFaultCountForCurrentProcess(uint32_t* hard_fault_count) {
do { \
UMA_HISTOGRAM_WITH_TEMPERATURE(type, basename, end_ticks - begin_ticks); \
TRACE_EVENT_ASYNC_BEGIN_WITH_TIMESTAMP1( \
- "startup", basename, 0, begin_ticks.ToInternalValue(), "Temperature", \
+ "startup", basename, 0, begin_ticks, "Temperature", \
g_startup_temperature); \
TRACE_EVENT_ASYNC_END_WITH_TIMESTAMP1( \
- "startup", basename, 0, end_ticks.ToInternalValue(), "Temperature", \
+ "startup", basename, 0, end_ticks, "Temperature", \
g_startup_temperature); \
} while (0)
@@ -271,11 +271,11 @@ bool GetHardFaultCountForCurrentProcess(uint32_t* hard_fault_count) {
UMA_HISTOGRAM_WITH_TEMPERATURE_AND_SAME_VERSION_COUNT( \
type, basename, end_ticks - begin_ticks); \
TRACE_EVENT_ASYNC_BEGIN_WITH_TIMESTAMP2( \
- "startup", basename, 0, begin_ticks.ToInternalValue(), "Temperature", \
+ "startup", basename, 0, begin_ticks, "Temperature", \
g_startup_temperature, "Startups with current version", \
g_startups_with_current_version); \
TRACE_EVENT_ASYNC_END_WITH_TIMESTAMP2( \
- "startup", basename, 0, end_ticks.ToInternalValue(), "Temperature", \
+ "startup", basename, 0, end_ticks, "Temperature", \
g_startup_temperature, "Startups with current version", \
g_startups_with_current_version); \
} while (0)
@@ -472,13 +472,13 @@ void AddStartupEventsForTelemetry()
TRACE_EVENT_INSTANT_WITH_TIMESTAMP0(
"startup", "Startup.BrowserMainEntryPoint", 0,
- g_browser_main_entry_point_ticks.Get().ToInternalValue());
+ g_browser_main_entry_point_ticks.Get());
if (!g_process_creation_ticks.Get().is_null())
{
TRACE_EVENT_INSTANT_WITH_TIMESTAMP0(
"startup", "Startup.BrowserProcessCreation", 0,
- g_process_creation_ticks.Get().ToInternalValue());
+ g_process_creation_ticks.Get());
}
}
« no previous file with comments | « base/trace_event/trace_event_unittest.cc ('k') | content/browser/devtools/devtools_frame_trace_recorder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698