This is only ever used to check for CHECKSTATE_MIXED which in turn is used to set mozilla::a11y::states::MIXED - https://searchfox.org/mozilla-central/search?q=%3A%3Acheckstate&path= However, there's only one place we ever set that state - in the about:addons 'Recent Updates' list. The mixed state is set when the list is ascending, which doesn't seem right. The element is 'checked' whether it's ascending or descending. https://searchfox.org/mozilla-central/rev/59a9a86553e9bfd9277202748ff791fd9bc0713b/toolkit/mozapps/extensions/content/extensions.xml#302 If we get rid of that consumer, I believe we can remove all the idl and accessibility code handling this case, which should help unblock switching to an actual html:input (since the only remaining attribute on nsIDOMXULCheckboxElement.idl after this and Bug 1455359 will be 'checked').

Andrew - we could update the checkState to always be '1' for the selected header. But.. could we just remove this UI? The default sort order is by Date (most recent first) which seems like the correct order for a 'recent updates' list.

Comment on attachment 8969410 [details] Bug 1455392 - Remove checkState from nsIDOMXULCheckboxElement and nsIDOMXULButtonElement https://reviewboard.mozilla.org/r/238164/#review243884 r=me for a11y part

(In reply to Brian Grinstead [:bgrins] from comment #1) > Andrew - we could update the checkState to always be '1' for the selected > header. But.. could we just remove this UI? The default sort order is by > Date (most recent first) which seems like the correct order for a 'recent > updates' list. Works for me, ccing Emanuela and Markus from UX so they are not surprised. Odds are high that somebody will pop up in the next few weeks and say that they have a hundred extensions installed and now they can't effectively find things in recent updates, but I don't think that should stand in the way of de-XBLing

Comment on attachment 8969409 [details] Bug 1455392 - Remove sortable UI in about:addons recent updates; https://reviewboard.mozilla.org/r/238162/#review243886

For reference - this is the UI we are talking about

Comment on attachment 8969410 [details] Bug 1455392 - Remove checkState from nsIDOMXULCheckboxElement and nsIDOMXULButtonElement Requesting review for the dom idl changes

Comment on attachment 8969410 [details] Bug 1455392 - Remove checkState from nsIDOMXULCheckboxElement and nsIDOMXULButtonElement https://reviewboard.mozilla.org/r/238164/#review244262 r=me, but you don't have to rev iids...

Pushed by bgrinstead@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8a24709c044b Remove sortable UI in about:addons recent updates;r=aswan https://hg.mozilla.org/integration/autoland/rev/e166407fe8b0 Remove checkState from nsIDOMXULCheckboxElement and nsIDOMXULButtonElement;r=bz,surkov

https://hg.mozilla.org/mozilla-central/rev/8a24709c044b https://hg.mozilla.org/mozilla-central/rev/e166407fe8b0

Moving to Core:XUL per https://bugzilla.mozilla.org/show_bug.cgi?id=1455336