Open
Bug 413251
Opened 18 years ago
Updated 2 years ago
Useless code in browser.xml
Categories
(Toolkit :: UI Widgets, defect)
Toolkit
UI Widgets
Tracking
()
NEW
People
(Reporter: neil, Unassigned)
References
()
Details
Attachments
(1 file)
|
772 bytes,
patch
|
Details | Diff | Splinter Review |
Most of the feed handling was moved out of tabbrowser.xml and browser.xml into browser.js however a couple of lines of redundant code remain, c.f. the URL.
|
Reporter | |
Comment 1•18 years ago
|
||
This is the only reference to the feeds property within toolkit.
|
||
Updated•18 years ago
|
|
|
Reporter | |
Comment 2•18 years ago
|
||
Ok, so gavin suggests that the linked code ben added to browser.js doesn't properly replace this code here, so browser.js needs some love too...
|
||
Comment 3•18 years ago
|
||
Comment on attachment 298151 [details] [diff] [review]
Proposed patch
per your comment.
Attachment #298151 -
Flags: review?(mano)
| Comment hidden (off-topic) |
|
||
Comment 5•4 years ago
|
||
Sorry, there was a problem with the detection of inactive users. I'm reverting the change.
Assignee: nobody → neil
|
||
Updated•3 years ago
|
Severity: normal → S3
|
||
Comment 6•2 years ago
|
||
The bug assignee is inactive on Bugzilla, so the assignee is being reset.
Assignee: neil → nobody
You need to log in
before you can comment on or make changes to this bug.
Description
•