Closed Bug 1670343 Opened 4 years ago Closed 4 years ago

add a few weak frame checks after bug 1651332

Categories

(Core :: Panning and Zooming, defect)

defect

Tracking

()

RESOLVED FIXED
83 Branch
Tracking Status
firefox83 --- fixed

People

(Reporter: tnikkel, Assigned: tnikkel)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Bug 1651332 added a UpdateScrollbarPosition call to SetVisualViewportOffset. UpdateScrollbarPosition calls are surrounded by weak frame checks from bug 898871. These may not be needed anymore, the stacks from bug 898871 might be impossible now, but I'm not sure, so I'll just add the weak frame checks etc here.

Keywords: leave-open
Pushed by tnikkel@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/725180017480 Call SetVisualViewportOffset after nsHTMLScrollFrame::Reflow so we can add a weak frame check around it. r=kats
Keywords: leave-open
Pushed by tnikkel@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e528f398e166 Re-get the frame in ScrollFrame in APZCCallbackHelper after calling SetVisualViewportOffset. r=kats
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
Has Regression Range: --- → yes
Keywords: regression
Blocks: 1753372
Type: task → defect
Blocks: 1753374
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: