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

Unified Diff: trace_processor/experimental/mappers/v8_map_function.html

Issue 2162963002: [polymer] Merge of master into polymer10-migration (Closed) Base URL: git@github.com:catapult-project/catapult.git@polymer10-migration
Patch Set: Merge polymer10-migration int polymer10-merge Created 4 years, 5 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 | « trace_processor/experimental/mappers/trace_stats_test.html ('k') | tracing/app.yaml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trace_processor/experimental/mappers/v8_map_function.html
diff --git a/perf_insights/perf_insights/mappers/v8_map_function.html b/trace_processor/experimental/mappers/v8_map_function.html
similarity index 98%
rename from perf_insights/perf_insights/mappers/v8_map_function.html
rename to trace_processor/experimental/mappers/v8_map_function.html
index a8dc9f60422273d9f4ac4cd1cb1dc3cb916dadb0..2942e2faec2d25a1fae45662d896ce3b110b29a2 100644
--- a/perf_insights/perf_insights/mappers/v8_map_function.html
+++ b/trace_processor/experimental/mappers/v8_map_function.html
@@ -34,12 +34,12 @@ tr.exportTo('pi.m', function() {
var sliceCosts = [];
- model.iterateAllEvents(function(event) {
+ for (var event of model.getDescendantEvents()) {
if (!(event instanceof tr.model.ThreadSlice))
- return;
+ continue;
if (!event.title.startsWith('V8.') && !event.title.startsWith('V8Test.'))
- return;
+ continue;
function _get_parent_data(event) {
var curSlice = event;
@@ -221,7 +221,7 @@ tr.exportTo('pi.m', function() {
value: sliceData
};
sliceCosts.push(newElement);
- });
+ }
result.addPair('wr', sliceCosts);
}
@@ -237,11 +237,11 @@ tr.exportTo('pi.m', function() {
}, this);
}, this);
- model.iterateAllEvents(function storeEventToUnassociatedSet(event) {
+ for (var event of model.getDescendantEvents()) {
if (stageTitleByGUID[event.guid] !== undefined)
return;
stageTitleByGUID[event.guid] = 'Unknown';
- });
+ }
return stageTitleByGUID;
}
« no previous file with comments | « trace_processor/experimental/mappers/trace_stats_test.html ('k') | tracing/app.yaml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698