Closed Bug 1677642 Opened 4 years ago Closed 4 years ago

Investigate whether the scrollable overflow area should use UnionRectEdges

Categories

(Core :: Layout, task)

task

Tracking

()

RESOLVED FIXED
85 Branch
Tracking Status
firefox85 --- fixed

People

(Reporter: TYLin, Assigned: TYLin)

References

Details

Attachments

(2 files)

We have comments in nsOverflowAreas::UnionWith and nsOverflowAreas::UnionAllWith about using UnionRectEdges for scrollable overflow rect.

This bug is to investigate whether these comments make sense or not. If yes, we should update the code, or the comments should be deleted.

Attached file Testcase

I tend to think the comments should be deleted. Chrome agrees that empty boxes doesn't affect the scrollable overflow, so we should probably add a WPT to assert that.

Assignee: nobody → aethanyc
Status: NEW → ASSIGNED
Pushed by aethanyc@gmail.com: https://hg.mozilla.org/integration/autoland/rev/ccf878a677f0 Add a wpt testing an empty box doesn't contribute to parent's scrollable overflow area. r=emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/26583 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: