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

Unified Diff: third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js

Issue 1699183003: [DevTools] Don't load sourcemap for blackboxed scripts (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@add-source-map-support-v3
Patch Set: Created 4 years, 10 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 | « third_party/WebKit/Source/devtools/front_end/bindings/DebuggerWorkspaceBinding.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js b/third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js
index 4841b8bd2552e87c53e75ba3fa6bc9ab2cdc627d..5a73f41a977b30a3d44825b33a889f7bd1e711c8 100644
--- a/third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js
+++ b/third_party/WebKit/Source/devtools/front_end/sources/JavaScriptSourceFrame.js
@@ -162,6 +162,10 @@ WebInspector.JavaScriptSourceFrame.prototype = {
this._blackboxInfobar = infobar;
infobar.createDetailsRowMessage(WebInspector.UIString("Debugger will skip stepping through this script, and will not stop on exceptions"));
+
+ var scriptFile = this._scriptFileForTarget.valuesArray()[0];
+ if (scriptFile.hasSourceMapURL())
+ infobar.createDetailsRowMessage(WebInspector.UIString("Source map found, but ignored for blackboxed file."));
infobar.createDetailsRowMessage();
infobar.createDetailsRowMessage(WebInspector.UIString("Possible ways to cancel this behavior are:"));
@@ -345,7 +349,7 @@ WebInspector.JavaScriptSourceFrame.prototype = {
*/
function populateSourceMapMembers()
{
- if (this.uiSourceCode().project().type() === WebInspector.projectTypes.Network && WebInspector.moduleSetting("jsSourceMapsEnabled").get()) {
+ if (this.uiSourceCode().project().type() === WebInspector.projectTypes.Network && WebInspector.moduleSetting("jsSourceMapsEnabled").get() && !WebInspector.blackboxManager.isBlackboxedUISourceCode(this.uiSourceCode())) {
if (this._scriptFileForTarget.size) {
var scriptFile = this._scriptFileForTarget.valuesArray()[0];
var addSourceMapURLLabel = WebInspector.UIString.capitalize("Add ^source ^map\u2026");
« no previous file with comments | « third_party/WebKit/Source/devtools/front_end/bindings/DebuggerWorkspaceBinding.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698