Bug 217475

Summary:

[Mac] imported/w3c/web-platform-tests/css/css-pseudo/text-selection.html is failing

Product:

WebKit

Reporter:

Oriol Brufau <obrufau>

Component:

Tools / Tests

Assignee:

Oriol Brufau <obrufau>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

aakash_jain, hector_i_lopez, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

Bug Blocks:

167729

Attachments:

Description	Flags
Patch	none
Patch	none
Patch	none
Patch	none

Oriol Brufau

Reported 2020-10-08 07:26:36 PDT

imported/w3c/web-platform-tests/css/css-pseudo/text-selection.html was already flaky (bug 217161). But bug 167729 made it worse. Before r267940, the expectation for the test in all platforms was (1): FAIL Selection ending in ::before assert_equals: toString expected "hello" but got "" PASS Selection contained in ::before FAIL Selection ending in ::marker assert_equals: toString expected "hello" but got "" PASS Selection contained in ::marker FAIL Selection ending in ::before-marker assert_equals: toString expected "hello" but got "" PASS Selection contained in ::before-marker But in r267940 I added a different expectation for Mac (2): FAIL Selection ending in ::before assert_equals: toString expected "hello" but got "" PASS Selection contained in ::before FAIL Selection ending in ::marker assert_equals: toString expected "hello" but got "" PASS Selection contained in ::marker FAIL Selection ending in ::before-marker assert_equals: toString expected "hello" but got "" FAIL Selection contained in ::before-marker assert_equals: toString expected "" but got "h" But this made it fail in Mac Catalina, which was producing (1). So r268171 changed the Mac expectation to match (1). But this made the test fail in Mac Mojave, which was expecting (2). Then, it seems: - LayoutTests/imported/w3c/web-platform-tests/css/css-pseudo/text-selection-expected.txt should keep having (1). - LayoutTests/platform/mac/imported/w3c/web-platform-tests/css/css-pseudo/text-selection-expected.txt should be removed. - LayoutTests/platform/mac-mojave/imported/w3c/web-platform-tests/css/css-pseudo/text-selection-expected.txt should be created with (2).

Attachments
Patch (2.60 KB, patch) 2020-10-08 07:36 PDT, Oriol Brufau	no flags	Details Formatted Diff Diff
Patch (3.48 KB, patch) 2020-10-08 09:42 PDT, Oriol Brufau	no flags	Details Formatted Diff Diff
Patch (2.54 KB, patch) 2020-10-09 18:53 PDT, Oriol Brufau	no flags	Details Formatted Diff Diff
Patch (2.79 KB, patch) 2020-10-09 19:05 PDT, Oriol Brufau	no flags	Details Formatted Diff Diff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.