1640493 - Make toolbarbutton's dropmarker optional instead of inserted by default, to reduce CE costs of toolbarbuttons

Status: RESOLVED FIXED

(Toolkit :: UI Widgets, task)

Description

[:Gijs (he/him)]

Split off from bug 1639925 because the text/multiline-text thing is causing weird test issues in seemingly-unrelated code, and the dropmarker part is ready to land.

Comment 1

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/66cbc5a786e2
stop including dropmarkers in all toolbarbuttons, r=mak

Comment 2

[:Gijs (he/him)]

Attachment: Bug 1640493 - stop including dropmarkers in all toolbarbuttons, r?mak

This makes dropmarkers 'optional extras', effectively, and opts the existing
consumers I could find (the places window on macOS, and the 'more engines'
item in the search field popup) into these dropmarkers.

This allows removing all the CSS that was hiding these items in various
places.

We deliberately do not support adding dropmarkers at runtime. I also noticed
that the 'more engines' item already adds the 'badged' attribute after
connecting the node to the DOM, which may result in it not applying properly,
and fixed this.

Comment 3

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/66cbc5a786e2