0

Use MainThreadScrollingReason::kNoScrollingLayer when no scrolling layer

This may help us find the real reason for regression of
"Non-fast scrollable regions" in CAP.

Bug: 1210209
Change-Id: Ia111e014a52e6c920fa3572842609b796cb4ac64
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2966728
Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#893102}
This commit is contained in:
Xianzhu Wang
2021-06-16 19:22:24 +00:00
committed by Chromium LUCI CQ
parent 2b097d0ef5
commit 948433c12d
2 changed files with 2 additions and 2 deletions

@ -1294,7 +1294,7 @@ InputHandler::ScrollStatus ThreadedInputHandler::TryScroll(
"LayerImpl::tryScroll: Failed due to no scrolling layer");
scroll_status.thread = InputHandler::ScrollThread::SCROLL_ON_MAIN_THREAD;
scroll_status.main_thread_scrolling_reasons =
MainThreadScrollingReason::kNonFastScrollableRegion;
MainThreadScrollingReason::kNoScrollingLayer;
return scroll_status;
}

@ -3252,7 +3252,7 @@ TEST_P(ScrollUnifiedLayerTreeHostImplTest, ScrollNodeWithoutScrollLayer) {
EXPECT_FALSE(status.needs_main_thread_hit_test);
} else {
EXPECT_EQ(ScrollThread::SCROLL_ON_MAIN_THREAD, status.thread);
EXPECT_EQ(MainThreadScrollingReason::kNonFastScrollableRegion,
EXPECT_EQ(MainThreadScrollingReason::kNoScrollingLayer,
status.main_thread_scrolling_reasons);
}
}