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

Unified Diff: tracing/tracing/model/clock_sync_manager.html

Issue 2712163002: [Systrace] Fix systrace clock syncing issue with BattOr. (Closed)
Patch Set: rebase and change todo Created 3 years, 9 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 | « systrace/systrace/tracing_controller.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/model/clock_sync_manager.html
diff --git a/tracing/tracing/model/clock_sync_manager.html b/tracing/tracing/model/clock_sync_manager.html
index 68e0a63e753a096cb221805a3ba123b065419bbf..03d8e07cc1adee5b5c9c650a9d884fa5e072f8d1 100644
--- a/tracing/tracing/model/clock_sync_manager.html
+++ b/tracing/tracing/model/clock_sync_manager.html
@@ -25,15 +25,16 @@ tr.exportTo('tr.model', function() {
'WIN_ROLLOVER_PROTECTED_TIME_GET_TIME',
WIN_QPC: 'WIN_QPC',
- // "Telemetry" isn't really a clock domain because Telemetry actually
- // can use one of several clock domains, just like Chrome. However,
- // because there's a chance that Telemetry is running off of the same
- // clock as Chrome (e.g. LINUX_CLOCK_MONOTONIC) but on a separate device
- // (i.e. on a host computer with Chrome running on an attached phone),
- // there's a chance that Chrome and Telemetry will erroneously get put into
+ // 'TELEMETRY' and 'SYSTRACE' aren't really clock domains because they
+ // actually can use one of several clock domains, just like Chrome. However,
+ // because there's a chance that they are running off of the same clock as
+ // Chrome (e.g. LINUX_CLOCK_MONOTONIC) but on a separate device (i.e. on a
+ // a host computer with Chrome running on an attached phone), there's a
+ // chance that Chrome and the controller will erroneously get put into
// the same clock domain. The solution for this is that clock domains should
// actually be some (unique_device_id, clock_id) tuple. For now, though,
// we'll hack around this by putting Telemetry into its own clock domain.
+ SYSTRACE: 'SYSTRACE',
TELEMETRY: 'TELEMETRY'
};
« no previous file with comments | « systrace/systrace/tracing_controller.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698