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

Unified Diff: build/android/pylib/json_perf_parser.py

Issue 11090082: Move PerfTest TraceEvent to async (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix build break Created 8 years, 2 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 | content/common/android/trace_event_binding.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/pylib/json_perf_parser.py
diff --git a/build/android/pylib/json_perf_parser.py b/build/android/pylib/json_perf_parser.py
index 88ca3e0d80cdc6701ed6a02a7091c6db60bb6a0d..1a8e617539c2d9c5332a218e78d25134b77fa458 100644
--- a/build/android/pylib/json_perf_parser.py
+++ b/build/android/pylib/json_perf_parser.py
@@ -16,12 +16,12 @@ def GetAverageRunInfo(json_data, name):
[
{ "cat": "Java",
"ts": 10000000000,
- "ph": "B",
+ "ph": "S",
"name": "TestTrace"
},
{ "cat": "Java",
"ts": 10000004000,
- "ph": "E",
+ "ph": "F",
"name": "TestTrace"
},
...
@@ -106,11 +106,11 @@ def GetAverageRunInfo(json_data, name):
raise Exception('Entry did not contain expected value type "%s" '
'information: %s' % (val_type, entry))
val = GetVal(entry)
- if (entry['ph'] == 'B' and
+ if (entry['ph'] == 'S' and
(result['type'] == 'Unknown' or result['type'] == 'Span')):
result['type'] = 'Span'
last_val = val
- elif ((entry['ph'] == 'E' and result['type'] == 'Span') or
+ elif ((entry['ph'] == 'F' and result['type'] == 'Span') or
(entry['ph'] == 'I' and (result['type'] == 'Unknown' or
result['type'] == 'Instant'))):
if last_val > 0:
« no previous file with comments | « no previous file | content/common/android/trace_event_binding.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698