Closed
Bug 1286390
Opened 8 years ago
Closed 8 years ago
Source Actor form should include sourceMapURL
Categories
(DevTools :: Debugger, defect, P3)
DevTools
Debugger
Tracking
(firefox49 fixed, firefox50 fixed)
RESOLVED
FIXED
Firefox 50
People
(Reporter: jlast, Assigned: jlast)
References
()
Details
Attachments
(1 file)
1.17 KB,
patch
|
jlong
:
review+
gchang
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-beta-
|
Details | Diff | Splinter Review |
The sourceMapURL is needed for client-side source map work. The goal is that the source actor form will include the source map's url so that the client can separately fetch the map.
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → jlaster
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8770304 -
Flags: review?(jlong)
Comment 2•8 years ago
|
||
Comment on attachment 8770304 [details] [diff] [review]
sourceMapUrl.patch
Review of attachment 8770304 [details] [diff] [review]:
-----------------------------------------------------------------
Nice! We should uplift this to beta. It's a harmless patch and will allow the new client to work on a few older versions of Firefox.
Attachment #8770304 -
Flags: review?(jlong) → review+
Updated•8 years ago
|
Priority: -- → P3
Assignee | ||
Comment 3•8 years ago
|
||
Assignee | ||
Comment 4•8 years ago
|
||
Comment on attachment 8770304 [details] [diff] [review]
sourceMapUrl.patch
Approval Request Comment
[Feature/regressing bug #]:
[User impact if declined]:
[Describe test coverage new/current, TreeHerder]:
[Risks and why]:
[String/UUID change made/needed]:
Attachment #8770304 -
Flags: approval-mozilla-beta?
Attachment #8770304 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 5•8 years ago
|
||
[Feature/regressing bug #]: 1286390
[User impact if declined]: Debugger clients will not be able to source map sources.
[Describe test coverage new/current, TreeHerder]: it doesn't need any new test coverage, we don't test form structures.
[Risks and why]: minimal, sourceMapURL is a source field.
[String/UUID change made/needed]: n/a
Updated•8 years ago
|
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/fx-team/rev/25d022b355be
Add sourceMapURL to source actor form. r=jlongster
Keywords: checkin-needed
Comment 7•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox50:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
Comment 8•8 years ago
|
||
Jason, can you tell me which bug introduced this issue?
> [Feature/regressing bug #]: 1286390
is incorrect.
Assignee | ||
Comment 9•8 years ago
|
||
Hi Sylvestre, sorry for the confusion, this isn't a current issue but a feature we hope to add to support client-side source maps.
Flags: needinfo?(jlaster)
Comment 10•8 years ago
|
||
Thanks for the information.
Unfortunately, this is way too late for 48 as we are the end of the cycle.
status-firefox48:
affected → ---
Updated•8 years ago
|
Attachment #8770304 -
Flags: approval-mozilla-beta? → approval-mozilla-beta-
Comment 11•8 years ago
|
||
Comment on attachment 8770304 [details] [diff] [review]
sourceMapUrl.patch
Review of attachment 8770304 [details] [diff] [review]:
-----------------------------------------------------------------
This patch adds support for Source Actor. Let's take it in 49 aurora.
Attachment #8770304 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 12•8 years ago
|
||
bugherder uplift |
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•