Closed Bug 1343843 Opened 7 years ago Closed 7 years ago

Remove chrome://global/content/url-classifier/unittests.xul

Categories

(Toolkit :: Safe Browsing, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox54 --- affected
firefox57 --- fixed

People

(Reporter: florian, Assigned: stevea1, Mentored)

References

Details

(Keywords: good-first-bug, Whiteboard: [lang=xul|js])

Attachments

(1 file)

chrome://global/content/url-classifier/unittests.xul is shipped on all 3 platforms, but that doesn't seem to be intentional.

That file was removed 10 years ago at bug 387196 comment 15, but that patch got backed out (bug 387196 comment 16) for reasons that don't seem related to this file, and the file is never mentioned again in the bug comments.
Keywords: good-first-bug
Priority: -- → P3
See Also: → 1297614
I'm happy to mentor this bug. Here's a place to look for what to remove:

http://searchfox.org/mozilla-central/search?q=unittests.xul&case=false&regexp=false&path=
Mentor: standard8
Whiteboard: [lang=xul|js]
I've posted a patch for this. One thing I was a little unclear on: removing unittests.xul from toolkit/components/url-classifier/tests/jar.mn left it essentially empty - wasn't sure if jar.mn should therefore be removed as well. I left it in place but I can remove it if that's preferred.
(In reply to Steve Armand from comment #3)
> I've posted a patch for this.

Thanks!

> One thing I was a little unclear on: removing
> unittests.xul from toolkit/components/url-classifier/tests/jar.mn left it
> essentially empty - wasn't sure if jar.mn should therefore be removed as
> well.

Yes, please. And also remove the reference to this jar.mn file at http://searchfox.org/mozilla-central/rev/2aa0806c598ec433e431728f5ddd3a6847c1a417/toolkit/components/url-classifier/tests/moz.build#12
Ok, I've made the update: removed jar.mn as well as the reference to the file in toolkit/components/url-classifier/tests/moz.build.
Attachment #8903874 - Flags: review?(standard8) → review?(florian)
Comment on attachment 8903874 [details]
Bug 1343843 - Remove unused chrome://global/content/url-classifier/unittests.xul.

Thanks for the update, I can review this.
Attachment #8903874 - Flags: review?(florian) → review?(standard8)
Assignee: nobody → stevea1
Comment on attachment 8903874 [details]
Bug 1343843 - Remove unused chrome://global/content/url-classifier/unittests.xul.

https://reviewboard.mozilla.org/r/175648/#review180902

This is great, thank you for the patch.
Attachment #8903874 - Flags: review?(standard8) → review+
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a7c3203be89a
Remove unused chrome://global/content/url-classifier/unittests.xul. r=standard8
https://hg.mozilla.org/mozilla-central/rev/a7c3203be89a
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.