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

Unified Diff: tools/telemetry/telemetry/web_perf/metrics/mainthread_jank_stats_unittest.py

Issue 1007583002: [Telemetry] Remove is_respsonsive flag. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
Index: tools/telemetry/telemetry/web_perf/metrics/mainthread_jank_stats_unittest.py
diff --git a/tools/telemetry/telemetry/web_perf/metrics/mainthread_jank_stats_unittest.py b/tools/telemetry/telemetry/web_perf/metrics/mainthread_jank_stats_unittest.py
index 6ba6176989b1139ca3fc95e273a93265efbf0f40..0c1bca372586bface331a89ca807ce67a96be35b 100644
--- a/tools/telemetry/telemetry/web_perf/metrics/mainthread_jank_stats_unittest.py
+++ b/tools/telemetry/telemetry/web_perf/metrics/mainthread_jank_stats_unittest.py
@@ -15,7 +15,7 @@ class MainthreadJankTests(unittest.TestCase):
def CreateTestRecord(self, name, start, end, thread_start, thread_end,
parent_thread):
s = async_slice.AsyncSlice(
- 'cat', 'Interaction.%s/is_responsive' % name,
+ 'cat', 'Interaction.%s' % name,
sullivan 2015/03/16 14:58:30 Do these names get propogated to the dashboard at
eakuefner 2015/03/16 15:20:28 What Ned said is correct -- we actually strip mark
nednguyen 2015/03/16 16:12:17 I don't think we will get rid of all flags. At lea
timestamp=start, duration=end - start, start_thread=parent_thread,
end_thread=parent_thread, thread_start=thread_start,
thread_duration=thread_end - thread_start)

Powered by Google App Engine
This is Rietveld 408576698