Closed
Bug 703633
Opened 14 years ago
Closed 14 years ago
TreePanel.jsm uses a resource:/// url that ought to be resource://gre/
Categories
(DevTools :: Inspector, defect)
DevTools
Inspector
Tracking
(firefox-esr10 fixed)
RESOLVED
FIXED
Firefox 11
| Tracking | Status | |
|---|---|---|
| firefox-esr10 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
Details
(Whiteboard: [fixed-in-fx-team][qa-])
Attachments
(1 file)
|
1.03 KB,
patch
|
rcampbell
:
review+
lsblakk
:
approval-mozilla-esr10+
|
Details | Diff | Splinter Review |
TreePanel.jsm includes resource:///modules/Services.jsm, which should really be resource://gre/modules/Services.jsm.
|
Assignee | |
Comment 1•14 years ago
|
||
|
||
Updated•14 years ago
|
Status: NEW → ASSIGNED
Component: Developer Tools → Developer Tools: Inspector
QA Contact: developer.tools → developer.tools.inspector
|
|
||
Comment 2•14 years ago
|
||
Comment on attachment 575489 [details] [diff] [review]
TreePanel.jsm uses a resource:/// url that ought to be resource://gre/
thanks!
Attachment #575489 -
Flags: review+
|
|
||
Updated•14 years ago
|
Whiteboard: [land-in-fx-team]
|
|
||
Comment 3•14 years ago
|
||
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
|
|
||
Comment 4•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 11
|
||
Comment 5•13 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•13 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 7•13 years ago
|
||
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•13 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-]
|
||
Updated•7 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•