The default bug view has changed. See this FAQ.

TreePanel.jsm uses a resource:/// url that ought to be resource://gre/

RESOLVED FIXED in Firefox -esr10

Status

()

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

People

(Reporter: glandium, Assigned: glandium)

Tracking

unspecified
Firefox 11
Points:
---

Firefox Tracking Flags

(firefox-esr10 fixed)

Details

(Whiteboard: [fixed-in-fx-team][qa-])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
TreePanel.jsm includes resource:///modules/Services.jsm, which should really be resource://gre/modules/Services.jsm.
(Assignee)

Comment 1

5 years ago
Created attachment 575489 [details] [diff] [review]
TreePanel.jsm uses a resource:/// url that ought to be resource://gre/
Status: NEW → ASSIGNED
Component: Developer Tools → Developer Tools: Inspector
QA Contact: developer.tools → developer.tools.inspector
Comment on attachment 575489 [details] [diff] [review]
TreePanel.jsm uses a resource:/// url that ought to be resource://gre/

thanks!
Attachment #575489 - Flags: review+
Whiteboard: [land-in-fx-team]
https://hg.mozilla.org/integration/fx-team/rev/60ecd909a6c2
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/60ecd909a6c2
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 11

Comment 5

5 years ago
Comment on attachment 575489 [details] [diff] [review]
TreePanel.jsm uses a resource:/// url that ought to be resource://gre/

[Approval Request Comment]
If this is not a sec:{high,crit} bug, please state case for ESR consideration:
User impact if declined: Inspect Element won't work
Fix Landed on Version: Firefox 11
Risk to taking this patch (and alternatives if risky): NONE
String or UUID changes made by this patch: NONE
Attachment #575489 - Flags: approval-mozilla-esr10?
(Assignee)

Comment 6

5 years ago
(In reply to Martin Stránský from comment #5)
> User impact if declined: Inspect Element won't work

won't work in Firefox as a Xulrunner application.
Comment on attachment 575489 [details] [diff] [review]
TreePanel.jsm uses a resource:/// url that ought to be resource://gre/

no risk, go ahead.
Attachment #575489 - Flags: approval-mozilla-esr10? → approval-mozilla-esr10+
(Assignee)

Comment 8

5 years ago
Realized too late that I pushed to the wrong branch:
http://hg.mozilla.org/releases/mozilla-esr10/rev/344e4df16abb
And landed again on the right one:
http://hg.mozilla.org/releases/mozilla-esr10/rev/454e3e33b5c3
status-firefox-esr10: --- → fixed
Does this have or need a test case in-testsuite?
Flags: in-testsuite?
Whiteboard: [fixed-in-fx-team] → [fixed-in-fx-team][qa-]
no.
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.