1797155 - OverflowableToolbar should be able to put overflowable WebExtension browser_actions into the Unified WebExtensions panel.

Status: RESOLVED FIXED

(Firefox :: Toolbars and Customization, enhancement)

Description

[Mike Conley (:mconley) (:⚙️)]

As part of the grander Unified WebExtensions project, we need to make it (when Unified WebExtensions is enabled) such that if a WebExtension browser_action toolbar button is overflowed, it goes into the dedicated Unified WebExtensions panel rather than the traditional overflow panel.

And when the toolbar underflows, it needs to draw from those two separate panels in the same order that the items were inserted in.

Comment 1

[Mike Conley (:mconley) (:⚙️)]

Attachment: Bug 1797155 - Part 1: Don't walk the #list children directly in OverflowableToolbar.#moveItemsBackToTheirOrigin. r?sclements!

This patch makes it so that OverflowableToolbar.#moveItemsBackToTheirOrigin
is not coupled to the existing overflowed item list, and instead takes
advantage of the insert-order provided when calling #overflowedInfo.entries()
and queries the DOM for the overflowed item IDs directly.

A later patch in this series is going to conditionally move overflowed
items into different panels based on whether or not they're provided
by WebExtensions, and so this change means that #moveItemsBackToTheirOrigin
is not going to care which panel the overflowed items are in, it should
behave the same.

Comment 2

[Mike Conley (:mconley) (:⚙️)]

Attachment: Bug 1797155 - Part 2: Make it possible to redirect browser action widgets to the Unified Extension panel on overflow. r?sclements!,willdurand!

Depends on D160292

Comment 3

[Mike Conley (:mconley) (:⚙️)]

Attachment: Bug 1797155 - Part 3: Add tests for Unified Extensions toolbar overflow behaviour. r?willdurand!

Depends on D160293

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3fbfc3d94e5a
Part 1: Don't walk the #list children directly in OverflowableToolbar.#moveItemsBackToTheirOrigin. r=sclements
https://hg.mozilla.org/integration/autoland/rev/863603ed770a
Part 2: Make it possible to redirect browser action widgets to the Unified Extension panel on overflow. r=sclements,willdurand
https://hg.mozilla.org/integration/autoland/rev/ee3f27f43bc5
Part 3: Add tests for Unified Extensions toolbar overflow behaviour. r=willdurand,extension-reviewers

Comment 5

[Marian-Vasile Laza]

Backed out 3 changesets (Bug 1797155) for causing bc failures on browser_unified_extensions_overflowable_toolbar.js.
Backout link
Push with failures <--> bc1
Failure Log

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ec7cdd68c414
Part 1: Don't walk the #list children directly in OverflowableToolbar.#moveItemsBackToTheirOrigin. r=sclements
https://hg.mozilla.org/integration/autoland/rev/b8a8bd646606
Part 2: Make it possible to redirect browser action widgets to the Unified Extension panel on overflow. r=sclements,willdurand
https://hg.mozilla.org/integration/autoland/rev/67f5e4af25b6
Part 3: Add tests for Unified Extensions toolbar overflow behaviour. r=willdurand,extension-reviewers

Comment 7

[Atila Butkovits]

Backed out for causing failures at browser_unified_extensions_overflowable_toolbar.js.

Backout link: https://hg.mozilla.org/integration/autoland/rev/4ca47ed4eeac7cec1f25920964bb6a8066dac69e

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=67f5e4af25b6a173bfbd02329e835c22e3ef599b&selectedTaskRun=LUXa4XTHRPmk2W2FDvG5RA.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=395128428&repo=autoland&lineNumber=5987

Comment 8

[Mike Conley (:mconley) (:⚙️)]

Sorry about that. :/ Doing a broader-range try push now with something that (I think) should address it.

https://treeherder.mozilla.org/jobs?repo=try&revision=c1a6e85bab4891558d65567b67eb2f94886d12a3

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f6168d45dde0
Part 1: Don't walk the #list children directly in OverflowableToolbar.#moveItemsBackToTheirOrigin. r=sclements
https://hg.mozilla.org/integration/autoland/rev/ddbd0486fb24
Part 2: Make it possible to redirect browser action widgets to the Unified Extension panel on overflow. r=sclements,willdurand
https://hg.mozilla.org/integration/autoland/rev/ff538b4eebc9
Part 3: Add tests for Unified Extensions toolbar overflow behaviour. r=willdurand,extension-reviewers

Comment 10

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/f6168d45dde0
https://hg.mozilla.org/mozilla-central/rev/ddbd0486fb24
https://hg.mozilla.org/mozilla-central/rev/ff538b4eebc9