1665453 - Poll native events while loading xul.dll and other early libraries to prevent being flagged as not responsive

Status: RESOLVED FIXED

(Firefox :: General, enhancement)

Description

[Alex Thayer [:alexical] (she/her)]

We need to poll for messages with PeekMessageW while waiting for xul.dll to load, otherwise the OS will flag us as not responsive and that will arguably be worse from a UX perspective than seeing nothing.

Comment 1

[Alex Thayer [:alexical] (she/her)]

Attachment: Bug 1665453 - Rename EarlyBlankWindow to PreXULSkeletonUI r?agashlin

This is just a more correct name for what's going on.

Comment 2

[Alex Thayer [:alexical] (she/her)]

Attachment: Bug 1665453 - Poll for native events in between prefetching early dlls r?agashlin

In the initial patches for bug 1656526, mhowell noticed that for startups which
take a very long time, if the user interacts with the skeleton UI window, the OS
will flag us as not responsive, which could be a poorer user experience than
seeing nothing. Since our UI is designed to look non-interactive anyway, we
assume that a better experience would be to simply squash the not responsive
response from the OS by trivially processing native events. It's not perfect in,
say, the event that startup is hung for some reason, but it's arguably preferable
to our old model of startup being hung, which was just nothing being displayed at
all.

Depends on D91004

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ec0bad6c498a
Rename EarlyBlankWindow to PreXULSkeletonUI r=agashlin

Comment 4

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/ec0bad6c498a

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0bd98379fad4
Poll for native events in between prefetching early dlls r=agashlin

Comment 6

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/0bd98379fad4