Refactor WebViewImpl::BestTapNode
This patch refactors |WebViewImpl::BestTapNode| to export the
parts of the logic, so that we can use it in the culled inline
code.
|ShowsHandCursor| and part of |FindCursorDefiningAncestor| are
merged into |Node::GetHandCursorType|. It avoids calling
|EventHandler::UsesHandCursor| twice, as it's an expensive
function that traverses the tree up to root.
There should be no behavior changes.
Bug: 1196549
Change-Id: I5fddbef50c15df27b74e4ab42ad033b0c0c5eac6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2810538
Auto-Submit: Koji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#871244}
GitOrigin-RevId: 08d83affbd4dcfc8c470ddcf4806013ddd0ac908
5 files changed