Open Bug 1488969 Opened 7 years ago Updated 3 years ago

Unify nsIPresShell::mVisualViewportOffset and mVisualViewportSize into a rect

Categories

(Core :: Panning and Zooming, enhancement, P3)

enhancement

Tracking

()

Tracking Status
firefox64 --- affected

People

(Reporter: botond, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [gfx-noted])

In bug 1470267, we started storing the visual viewport offset in nsIPresShell, in a field called mVisualViewportOffset. In bug 1471708, we renamed nsIPresShell::mScrollPositionClampingScrollPortSize to mVisualViewportSize. This offset and size are conceptually part of the same quantity, the visual viewport rect. We should reflect this by storing them together as a rect. However, currently these quantities are computed and set independently by different pieces of code. If we do combine them into the rect, we should make sure that the entire rect has a sensible value at all times.
Priority: -- → P3
Whiteboard: [gfx-noted]
Depends on: 1489910
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.