Bug 218220 - [GTK] Layout test editing/execCommand/switch-list-type-with-orphaned-li.html fails when WebKit is built with GCC
Summary: [GTK] Layout test editing/execCommand/switch-list-type-with-orphaned-li.html ...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-10-26 20:30 PDT by Carlos Alberto Lopez Perez
Modified: 2021-06-09 00:39 PDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2020-10-26 20:30:03 PDT 
Layout test editing/execCommand/switch-list-type-with-orphaned-li.html fails on GTK when WebKit is built with GCC (GCC 10.2.0 from flatpak env)

However the test passes fine when WebKit is built with Clang (Clang 10.0.1 from flatpak env).

The text diff for the failure (GCC) is the one below:

Regressions: Unexpected text-only failures (1)
  editing/execCommand/switch-list-type-with-orphaned-li.html [ Failure ]

------------------------------------------------------------------------------
--- /home/igalia/clopez/webkit/webkit-flatpak/layout-test-results/editing/execCommand/switch-list-type-with-orphaned-li-expected.txt
+++ /home/igalia/clopez/webkit/webkit-flatpak/layout-test-results/editing/execCommand/switch-list-type-with-orphaned-li-actual.txt
@@ -10,6 +10,13 @@
     "
 |   <li>
 |     "because of"
+| "
+"
+| "
+"
+| "
+"
+| <ol>
 |   "
     "
 |   <li>
@@ -18,9 +25,3 @@
 "
 | "
 "
-| "
-"
-| "
-"
-| "
-"
Comment 1 Carlos Alberto Lopez Perez 2020-10-26 20:33:41 PDT 
This is likely also an issue for WPE. Didn't tested.
Comment 2 Carlos Alberto Lopez Perez 2020-10-26 21:40:02 PDT 
Updated expectations in r269021
Comment 3 Diego Pino 2021-06-09 00:16:54 PDT 
The only test filed under this bug was:

  - editing/execCommand/switch-list-type-with-orphaned-li.html

GTK post-commit test bot reports the test has been failing consistently for the last 4000 revisions.

Diff:

--- /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/editing/execCommand/switch-list-type-with-orphaned-li-expected.txt
+++ /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/editing/execCommand/switch-list-type-with-orphaned-li-actual.txt
@@ -8,11 +8,12 @@
 |   "\n    "
 |   <li>
 |     "because of"
+| "\n"
+| "\n"
+| "\n"
+| <ol>
 |   "\n    "
 |   <li>
 |     "you<#selection-focus>"
 |   "\n"
 | "\n"
-| "\n"
-| "\n"
-| "\n"
Comment 4 Diego Pino 2021-06-09 00:39:55 PDT 
Sorry for the noise, this test should remain flakey as it passes with Clang but fails with GCC.