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

Unified Diff: third_party/WebKit/Source/devtools/front_end/sdk/ProfileTreeModel.js

Issue 2144153002: [DevTools] Remove CallUID from CPUProfileNode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@make-profile-0-based
Patch Set: rebased tests 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: third_party/WebKit/Source/devtools/front_end/sdk/ProfileTreeModel.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sdk/ProfileTreeModel.js b/third_party/WebKit/Source/devtools/front_end/sdk/ProfileTreeModel.js
index 90b6e26eb3094f165579c19764584e056d03a8a4..ca885ed9d775c53cf74b315f4e7dd9310e92802f 100644
--- a/third_party/WebKit/Source/devtools/front_end/sdk/ProfileTreeModel.js
+++ b/third_party/WebKit/Source/devtools/front_end/sdk/ProfileTreeModel.js
@@ -15,8 +15,8 @@ WebInspector.ProfileNode = function(functionName, scriptId, url, lineNumber, col
lineNumber: lineNumber,
columnNumber: columnNumber
};
- /** @type {number|string} */
- this.callUID;
+ /** @type {string} */
+ this.callUID = `${this.frame.functionName}@${this.frame.scriptId}:${this.frame.lineNumber}`;
/** @type {number} */
this.self = 0;
/** @type {number} */

Powered by Google App Engine
This is Rietveld 408576698