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

Unified Diff: tools/tickprocessor.js

Issue 7247018: Remove obsolete aggregating and non-working producers heap profilers. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 9 years, 6 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
« src/api.cc ('K') | « test/cctest/test-log.cc ('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 9d6bfb60ea9f5ee83917f1c9263bfe2a6f666e1e..5f57835524465066f9ba2e8d431485674e8a4020 100644
--- a/tools/tickprocessor.js
+++ b/tools/tickprocessor.js
@@ -169,17 +169,12 @@ function TickProcessor(
processor: this.processHeapSampleBegin },
'heap-sample-end': { parsers: [null, null],
processor: this.processHeapSampleEnd },
- 'heap-js-prod-item': { parsers: [null, 'var-args'],
- processor: this.processJSProducer },
// Ignored events.
'profiler': null,
'function-creation': null,
'function-move': null,
'function-delete': null,
- 'heap-sample-stats': null,
'heap-sample-item': null,
- 'heap-js-cons-item': null,
- 'heap-js-ret-item': null,
// Obsolete row types.
'code-allocate': null,
'begin-code-region': null,
@@ -401,17 +396,6 @@ TickProcessor.prototype.processHeapSampleEnd = function(space, state) {
};
-TickProcessor.prototype.processJSProducer = function(constructor, stack) {
- if (!this.currentProducerProfile_) return;
- if (stack.length == 0) return;
- var first = stack.shift();
- var processedStack =
- this.profile_.resolveAndFilterFuncs_(this.processStack(first, 0, stack));
- processedStack.unshift(constructor);
- this.currentProducerProfile_.addPath(processedStack);
-};
-
-
TickProcessor.prototype.printStatistics = function() {
print('Statistical profiling result from ' + this.lastLogFileName_ +
', (' + this.ticks_.total +
« src/api.cc ('K') | « test/cctest/test-log.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698