dbg-server.jsm includes dbg-client.jsm with resource:/// instead of resource://gre/

RESOLVED FIXED in Firefox 14

Status

()

Firefox
Developer Tools: Debugger
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

13 Branch
Firefox 15
Points:
---

Firefox Tracking Flags

(firefox12 unaffected, firefox13 unaffected, firefox14 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
status-firefox12: --- → unaffected
status-firefox13: --- → unaffected
status-firefox14: --- → affected
Created attachment 618957 [details] [diff] [review]
Import dbg-client.jsm from resource://gre/ instead of resource:///
Attachment #618957 - Flags: review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/925d9ca6a866
Target Milestone: --- → Firefox 15
https://hg.mozilla.org/mozilla-central/rev/925d9ca6a866
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Comment on attachment 618957 [details] [diff] [review]
Import dbg-client.jsm from resource://gre/ instead of resource:///

[Approval Request Comment]
Regression caused by: bug 740803
User impact if declined: Debugger probably doesn't work on Firefox-on-xulrunner setups (used by some Linux distros)
Testing completed (on m-c, etc.): Landed on m-c a couple days ago.
Risk to taking this patch (and alternatives if risky): None. on Mozilla builds, resource://gre/ and resource:/// are strictly identical.
String changes made by this patch: None.
Attachment #618957 - Flags: approval-mozilla-aurora?

Comment 5

5 years ago
Comment on attachment 618957 [details] [diff] [review]
Import dbg-client.jsm from resource://gre/ instead of resource:///

(In reply to Mike Hommey [:glandium] from comment #4)
> User impact if declined: Debugger probably doesn't work on
> Risk to taking this patch (and alternatives if risky): None. on Mozilla
> builds, resource://gre/ and resource:/// are strictly identical.

Although we're not 100% sure of the user impact, it sounds like we're nearly 100% sure we won't cause a regression. Approving for Aurora 14.
Attachment #618957 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
http://hg.mozilla.org/releases/mozilla-aurora/rev/58fb08cbb6d3
status-firefox14: affected → fixed
You need to log in before you can comment on or make changes to this bug.