[M140] [carousel] Fix scroll marker group data not working with inline targets [chromium/src : refs/branch-heads/7339]

0 views

Skip to first unread message

Chromium LUCI CQ (Gerrit)

unread,

Sep 15, 2025, 2:21:28 PM (4 days ago) Sep 15

to Daniil Sakhapov, Chrome Cherry Picker, AyeAye, Rubber Stamper, blink-re...@chromium.org, blink-revie...@chromium.org, blink-rev...@chromium.org, blink-...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:

[M140] [carousel] Fix scroll marker group data not working with inline targets

Original change's description:
> [carousel] Fix scroll marker group data not working with inline targets
> 
> As those require layout to be done (lifecycle of the document should be
> kAfterPerformLayout), selecting the active scroll marker is moved to the
> snapshoting time.
> 
> Also, this CL changes LayoutBoxes to LayoutObjects in a few places to
> work with inline targets.
> 
> Fixed: 443283020
> Change-Id: I26438cf1156b2dca8cc69930371969a507a71101
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6925380
> Commit-Queue: Daniil Sakhapov <sakh...@chromium.org>
> Reviewed-by: Robert Flack <fla...@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1514809}

(cherry picked from commit 75bfbaf0b923e19e6e0341c2248e158d4c26f90e)

Bug: 444672660,443283020

Change-Id: I26438cf1156b2dca8cc69930371969a507a71101

Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6950615

Bot-Commit: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>

Auto-Submit: Chrome Cherry Picker <chrome-che...@chops-service-accounts.iam.gserviceaccount.com>

Commit-Queue: Rubber Stamper <rubber-...@appspot.gserviceaccount.com>

Cr-Commit-Position: refs/branch-heads/7339@{#1924}

Cr-Branched-From: 27be8b77710f4405fdfeb4ee946fcabb0f6c92b2-refs/heads/main@{#1496484}

Files:

M third_party/blink/renderer/core/dom/scroll_marker_group_data.cc
M third_party/blink/renderer/core/dom/scroll_marker_group_data.h
M third_party/blink/renderer/core/html/html_anchor_element.cc
A third_party/blink/web_tests/external/wpt/css/css-overflow/scroll-target-group-inline-targets-ref.html
A third_party/blink/web_tests/external/wpt/css/css-overflow/scroll-target-group-inline-targets.html

Change size: M

Delta: 5 files changed, 106 insertions(+), 26 deletions(-)

Branch: refs/branch-heads/7339

Submit Requirements:

Code-Review: Bot-Commit+1 by Rubber Stamper

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

open

diffy

satisfied_requirement

Reply all

Reply to author

Forward

0 new messages