Closed Bug 1677164 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 26531 - [layout] Don't freeze OOF-positioned scrollbars within flex layout.

Categories

(Core :: Layout: Flexbox, task, P4)

Product:
Core
Component:
Layout: Flexbox
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
85 Branch
Tracking Flags:
Tracking Status
firefox85 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 26531 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/26531
Details from upstream follow.

b'Ian Kilpatrick <ikilpatrick@chromium.org>' wrote:

[layout] Don't freeze OOF-positioned scrollbars within flex layout.

Change:
https://chromium-review.googlesource.com/c/chromium/src/+/2424899
... introdcued a fix for freeze more consistent scrollbars for
arbitrary descendants of flex layout. However the original code
(detected direct descendant scrollbar changes) contained an issue that
it would also freeze scrollbars for any OOF positioned children.

There exists a wider issue which I've detailed here:
https://bugs.chromium.org/p/chromium/issues/detail?id=1148835
But it's relative rare.

The fix for this specific regression is to unfreeze the scrollbars
before running the OOF layout part.

Bug: 1148288
Change-Id: I74be0bf141d13988548d38412e912c8c98d628b1
Reviewed-on: https://chromium-review.googlesource.com/2538041
WPT-Export-Revision: 798e8d47517cc82166f1f9c9f52636204f346278

Component: web-platform-tests → Layout: Flexbox
Product: Testing → Core

CI Results

Ran 14 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests

Status Summary

Firefox

PASS: 1

Chrome

FAIL: 1

Safari

PASS: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e3400086c1fa
[wpt PR 26531] - [layout] Don't freeze OOF-positioned scrollbars within flex layout., a=testonly

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

Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox85: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch
You need to log in before you can comment on or make changes to this bug.