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

Unified Diff: tools/tickprocessor.js

Issue 99181: Enhancing profiling data processing code with functionality needed for the Dev Tools Profiler. (Closed)
Patch Set: Created 11 years, 8 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
« tools/profile_view.js ('K') | « tools/splaytree.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/tickprocessor.js
diff --git a/tools/tickprocessor.js b/tools/tickprocessor.js
index 9d19719713fafc3acf62e999b36e61b7405a46ac..64020cabbcae83aa4af45c9d7d318b7a706695f2 100644
--- a/tools/tickprocessor.js
+++ b/tools/tickprocessor.js
@@ -273,6 +273,10 @@ TickProcessor.prototype.printStatistics = function() {
this.printCounter(this.ticks_.unaccounted, this.ticks_.total);
}
+ // Disable initialization of 'funcName', 'url', 'lineNumber' as
+ // we don't use it and it just wastes time.
+ devtools.profiler.ProfileView.Node.prototype.initFuncInfo = function() {};
+
var flatProfile = this.profile_.getFlatProfile();
var flatView = this.viewBuilder_.buildView(flatProfile);
// Sort by self time, desc, then by name, desc.
@@ -361,8 +365,7 @@ TickProcessor.prototype.processProfile = function(
profile, filterP, func) {
for (var i = 0, n = profile.length; i < n; ++i) {
var rec = profile[i];
- // An empty record corresponds to a tree root.
- if (!rec.internalFuncName || !filterP(rec.internalFuncName)) {
+ if (!filterP(rec.internalFuncName)) {
continue;
}
func(rec);
« tools/profile_view.js ('K') | « tools/splaytree.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698