Get rid of unused bindings

RESOLVED FIXED in Firefox 16

Status

()

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Margaret, Assigned: Margaret)

Tracking

Trunk
Firefox 16
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
Created attachment 632832 [details] [diff] [review]
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.
Attachment #632832 - Flags: review?(mbrubeck)
Attachment #632832 - Flags: review?(mbrubeck) → review+
(Assignee)

Comment 1

6 years ago
Created attachment 632841 [details] [diff] [review]
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.
Assignee: nobody → margaret.leibovic
Attachment #632841 - Flags: review?(mbrubeck)
Attachment #632841 - Flags: review?(mbrubeck) → review+
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.
(Assignee)

Comment 4

6 years ago
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

6 years ago
https://hg.mozilla.org/mozilla-central/rev/887bc4390e12
https://hg.mozilla.org/mozilla-central/rev/b3a209238d8b
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.