Closed
Bug 1228500
Opened 9 years ago
Closed 8 years ago
Handle non-existent mapped sources
Categories
(DevTools :: Debugger, defect)
DevTools
Debugger
Tracking
(firefox45 affected)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox45 | --- | affected |
People
(Reporter: sebo, Unassigned)
References
(Blocks 2 open bugs)
Details
If a mapped source is not available on the server, i.e. the "sources" property in the source map refers to a file that doesn't exist, then the Debugger currently shows the 404 response from the server in the source view.
Instead of showing the server response, the missing source should be handled gracefully. I see two ways to achieve this:
1. Let the source view stay empty and handle the case as if the source were black boxed.
2. Show the minified source instead and handle the case as if no source mapping existed.
Sebastian
Reporter | ||
Updated•9 years ago
|
Updated•9 years ago
|
Blocks: dbg-sourcemap
Comment 1•8 years ago
|
||
Is this a duplicate of bug #1228496?
Reporter | ||
Updated•8 years ago
|
Reporter | ||
Comment 2•8 years ago
|
||
(In reply to devurandom from comment #1)
> Is this a duplicate of bug #1228496?
No, bug 1228496 is meant to add a notification message at the top of the Debugger panel when the source map could not be loaded. This bug is about what to show within the panel instead of the 404 response.
Sebastian
Updated•8 years ago
|
Blocks: source-maps
Comment 3•8 years ago
|
||
(In reply to Sebastian Zartner [:sebo] from comment #0)
> 1. Let the source view stay empty and handle the case as if the source were
> black boxed.
> 2. Show the minified source instead and handle the case as if no source
> mapping existed.
The new debugger takes option 2. If a source mapped file doesn't exist it allows you see and pretty print the original file ( see: https://github.com/devtools-html/debugger.html/pull/1937 )
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•