Closed Bug 1680258 Opened 3 years ago Closed 3 years ago

Enable Pre-XUL skeleton UI on Nightly

Categories

(Firefox :: General, task)

task

Tracking

()

RESOLVED FIXED
86 Branch
Tracking Status
firefox86 --- fixed

People

(Reporter: alexical, Assigned: alexical)

References

(Blocks 1 open bug)

Details

(Whiteboard: [skeleton-ui-experiment-blocker])

Attachments

(8 files)

No description provided.

Previously, we implemented arg checking with marionette just carrying a free
pass, so we could let the arguments which typically come when running tests.
However, some marionette tests do like to play with arguments which we do not
want to get a free pass, such as -safe-mode. These changes allow just the
-profile argument through, as that is necessary for running tests.

Some env vars have effects similar to command line arguments which present
problems for the skeleton UI, and we want to treat these env vars similarly.

Depends on D98474

This prevents an assertion from tripping below.

Depends on D98475

Having the window present earlier changes the ordering of these events, such
that it doesn't come through at the time that the browser_startup test needs it
to. I think this event should be correct anyway, given that we already painted
the skeleton UI?

Depends on D98476

This was causing us to have slightly wrong focusring behavior, leading to wpt
failures in focus-visible-005.html.

Depends on D98478

Depends on D98479

I don't have a crystal clear story of why these timings are different with the
skeleton UI enabled. However, it's not remarkably surprising that it changes the
order of some events during startup which come from the core Windows event loop.
I don't think it's worth diving incredibly deep to understand this - looking at
the profile we still only see one of each of these events.

Depends on D99110

Keywords: leave-open
Pushed by dothayer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e9b9f97caa6c
Improve skeleton UI arg checking for marionette r=Gijs
https://hg.mozilla.org/integration/autoland/rev/268d2aac128e
Check for problematic env vars for skeleton UI r=Gijs
https://hg.mozilla.org/integration/autoland/rev/183d89827d4b
Don't try to draw rects with 0 width or height r=emalysz
https://hg.mozilla.org/integration/autoland/rev/a253801a3155
Change first paint notification event when skele UI enabled r=florian
https://hg.mozilla.org/integration/autoland/rev/403d7441eb0c
Ensure UI state initialized for skeleton UI r=mhowell
https://hg.mozilla.org/integration/autoland/rev/8eb4ebbd13d0
Ensure we properly clamp values to draw region r=mstange
https://hg.mozilla.org/integration/autoland/rev/4b885b40c787
Relax startup IPC test for skeleton UI due to paint timings r=florian
Backout by malexandru@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a4dc1badd9c7
Backed out 7 changesets for causing osx mochitest timeouts.

I'm very confused by this? None of this is even compiled on OSX.

Flags: needinfo?(dothayer)
Pushed by dothayer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b2ec458a63aa
Improve skeleton UI arg checking for marionette r=Gijs
https://hg.mozilla.org/integration/autoland/rev/b937607a2086
Check for problematic env vars for skeleton UI r=Gijs
https://hg.mozilla.org/integration/autoland/rev/dc51252ef65c
Don't try to draw rects with 0 width or height r=emalysz
https://hg.mozilla.org/integration/autoland/rev/4b8533805da7
Change first paint notification event when skele UI enabled r=florian
https://hg.mozilla.org/integration/autoland/rev/3c29ebb65840
Ensure UI state initialized for skeleton UI r=mhowell
https://hg.mozilla.org/integration/autoland/rev/00e863166a86
Ensure we properly clamp values to draw region r=mstange
https://hg.mozilla.org/integration/autoland/rev/4f79a1f41723
Relax startup IPC test for skeleton UI due to paint timings r=florian
Keywords: leave-open
Pushed by dothayer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c5bcb9f8eab9
Enable skeleton UI on nightly r=mhowell
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 86 Branch
Blocks: 1687531
Regressions: 1697091
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: