1803505 - Simplify the tab-pickup-container initialization

Status: RESOLVED FIXED

(Firefox :: Firefox View, task, P3)

Description

[Sam Foster [:sfoster] (he/him)]

While working on bug 1792040 I've run into a number of issues which needlessly complicate the sequence of events which lead up to the tab pickup list being in a known and testable state.

I'd like to remove the lazy-insertion of the setup steps and tab-pickup-list children. The original thought was to avoid adding bloat to the document until those elements were needed, but with hindsight this seems like premature optimization, and not knowing exactly whether and when those elements will exist makes reasoning about the sequence and state of things more difficult than it needs to be.

Comment 1

[Sam Foster [:sfoster] (he/him)]

Attachment: Bug 1803505 - Simplify the tab-pickup-container initialization. r?sclements!

• Adds a promise that resolves when the tab-pickup-list is ready
• Use that promise to trigger tab data request for the list when we enter the setup-complete state
• Remove the workarounds for overlapping setup and tabs requests

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/35f1fc58f00f
Simplify the tab-pickup-container initialization. r=sclements

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/35f1fc58f00f