third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js - Issue 2422803002: DevTools: prefix node.js scripts's sourceURL with "file://" to make them a valid url

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

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

Issue 2422803002: DevTools: prefix node.js scripts's sourceURL with "file://" to make them a valid url (Closed)

Patch Set: better name Created 4 years, 2 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/persistence/Persistence.js ('k') | third_party/WebKit/Source/devtools/front_end/sdk/Target.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js

diff --git a/third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js b/third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js

index 20f722c8d7d6d60b5253f968e0b66e5d9df16082..e51686206e106dcafdb136c00bd7ec8a6d96315f 100644

--- a/third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js

+++ b/third_party/WebKit/Source/devtools/front_end/sdk/DebuggerModel.js

@@ -474,6 +474,9 @@ WebInspector.DebuggerModel.prototype = {

var isContentScript = false;

if (executionContextAuxData && ("isDefault" in executionContextAuxData))

isContentScript = !executionContextAuxData["isDefault"];

+ // Support file URL for node.js.

+ if (this.target().isNodeJS() && sourceURL && sourceURL.startsWith("/"))

+ sourceURL = "file://" + sourceURL;

dgozman 2016/10/18 21:46:12 WebInspector.IsolatedFileSystemManager.normalizePa

var script = new WebInspector.Script(this, scriptId, sourceURL, startLine, startColumn, endLine, endColumn, executionContextId, hash, isContentScript, isLiveEdit, sourceMapURL, hasSourceURL);

this._registerScript(script);

if (!hasSyntaxError)