Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 6caa429b36c573bab41d355f81ee2ecfdd9e0aa2
      
https://github.com/WebKit/WebKit/commit/6caa429b36c573bab41d355f81ee2ecfdd9e0aa2
  Author: Vitor Roriz <vitor.ro...@apple.com>
  Date:   2023-12-05 (Tue, 05 Dec 2023)

  Changed paths:
    M 
LayoutTests/editing/caret/color-span-inside-editable-background-expected.html
    M LayoutTests/editing/caret/color-span-inside-editable-background.html
    M LayoutTests/editing/caret/color-span-inside-editable-expected.html
    M LayoutTests/editing/caret/color-span-inside-editable.html
    A 
LayoutTests/platform/gtk/editing/caret/color-span-inside-editable-background-expected.html
    A 
LayoutTests/platform/gtk/editing/caret/color-span-inside-editable-expected.html
    A 
LayoutTests/platform/mac-monterey/editing/caret/color-span-inside-editable-background-expected.html
    A 
LayoutTests/platform/mac-monterey/editing/caret/color-span-inside-editable-expected.html
    A 
LayoutTests/platform/mac-ventura/editing/caret/color-span-inside-editable-background-expected.html
    A 
LayoutTests/platform/mac-ventura/editing/caret/color-span-inside-editable-expected.html
    R LayoutTests/platform/mac-wk2/editing/caret/caret-color-expected.png
    R 
LayoutTests/platform/mac-wk2/editing/caret/color-span-inside-editable-background-expected.html
    R 
LayoutTests/platform/mac-wk2/editing/caret/color-span-inside-editable-expected.html
    M LayoutTests/platform/mac/TestExpectations

  Log Message:
  -----------
  REGRESSION: Two editing/caret/color-span-inside-editable tests are constant 
image failures
https://bugs.webkit.org/show_bug.cgi?id=265871
rdar://113875623

Reviewed by Brent Fulgham.

There were 2 caret-related changes:
A) Caret shape changed between Ventura and Sonoma (from squared to rounded).
B) Caret color behavior changed. The caret color should now update in content
editable elements if a foreground color is specified. This change was 
introduced by https://commits.webkit.org/266070@main.

Because of (A) we need to rebaseline these tests for the expected values
to match the shape of the updated caret. This change is only valid from Sonoma,
so we are adding -expected values for Ventura and Monterey, that expect
the caret to still be squared.

Because of (B) we need to rebaseline these tests for the expected values
to match the color of the caret in content editable elements. We also need
to update the description of the test themselves to match the new behavior, 
since
before the WebKit change on (B) caret would preserve its color even if set
by author.

* LayoutTests/editing/caret/color-span-inside-editable-background-expected.html:
* LayoutTests/editing/caret/color-span-inside-editable-background.html:
* LayoutTests/editing/caret/color-span-inside-editable-expected.html:
* LayoutTests/editing/caret/color-span-inside-editable.html:
* 
LayoutTests/platform/gtk/editing/caret/color-span-inside-editable-background-expected.html:
 Added.
* 
LayoutTests/platform/gtk/editing/caret/color-span-inside-editable-expected.html:
 Added.
* 
LayoutTests/platform/mac-monterey/editing/caret/color-span-inside-editable-background-expected.html:
 Added.
* 
LayoutTests/platform/mac-monterey/editing/caret/color-span-inside-editable-expected.html:
 Added.
* 
LayoutTests/platform/mac-ventura/editing/caret/color-span-inside-editable-background-expected.html:
 Added.
* 
LayoutTests/platform/mac-ventura/editing/caret/color-span-inside-editable-expected.html:
 Added.
* LayoutTests/platform/mac-wk2/editing/caret/caret-color-expected.png: Removed.
* 
LayoutTests/platform/mac-wk2/editing/caret/color-span-inside-editable-background-expected.html:
 Removed.
* 
LayoutTests/platform/mac-wk2/editing/caret/color-span-inside-editable-expected.html:
 Removed.
* LayoutTests/platform/mac/TestExpectations:

Canonical link: https://commits.webkit.org/271563@main


_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to