Closed Bug 1665459 Opened 5 years ago Closed 5 years ago

Early skeleton UI resizes sometimes halfway through startup

Categories

(Firefox :: General, defect, P2)

defect

Tracking

()

RESOLVED FIXED
83 Branch
Tracking Status
firefox83 --- fixed

People

(Reporter: alexical, Assigned: alexical)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

NOTE: this is behind the currently disabled Windows skeleton UI feature.

Before we would normally show the early blank window, we apparently sometimes resize the window to a small size. Not sure what this is about, but it is certainly a problem.

This patch uses the prefs relevant to the pre-xul skeleton UI to control the
registry value. There are three prefs:

  • A new pref intended to be the global toggle once all themes can be handled.
  • browser.startup.blankWindow - we currently depend on the code in BrowserGlue.jsm,
    gated on this pref, which immediately specifies the width and height of the
    window and ultimately causes XUL to consume and take ownership of our pre-xul
    window. Without this, we resize to fit a small content area, and then resize
    again back to the correct size.
  • extensions.activeThemeID - Given that we have hardcoded layout and colors, we
    need to ensure that we're only presenting the skeleton UI for the default theme.

We're hoping to not need to gate on any prefs other than the new pref that we're
adding, but this allows us to provide a simple direction for users who may want
to dogfood our changes in the mean time: use the default theme, ensure the
browser.startup.blankWindow pref is set, and turn our new pref on.

Assignee: nobody → dothayer
Status: NEW → ASSIGNED
Pushed by dothayer@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/98c40963e6bb Control pre-xul skele UI registry value via prefs r=mhowell
Severity: -- → S3
Priority: -- → P2
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: