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

Unified Diff: trace_processor/experimental/mappers/slice_cost.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
Index: trace_processor/experimental/mappers/slice_cost.html
diff --git a/perf_insights/perf_insights/mappers/slice_cost.html b/trace_processor/experimental/mappers/slice_cost.html
similarity index 98%
rename from perf_insights/perf_insights/mappers/slice_cost.html
rename to trace_processor/experimental/mappers/slice_cost.html
index 6a143492416ce32a189fd2d4457e1ededaa78245..8860f87c7001aa16cc396ab1a75d72045a79d3a1 100644
--- a/perf_insights/perf_insights/mappers/slice_cost.html
+++ b/trace_processor/experimental/mappers/slice_cost.html
@@ -137,16 +137,16 @@ tr.exportTo('pi.m', function() {
});
}
- model.iterateAllEvents(function(event) {
+ for (var event of model.descendentEvents()) {
if (!(event instanceof tr.model.ThreadSlice))
- return;
+ continue;
if (filterFunction && !filterFunction(event))
- return;
+ continue;
var threadSlice = event;
if (threadSlice.title === 'V8.Execute') {
generateDomainCosts(threadSlice);
- return;
+ continue;
}
var sliceCostInfo = new SliceCostInfo();
@@ -166,7 +166,7 @@ tr.exportTo('pi.m', function() {
sliceCostInfo.railTypeName + '/' +
sliceCostInfo.title;
reduce.push(key, sliceCostInfo);
- });
+ }
var sliceCostInfos = [];
reduce.finalizeAndIterResults(function(key, sliceCostInfo) {

Powered by Google App Engine
This is Rietveld 408576698