Bug 1771439 [wpt PR 34229] - Revert "Reland "CSS highlight painting: bump HighlightOv...
commit9fdf28dc64191cefe8a91e54a6068497d4f1b98e
authorEliot Courtney <edcourtney@google.com>
Tue, 31 May 2022 04:26:28 +0000 (31 04:26 +0000)
committermoz-wptsync-bot <wptsync@mozilla.com>
Thu, 9 Jun 2022 08:27:05 +0000 (9 08:27 +0000)
treea41001c511ac24700558d05264c132ef03df88b0
parent388b0c363c45dd676f837efa26869e6d2c7b7736
Bug 1771439 [wpt PR 34229] - Revert "Reland "CSS highlight painting: bump HighlightOverlayPainting to stable"", a=testonly

Automatic update from web-platform-tests
Revert "Reland "CSS highlight painting: bump HighlightOverlayPainting to stable""

This reverts commit 387b61732a4ca2f25d7aa5b376de0d373e4e72c2.

Reason for revert: caused a performance regression in input latency

Original change's description:
> Reland "CSS highlight painting: bump HighlightOverlayPainting to stable"
>
> This is a reland of commit abc6f616c7bc2c53cc9d8c44b82384d43dd77d02
>
> Original change's description:
> > CSS highlight painting: bump HighlightOverlayPainting to stable
> >
> > This patch enables the new highlight overlay painter. The rebaselined
> > tests and their reasons are as follows:
> >
> > • no more double painting bug
> >     • dark-mode/colors/selection.html
> > • originating decorations are now recolored per spec
> >     • editing/selection/line-wrap-2.html
> >     • virtual/text-antialias/selection/khmer-selection.html
> > • spelling markers are now *consistently* painted over text
> >     • paint/markers/document-markers-font-8px.html
> >     • paint/markers/document-markers-zoom-2000.html
> > • decorations are now clipped to highlight background rect
> >     • svg/text/text-selection-deco-01-b.svg
> >     • svg/text/text-selection-text-03-b.svg
> >
> > The only new failure expectations are as follows:
> >
> > • external/wpt/css/css-pseudo/highlight-painting-003.html broadens
> >   from Mac11 to Mac (like we had to do in CL:3608432)
> > • external/wpt/css/css-text-decor/text-decoration-color-selection-001
> >   fails because the test is incorrect (bug 1321106)
> > • paint/markers/suggestion-marker-basic.html fails because it’s not
> >   clear the assertion still complies with the spec (bug 1321114)
> >
> > Some tests had bugs that we’ve fixed in this patch:
> >
> > • external/wpt/css/css-pseudo/target-text-text-decoration-001.html
> >   had a bug where it assumed UA default highlight colors would be
> >   suppressed by an author ‘text-decoration’
> >
> > Bug: 1147859
> > Change-Id: Ie4d8dae515a786fa01e8f2852d645f3a8ad313be
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3616170
> > Reviewed-by: Stephen Chenney <schenney@chromium.org>
> > Commit-Queue: Delan Azabani <dazabani@igalia.com>
> > Reviewed-by: Koji Ishii <kojii@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1001604}
>
> Bug: 1147859
> Change-Id: Icd150ff8c695d20a5c9f44df45d5e0751c31f79a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3640642
> Reviewed-by: Koji Ishii <kojii@chromium.org>
> Commit-Queue: Delan Azabani <dazabani@igalia.com>
> Cr-Commit-Position: refs/heads/main@{#1001937}

Bug: 11478591329434
Change-Id: I73ff7b6bad3187f4227b5ca8f20053b02b08438f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3671846
Auto-Submit: Eliot Courtney <edcourtney@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1008448}

--

wpt-commits: 83dc9e3914a98d5e923463c054db286fbf368bb3
wpt-pr: 34229
testing/web-platform/tests/css/css-pseudo/target-text-text-decoration-001-ref.html
testing/web-platform/tests/css/css-pseudo/target-text-text-decoration-001.html