author | Ryan Hunt <rhunt@eqrion.net> |
Sun, 20 Jan 2019 07:21:55 +0000 | |
changeset 454631 | 60944b0fca8ffa2371774326101bffc9a1ed6ec5 |
parent 454630 | 2b62559fb02b566aa10476ea77169df02cee7807 |
child 454632 | f1694d9e640bd1768178e121a5d6fb73565bd32c |
push id | 35406 |
push user | rmaries@mozilla.com |
push date | Mon, 21 Jan 2019 11:32:31 +0000 |
treeherder | mozilla-central@f1694d9e640b [default view] [failures only] |
perfherder | [talos] [build metrics] [platform microbench] (compared to previous push) |
reviewers | dholbert |
bugs | 1521278 |
milestone | 66.0a1 |
first release with | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
last release without | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
--- a/layout/generic/ScrollAnchorContainer.cpp +++ b/layout/generic/ScrollAnchorContainer.cpp @@ -247,28 +247,28 @@ void ScrollAnchorContainer::UserScrolled } void ScrollAnchorContainer::SuppressAdjustments() { ANCHOR_LOG("Received a scroll anchor suppression for %p.\n", this); mSuppressAnchorAdjustment = true; } void ScrollAnchorContainer::InvalidateAnchor() { - if (!StaticPrefs::layout_css_scroll_anchoring_enabled()) { - return; - } - ANCHOR_LOG("Invalidating scroll anchor %p for %p.\n", mAnchorNode, this); if (mAnchorNode) { SetAnchorFlags(mScrollFrame->mScrolledFrame, mAnchorNode, false); } mAnchorNode = nullptr; mAnchorNodeIsDirty = true; mLastAnchorOffset = 0; + + if (!StaticPrefs::layout_css_scroll_anchoring_enabled()) { + return; + } Frame()->PresShell()->PostPendingScrollAnchorSelection(this); } void ScrollAnchorContainer::Destroy() { if (mAnchorNode) { SetAnchorFlags(mScrollFrame->mScrolledFrame, mAnchorNode, false); } mAnchorNode = nullptr;