Closed
Bug 703633
Opened 13 years ago
Closed 13 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•13 years ago
|
||
Updated•13 years ago
|
Status: NEW → ASSIGNED
Component: Developer Tools → Developer Tools: Inspector
QA Contact: developer.tools → developer.tools.inspector
Comment 2•13 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•13 years ago
|
Whiteboard: [land-in-fx-team]
Comment 3•13 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/60ecd909a6c2
Whiteboard: [land-in-fx-team] → [fixed-in-fx-team]
Comment 4•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/60ecd909a6c2
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 11
Comment 5•12 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•12 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•12 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•12 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•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•