merge_pr_46190

[css-anchor-position-1] Fix flicker with try-option with fixed-position anchor

Previously we supposed the default anchor wouldn't change in
try-options, but the latest draft spec allows that.

When the currently tried option will use a default anchor that is
different from the currently used default anchor, checking overflow
with the snapshotted AnchorPositionScrollData will be incorrect,
and the incorrect results caused oscillation between try options.

To resolve that, when the total offset is requested for a different
default anchor, we need to calculate it on the fly instead of getting
it from the snapshotted result.

Also allow one more layout if ValidateSnapshot() (called after the
first layout in a document lifecycle update) finds that the
snapshotted scroll data needs update, to ensure correct layout and
paint with the changed anchor and scroll adjustment in most cases.

In rare cases when we need more than one additional layout to get
the stable layout, ShouldScheduleNextService() will schedule another
lifecycle.

Fixed: 338929745
Change-Id: I295fbc605ac8b7f0298eef7aeff4434c6bd54628
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5529048
Reviewed-by: Ian Kilpatrick ikilpatrick@chromium.org
Reviewed-by: Philip Rogers pdr@chromium.org
Commit-Queue: Xianzhu Wang wangxianzhu@chromium.org
Cr-Commit-Position: refs/heads/main@{#1299608}

merge_pr_46120

Add a REFERENCE-IN-OTHER-TYPE lint

This is similar to the TESTHARNESS-IN-OTHER-TYPE lint

merge_pr_46212

Revert "Reland "Reland "Service worker web platform tests."""

This reverts commit 9bd67ed90e90d6a2c2134d26bb8bcc4cd9436b19.

Reason for revert: Failing in some cases.

Original change's description:

Reland "Reland "Service worker web platform tests.""

This reverts commit 93a580699ecf6d102a5853455b5faeceea4cd31e.

Reason for revert: Removed the test case that times out,
(the one that waits for the update url).

Original change's description:

Revert "Reland "Service worker web platform tests.""

This reverts commit 436f1b9f6153002fd41599d5d4ec8343cf930a43.

Reason for revert: There are still timeouts in some tests

Original change's description:

Reland "Service worker web platform tests."

This reverts commit 36a1c457dac0ecaa54c0553b28f67c0c61752494.

Reason for revert: The test that looks for 'update-url.py' takes too long and therefore will require this test file to be part of slowTests

Original change's description:

Revert "Service worker web platform tests."

This reverts commit 63858f1b1ed058d07e73cebb0a98214aa2cfc715.

Reason for revert: New test added; fails immediately on some bots.

See
https://ci.chromium.org/ui/p/chromium/builders/ci/mac11-arm64-rel-tests/38089/overview
for the first failures. (And consistently in subsequent builds)

Original change's description:

Service worker web platform tests.

This tests that when an auction is ran, service workers do not intercept requests with URLs that are meant to be private.

Bug: 293383734
Change-Id: I06858f7cae4794a35c045fb8dad30d6316a26ead
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5459094
Commit-Queue: Youssef Bourouphael ybourouphael@google.com
Reviewed-by: mmenke mmenke@chromium.org
Reviewed-by: Orr Bernstein orrb@google.com
Cr-Commit-Position: refs/heads/main@{#1294960}

Bug: 293383734
Change-Id: I3431f0e45e65767ff1529002d8a2d14657d0cb5a
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5506634
Auto-Submit: Ian Clelland iclelland@chromium.org
Owners-Override: Ian Clelland iclelland@chromium.org
Commit-Queue: Rubber Stamper rubber-stamper@appspot.gserviceaccount.com
Bot-Commit: Rubber Stamper rubber-stamper@appspot.gserviceaccount.com
Cr-Commit-Position: refs/heads/main@{#1295015}

Bug: 293383734
Change-Id: I762dd50282f99617796df64d73a27d149cac0a86
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5512083
Commit-Queue: Youssef Bourouphael ybourouphael@google.com
Bot-Commit: Rubber Stamper rubber-stamper@appspot.gserviceaccount.com
Reviewed-by: Orr Bernstein orrb@google.com
Reviewed-by: mmenke mmenke@chromium.org
Cr-Commit-Position: refs/heads/main@{#1296948}

Bug: 293383734
Change-Id: I9b06c0453656aaa448d0030960a48fc264bcf867
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5515623
Auto-Submit: Youssef Bourouphael ybourouphael@google.com
Commit-Queue: Rubber Stamper rubber-stamper@appspot.gserviceaccount.com
Commit-Queue: mmenke mmenke@chromium.org
Reviewed-by: mmenke mmenke@chromium.org
Bot-Commit: Rubber Stamper rubber-stamper@appspot.gserviceaccount.com
Cr-Commit-Position: refs/heads/main@{#1297048}

Bug: 293383734
Change-Id: I7ae169c2750432bb1cb9c1c340585dc27d8cd419
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5521778
Commit-Queue: Youssef Bourouphael ybourouphael@google.com
Bot-Commit: Rubber Stamper rubber-stamper@appspot.gserviceaccount.com
Reviewed-by: mmenke mmenke@chromium.org
Reviewed-by: Orr Bernstein orrb@google.com
Cr-Commit-Position: refs/heads/main@{#1298807}

Bug: 293383734
Change-Id: I182736b58e83dbe7fc2c5fc7e20a3e3f909772a2
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5526425
Bot-Commit: Rubber Stamper rubber-stamper@appspot.gserviceaccount.com
Commit-Queue: Youssef Bourouphael ybourouphael@google.com
Cr-Commit-Position: refs/heads/main@{#1298902}

merge_pr_46201

VT: Ensure view-transition-names are tree scoped.

This only changes vt name discovery to match the scope of the
originating element (document). The presence of vt names still causes
things like stacking context, which I think is correct.

R=khushalsagar@chromium.org

Bug: 339438836
Change-Id: I47d3d1d78ca4938f259841ea04ab4b0b472914c9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5531882
Reviewed-by: Khushal Sagar khushalsagar@chromium.org
Commit-Queue: Vladimir Levin vmpstr@chromium.org
Cr-Commit-Position: refs/heads/main@{#1299414}

merge_pr_46200

Fix crash during counters attachment

Counters stack can be empty when we do LeaveElement, instead of DCHECK,
check it in if and just leave function.

DCHECK was hit when there is some li without ol/ul element at first that
create ListOrdinal, but after it in the tree there is another li that
is not sibling or descendant of previous li and doesn't have
ListOrdinal. In this case second li would remove first one during
RemoveStaleCounters, and later would try to remove itself during
RemoveCounterIfAncestorExists, assuming that it created the counter
itself.

Fixed: 339592908, 339641168
Change-Id: I09caa992256bba4e4844b3d0943f6d6c7047f6a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5526739
Reviewed-by: Anders Hartvoll Ruud andruud@chromium.org
Commit-Queue: Daniil Sakhapov sakhapov@chromium.org
Cr-Commit-Position: refs/heads/main@{#1299205}

merge_pr_46196

RubyLB: ruby-position should be valid for ::first-line

https://drafts.csswg.org/css-pseudo/#first-line-styling

• the ruby-position property (see [CSS-RUBY-1])

Bug: 324111880
Change-Id: Icc8901b29d0352b1cfa50238fca969884c5fa9a7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5526384
Commit-Queue: Koji Ishii kojii@chromium.org
Reviewed-by: Koji Ishii kojii@chromium.org
Auto-Submit: Kent Tamura tkent@chromium.org
Cr-Commit-Position: refs/heads/main@{#1299087}

merge_pr_46194

Mark popover-focus-2.html as a slow test

Chromium is failing due to a timeout:

https://wpt.fyi/results/html/semantics/popovers/popover-focus-2.html

with the test taking ~10 seconds. Let's leave it more time.

Fixed: 40287045
Change-Id: I2bbcfb8f691cb9992cdba2e1a180c0515e1ef053
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5530626
Reviewed-by: Joey Arhar jarhar@chromium.org
Commit-Queue: Joey Arhar jarhar@chromium.org
Auto-Submit: Mason Freed masonf@chromium.org
Cr-Commit-Position: refs/heads/main@{#1299322}

merge_pr_46191

webcodecs: WPT test for unsupported formats in VF.allocationSize()

Bug: 40267195
Change-Id: I8178bea9a753e0cc1a2c0092760ec468ed04999f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5530313
Commit-Queue: Eugene Zemtsov eugene@chromium.org
Reviewed-by: Dale Curtis dalecurtis@chromium.org
Cr-Commit-Position: refs/heads/main@{#1298991}

merge_pr_46189

WebKit export: [view-transitions] Refactor timing of "update callback done" and related (#46189)

https://bugs.webkit.org/show_bug.cgi?id=273969

merge_pr_46171

add test for shadowroots falling back to global scope (#46171)