Regression on macOS: Fullscreen behavior is broken
Categories
(Core :: Widget: Cocoa, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr102 | --- | unaffected |
firefox107 | --- | unaffected |
firefox108 | --- | verified |
firefox109 | --- | verified |
People
(Reporter: mehmet.sahin, Assigned: edgar)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
diannaS
:
approval-mozilla-beta+
|
Details | Review |
Nightly 109.0a1 (2022-11-18) (64-Bit)
macOS 12.6.1
1.) Open a new window on the Desktop
2.) Click on the Green Window Control Button to switch into Fullscreen
3.) Click on the Red Window Control Button or press CMD-W to close the window so that you are back on the Desktop again
4.) Open a new window with CMD-N
Actual: Fullscreen window opens
Expected: The new window should open on the Desktop
This is a regression.
Strangely I can no longer reproduce this issue?!
I see this issue again in Nighty 109.0a1 (2022-11-22) (64-Bit) on macOS 12.6.1 :(
Okay, I have now the correct steps to reproduce:
1.) Open a new window on the Desktop
2.) Click on the Green Window Control Button to switch into Fullscreen
3.) Press CMD-N to open a second new window in fullscreen (notice the janky animation - see bug 1801460)
3.) Now, close both fullscreen windows with CMD-W or by clicking on the red window control button
4.) Click on the Nightly Dock icon to open a new window on the desktop
Actual: New window in Fullscreen open.
Expected: New window should open on the desktop.
This is really a bad regression for MacOS experience. Here is the regression range:
Good build: 2022-10-20-10-56-05-mozilla-central
Bad build: 2022-10-20-21-51-26-mozilla-central
Looks like Bug 1795139 in this range touched fullscreen related code. Maybe this is cause of this regression?
echen@ Can you please take a look, if this is related to your change? Many Thanks :)
Assignee | ||
Comment 5•1 year ago
|
||
Bug 1795139 only changed the way to show fullscreen notification, it should not affect how fullscreen is handled in widget, so it is unlikely a regression of Bug 1795139. But I will try to get regression window if possible.
(In reply to Edgar Chen [:edgar] from comment #5)
Bug 1795139 only changed the way to show fullscreen notification, it should not affect how fullscreen is handled in widget, so it is unlikely a regression of Bug 1795139. But I will try to get regression window if possible.
Many thanks in advance !
Assignee | ||
Comment 7•1 year ago
|
||
mozregression points me to Bug 1795804.
Assignee | ||
Comment 8•1 year ago
|
||
It is bascially for macOS as user could switch widget to fullscreen via
window control button in macOS, and the fullscreen state on chrome window isn't
updated yet.
Updated•1 year ago
|
Comment 9•1 year ago
|
||
Set release status flags based on info from the regressing bug 1795804
Updated•1 year ago
|
Comment 11•1 year ago
|
||
Pushed by echen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/06a19335d057 Use widget sizemode to decide whether persistent should be saved; r=emilio
Comment 12•1 year ago
|
||
bugherder |
Comment 13•1 year ago
|
||
The patch landed in nightly and beta is affected.
:edgar, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox108
towontfix
.
For more information, please visit auto_nag documentation.
Assignee | ||
Comment 14•1 year ago
|
||
Comment on attachment 9305178 [details]
Bug 1801459 - Use widget sizemode to decide whether persistent should be saved; r?emilio
Beta/Release Uplift Approval Request
- User impact if declined: The new window might be opened in fullscreen mode wrongly and fullscreen transition is not smooth.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: Follow the STR in https://bugzilla.mozilla.org/show_bug.cgi?id=1801459#c3.
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Small tweak on how we check widget is in fullscreen mode, the change should be safe.
- String changes made/needed: None
- Is Android affected?: No
Assignee | ||
Updated•1 year ago
|
Comment 15•1 year ago
|
||
Comment on attachment 9305178 [details]
Bug 1801459 - Use widget sizemode to decide whether persistent should be saved; r?emilio
Approved for 108.0b8
Comment 16•1 year ago
|
||
bugherder uplift |
Updated•1 year ago
|
Comment 17•1 year ago
|
||
Reproduced in Nightly 109.0a1 (2022-11-22) using macOS 12.
Verified - Fixed in Beta 108.0b9 and the latest Nightly 109.0a1 (2022-12-02) on macOS 12.
Reporter | ||
Comment 18•1 year ago
|
||
Many thanks for fixing this issue!
Description
•