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

Unified Diff: tools/tickprocessor.js

Issue 2753543006: [profiler] Web UI: add summary of opts/deopts. (Closed)
Patch Set: Address reviewer comments Created 3 years, 9 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 | « tools/profview/profview.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 43d620c25bc463ff0ac403d5feeae9a25ddef632..788b9c9731f1cffaa77b776b0180a0615fbf3672 100644
--- a/tools/tickprocessor.js
+++ b/tools/tickprocessor.js
@@ -88,9 +88,14 @@ function TickProcessor(
'shared-library': { parsers: [null, parseInt, parseInt, parseInt],
processor: this.processSharedLibrary },
'code-creation': {
- parsers: [null, parseInt, parseInt, parseInt, null, 'var-args'],
+ parsers: [null, parseInt, parseInt, parseInt, parseInt,
+ null, 'var-args'],
processor: this.processCodeCreation },
- 'code-move': { parsers: [parseInt, parseInt],
+ 'code-deopt': {
+ parsers: [parseInt, parseInt, parseInt, parseInt, parseInt,
+ null, null, null],
+ processor: this.processCodeDeopt },
+ 'code-move': { parsers: [parseInt, parseInt, ],
processor: this.processCodeMove },
'code-delete': { parsers: [parseInt],
processor: this.processCodeDelete },
@@ -266,18 +271,26 @@ TickProcessor.prototype.processSharedLibrary = function(
TickProcessor.prototype.processCodeCreation = function(
- type, kind, start, size, name, maybe_func) {
+ type, kind, timestamp, start, size, name, maybe_func) {
name = this.deserializedEntriesNames_[start] || name;
if (maybe_func.length) {
var funcAddr = parseInt(maybe_func[0]);
var state = parseState(maybe_func[1]);
- this.profile_.addFuncCode(type, name, start, size, funcAddr, state);
+ this.profile_.addFuncCode(type, name, timestamp, start, size, funcAddr, state);
} else {
- this.profile_.addCode(type, name, start, size);
+ this.profile_.addCode(type, name, timestamp, start, size);
}
};
+TickProcessor.prototype.processCodeDeopt = function(
+ timestamp, size, code, inliningId, scriptOffset, bailoutType,
+ sourcePositionText, deoptReasonText) {
+ this.profile_.deoptCode(timestamp, code, inliningId, scriptOffset,
+ bailoutType, sourcePositionText, deoptReasonText);
+};
+
+
TickProcessor.prototype.processCodeMove = function(from, to) {
this.profile_.moveCode(from, to);
};
« no previous file with comments | « tools/profview/profview.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698