Closed
Bug 140414
Opened 22 years ago
Closed 21 years ago
CVS remove bookmarksDD.js
Categories
(SeaMonkey :: Bookmarks & History, defect)
SeaMonkey
Bookmarks & History
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: p_ch, Assigned: bugs)
Details
Attachments
(1 file)
|
3.75 KB,
patch
|
timeless
:
review+
alecf
:
superreview+
|
Details | Diff | Splinter Review |
bookmarksDD.js is not used any more, since DND is handled in bookmarks.xml this routine defines: - bookmarksDNDObserver - gSpringLoadTracker - NC_NS, RDF_NS all of them are not used outside the bookmarksDD.js I will attach a patch that removes all the occurrences of bookmarksDD.js in LXR. I have tested that everything works as expected.
|
Reporter | |
Comment 1•22 years ago
|
||
|
|
Reporter | |
Comment 2•22 years ago
|
||
since I have no cvs-write access, reassigning to the module owner.
Assignee: pierrechanial → ben
|
||
Updated•22 years ago
|
|
||
Comment 3•22 years ago
|
||
Pierre, is this bug still valid?
|
|
Reporter | |
Comment 4•22 years ago
|
||
I am not sure if the patch still apply. But the bug is still valid.
|
||
Comment 5•22 years ago
|
||
pch, can you update the patch if needed and get r=/sr=?
Attachment #81210 -
Flags: superreview?(alecf)
Attachment #81210 -
Flags: review+
|
||
Comment 6•21 years ago
|
||
Comment on attachment 81210 [details] [diff] [review] removal from LXR again, taking timeless's word on this. sr=alecf
Attachment #81210 -
Flags: superreview?(alecf) → superreview+
|
|
||
Comment 7•21 years ago
|
||
timeless checked in the patch; -> fixed.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
||
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•