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

Unified Diff: Source/devtools/front_end/ConsoleViewMessage.js

Issue 211493002: DevTools: Show user code location for wrapped console.log() calls. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: addressed Created 6 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 | « LayoutTests/inspector/console/resources/framework.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/ConsoleViewMessage.js
diff --git a/Source/devtools/front_end/ConsoleViewMessage.js b/Source/devtools/front_end/ConsoleViewMessage.js
index 18f54e27ffa3d8275ec9a5b94247573f2ec6f2e8..bc530d79165b8b94fa9e968cc98e191f21066997 100644
--- a/Source/devtools/front_end/ConsoleViewMessage.js
+++ b/Source/devtools/front_end/ConsoleViewMessage.js
@@ -30,7 +30,7 @@
/**
* @constructor
- *
+ * @param {!WebInspector.Target} target
* @param {!WebInspector.ConsoleMessage} consoleMessage
* @param {?WebInspector.Linkifier} linkifier
*/
@@ -161,11 +161,11 @@ WebInspector.ConsoleViewMessage.prototype = {
}
if (consoleMessage.source !== WebInspector.ConsoleMessage.MessageSource.Network || consoleMessage.request) {
- if (consoleMessage.stackTrace && consoleMessage.stackTrace.length && consoleMessage.stackTrace[0].scriptId) {
- this._anchorElement = this._linkifyCallFrame(consoleMessage.stackTrace[0]);
- } else if (consoleMessage.url && consoleMessage.url !== "undefined") {
+ var callFrame = this._callFrameAnchorFromStackTrace(consoleMessage.stackTrace);
+ if (callFrame)
+ this._anchorElement = this._linkifyCallFrame(callFrame);
+ else if (consoleMessage.url && consoleMessage.url !== "undefined")
this._anchorElement = this._linkifyLocation(consoleMessage.url, consoleMessage.line, consoleMessage.column);
- }
}
this._formattedMessage.appendChild(this._messageElement);
@@ -248,6 +248,30 @@ WebInspector.ConsoleViewMessage.prototype = {
},
/**
+ * @param {?Array.<!ConsoleAgent.CallFrame>} stackTrace
+ * @return {?ConsoleAgent.CallFrame}
+ */
+ _callFrameAnchorFromStackTrace: function(stackTrace)
+ {
+ if (!stackTrace || !stackTrace.length)
+ return null;
+ var callFrame = stackTrace[0].scriptId ? stackTrace[0] : null;
+ if (!WebInspector.experimentsSettings.frameworksDebuggingSupport.isEnabled())
+ return callFrame;
+ if (!WebInspector.settings.skipStackFramesSwitch.get())
+ return callFrame;
+ var regex = WebInspector.settings.skipStackFramesPattern.asRegExp();
+ if (!regex)
+ return callFrame;
+ for (var i = 0; i < stackTrace.length; ++i) {
+ var script = this._target.debuggerModel.scriptForId(stackTrace[i].scriptId);
+ if (!script || !regex.test(script.sourceURL))
+ return stackTrace[i].scriptId ? stackTrace[i] : null;
+ }
+ return callFrame;
+ },
+
+ /**
* @return {boolean}
*/
isErrorOrWarning: function()
« no previous file with comments | « LayoutTests/inspector/console/resources/framework.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698