commit 68d7dee11c9b6217d27521cbe3ad66e8437402a2
author Philip Rogers <pdr@chromium.org> Fri Jan 17 02:18:39 2025
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Jan 17 02:18:39 2025
tree 149370cc0961df59790ced0b2e06e8c25cd737ba
parent 3e82427cc12f90f01444e1dcb65f48740e62596f

Cleanup PaintLayer::has_visible_content_

This patch cleans up PaintLayer::has_visible_content_ for future
optimizations. There should be no change in behavior. The changes are:
1. Move has_visible_content_ logic into a new function:
  `PaintLayer::UpdateHasVisibleContent`.
2. Move update of `has_visible_content_` into the
  `needs_descendant_dependent_flags_update_` conditional of
  `PaintLayer::UpdateDescendantDependentFlags`. Though this field does
  not depend on PaintLayer descendants, it is used to update
  `has_visible_self_painting_descendant_`.
3. Check `child->HasVisibleContent` instead of
  `child->has_visible_content_`, as the former has a CHECK that the
  dirty bit is clean.

Change-Id: I08f74ccc49956a48eaad94dec401865934b754eb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6173294
Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1407688}