Closed Bug 1343837 Opened 3 years ago Closed 2 years ago

Remove obsolete chrome://global/content/findUtils.js

Categories

(Toolkit :: Find Toolbar, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: florian, Assigned: dao)

References

Details

Attachments

(1 file)

chrome://global/content/findUtils.js is reported as unreferenced by my test in bug 1316187.

I can't find any use of it in our tree, and dxr doesn't seem to show current usage of it in add-ons. SeaMonkey uses its own fork of this file. I think we can just remove it.
I believe finddialog.xul, finddialog.dtd, finddialog.properties can be removed as well. Although finddialog.xul (and hence the .dtd and .properties) is used by SeaMonkey, so it should probably move those there.
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Comment on attachment 8939773 [details]
Bug 1343837 - Remove find dialog resources from toolkit.

https://reviewboard.mozilla.org/r/210082/#review215856

Looks good to me, thanks for the cleanup :-). This should land on inbound to avoid bitrot with bug 1356029.
Attachment #8939773 - Flags: review?(florian) → review+
Pushed by florian@queze.net:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c28fb6a2c61a
Remove find dialog resources from toolkit. r=florian
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/mozilla-inbound/rev/343de817418c
Remove find dialog resources from toolkit: Remove moz.build rules for removed files to fix build bustage. r=bustage-fix on a CLOSED TREE
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/mozilla-inbound/rev/40ad3b14439f
Remove find dialog resources from toolkit: Remove another moz.build rule for removed files to fix build bustage. r=bustage-fix on a CLOSED TREE
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d63b4b5aab76
Remove find dialog resources from toolkit: Re-add moz.build rule for test files. DONTBUILD on a CLOSED TREE
You need to log in before you can comment on or make changes to this bug.