Closed
Bug 764535
Opened 12 years ago
Closed 12 years ago
Get rid of unused bindings
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 16
People
(Reporter: Margaret, Assigned: Margaret)
Details
Attachments
(2 files)
15.45 KB,
patch
|
mbrubeck
:
review+
|
Details | Diff | Splinter Review |
2.46 KB,
patch
|
mbrubeck
:
review+
|
Details | Diff | Splinter Review |
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)
Updated•12 years ago
|
Attachment #632832 -
Flags: review?(mbrubeck) → review+
Assignee | ||
Comment 1•12 years ago
|
||
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)
Updated•12 years ago
|
Attachment #632841 -
Flags: review?(mbrubeck) → review+
Assignee | ||
Comment 2•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/639edd2b32d4 https://hg.mozilla.org/integration/mozilla-inbound/rev/ed90d02fd908
Target Milestone: --- → Firefox 16
Comment 3•12 years ago
|
||
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•12 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•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/887bc4390e12 https://hg.mozilla.org/mozilla-central/rev/b3a209238d8b
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•