764535 - Get rid of unused bindings

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[:Margaret Leibovic]

Attachment: patch

I managed to find some more XUL fennec cruft while investigating my XBL issues. We don't use browser.css in native fennec, so we can get rid of it, as well as the bindings it references. Note that I left the menulist binding in bindings.xml, as well as checkbox.xml and settings.xml, since we do use those in native fennec.

Comment 1

[:Margaret Leibovic]

Attachment: Part 2 - Move menulist binding into its own file

Now that we only have one binding in bindings.xml, I think it makes more sense to stick it in its own file under /bindings, for consistency with the other bindings.

Comment 2

[:Margaret Leibovic]

https://hg.mozilla.org/integration/mozilla-inbound/rev/639edd2b32d4
https://hg.mozilla.org/integration/mozilla-inbound/rev/ed90d02fd908

Comment 3

[Phil Ringnalda (:philor)]

Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/e4861bc9f86f - other than the orange in mochitests 1-3 and 6-8, robocop, crashtests and reftests, everything when swimmingly with that push.

Comment 4

[:Margaret Leibovic]

I pushed to try to make sure these patches weren't the guilty ones. Re-landing:
https://hg.mozilla.org/integration/mozilla-inbound/rev/887bc4390e12
https://hg.mozilla.org/integration/mozilla-inbound/rev/b3a209238d8b

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/887bc4390e12
https://hg.mozilla.org/mozilla-central/rev/b3a209238d8b