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

Unified Diff: third_party/WebKit/Source/devtools/front_end/profiler/ProfileDataGrid.js

Issue 2151653005: DevTools: Do not linkify to pseudo (program) node when there's no URL (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressing comments 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/profiler/ProfileDataGrid.js
diff --git a/third_party/WebKit/Source/devtools/front_end/profiler/ProfileDataGrid.js b/third_party/WebKit/Source/devtools/front_end/profiler/ProfileDataGrid.js
index c244cf00e0aa6ab4d973adfe5b2be4066a0d8469..0ad215db5d3d430599e4778fbab8c0df75ae3736 100644
--- a/third_party/WebKit/Source/devtools/front_end/profiler/ProfileDataGrid.js
+++ b/third_party/WebKit/Source/devtools/front_end/profiler/ProfileDataGrid.js
@@ -80,6 +80,8 @@ WebInspector.ProfileDataGridNode.prototype = {
if (this.profileNode.scriptId === "0")
break;
var urlElement = this.tree._formatter.linkifyNode(this);
+ if (!urlElement)
+ break;
urlElement.style.maxWidth = "75%";
cell.appendChild(urlElement);
break;
@@ -648,7 +650,7 @@ WebInspector.ProfileDataGridNode.Formatter.prototype = {
/**
* @param {!WebInspector.ProfileDataGridNode} node
- * @return {!Element}
+ * @return {?Element}
*/
linkifyNode: function(node) { }
}

Powered by Google App Engine
This is Rietveld 408576698