Attachment 368412 Details for Bug 197326 – Patch

[patch] Patch

bug-197326-20190427231353.patch (text/plain), 34.96 KB, created by Javier Fernandez on 2019-04-27 14:13:55 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Javier Fernandez

Created: 2019-04-27 14:13:55 PDT

Size: 34.96 KB

patch

obsolete

>Subversion Revision: 244694
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index 1d3c56aa7b287feec8b4030418684c4c377ebcc5..bf14d8e6a255fb2b4813408d54005e7a62e8d2ab 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,13 @@
>+2019-04-26  Javier Fernandez  <jfernandez@igalia.com>
>+
>+        Update the CSS Text WPT test suite
>+        https://bugs.webkit.org/show_bug.cgi?id=197326
>+
>+        Unreviewed test gardening.
>+
>+        * TestExpectations:
>+          - The break-spaces-008.html fails due to bug 197277
>+
> 2019-04-26  Said Abou-Hallawa  <sabouhallawa@apple.com>
> 
>         propertyRegistry() was not overridden for SVGFEFloodElement and SVGFEMergeElement
>diff --git a/LayoutTests/imported/w3c/ChangeLog b/LayoutTests/imported/w3c/ChangeLog
>index 22480e7daeb378cb7c4f38edc86a8f69c5178b5c..7490c6cc375222ab5426852d08fe2e73c1ed959e 100644
>--- a/LayoutTests/imported/w3c/ChangeLog
>+++ b/LayoutTests/imported/w3c/ChangeLog
>@@ -1,3 +1,38 @@
>+2019-04-26  Javier Fernandez  <jfernandez@igalia.com>
>+
>+        Update the CSS Text WPT test suite
>+        https://bugs.webkit.org/show_bug.cgi?id=197326
>+
>+        Unreviewed test gardening.
>+
>+        * web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-break-word-007.html:
>+        * web-platform-tests/css/css-text/overflow-wrap/w3c-import.log:
>+        * web-platform-tests/css/css-text/parsing/text-transform-valid.html:
>+        * web-platform-tests/css/css-text/parsing/word-break-invalid-expected.txt:
>+        * web-platform-tests/css/css-text/parsing/word-break-invalid.html:
>+        * web-platform-tests/css/css-text/parsing/word-break-valid-expected.txt:
>+        * web-platform-tests/css/css-text/parsing/word-break-valid.html:
>+        * web-platform-tests/css/css-text/text-transform/w3c-import.log:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-003-expected.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-003.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-004.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-005-expected.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-005.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-006-expected.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-006.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-007-expected.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-007.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-008-expected.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-008.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-009-expected.html:
>+        * web-platform-tests/css/css-text/white-space/break-spaces-009.html:
>+        * web-platform-tests/css/css-text/white-space/pre-wrap-008-expected.html:
>+        * web-platform-tests/css/css-text/white-space/pre-wrap-008.html:
>+        * web-platform-tests/css/css-text/white-space/pre-wrap-015.html:
>+        * web-platform-tests/css/css-text/white-space/pre-wrap-016.html:
>+        * web-platform-tests/css/css-text/white-space/textarea-break-spaces-001-expected.html:
>+        * web-platform-tests/css/css-text/white-space/w3c-import.log:
>+
> 2019-04-26  Youenn Fablet  <youenn@apple.com>
> 
>         [Mac WK2 iOS Sim] Layout Test imported/w3c/web-platform-tests/webrtc/RTCRtpReceiver-getSynchronizationSources.https.html is a flaky failure
>diff --git a/LayoutTests/TestExpectations b/LayoutTests/TestExpectations
>index bdc45ae7888fb4d0ddc132317aaf91d56c850e10..23bba13d81333df0d1dd8d76f8334ea11d2e21e2 100644
>--- a/LayoutTests/TestExpectations
>+++ b/LayoutTests/TestExpectations
>@@ -1961,6 +1961,8 @@ webkit.org/b/195345 imported/w3c/web-platform-tests/css/css-text/overflow-wrap/o
> webkit.org/b/195345 imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-min-content-size-004.html [ Skip ]
> webkit.org/b/195345 imported/w3c/web-platform-tests/css/css-text/word-break/word-break-break-word-overflow-wrap-interactions.html [ Skip ]
> 
>+webkit.org/b/197277 imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-008.html [ ImageOnlyFailure ]
>+
> 
> ########################################
> ### START OF display: contents failures
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-break-word-007.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-break-word-007.html
>index 0491b921e1aed4b030ca5cf0214afad7bdf3aba1..e701f765da27b75b71b261878e217b7a4c5c593e 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-break-word-007.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-break-word-007.html
>@@ -3,15 +3,16 @@
> <title>CSS Text Test: overflow-wrap: break-word</title>
> <link rel="author" title="Javier Fernandez Garcia-Boente" href="mailto:jfernandez@igalia.com">
> <link rel="help" href="https://drafts.csswg.org/css-text-3/#valdef-overflow-wrap-break-word">
>-<link rel="help" href="https://drafts.csswg.org/css-text-3/#valdef-white-space-break-spaces">
>+<link rel="help" href="https://drafts.csswg.org/css-text-3/#valdef-white-space-pre-wrap">
> <meta name="flags" content="ahem">
> <link rel="match" href="reference/overflow-wrap-break-word-001-ref.html">
>-<meta name="assert" content="A Single leading white-space constitutes a soft breaking opportunity, honoring the 'white-space: break-spaces' property, that must prevent the word to be broken.">
>+<meta name="assert" content="A Single leading white-space constitutes a soft breaking opportunity, honoring the 'white-space: pre-wrap' property, that must prevent the word to be broken.">
> <style>
> div {
>-   position: relative;
>-   font-size: 20px;
>-   font-family: Ahem;
>+  position: relative;
>+  font-size: 20px;
>+  font-family: Ahem;
>+  line-height: 1em;
> }
> .red {
>   position: absolute;
>@@ -24,7 +25,6 @@ div {
> }
> .test {
>   color: green;
>-  line-height: 1em;
>   width: 5ch;
> 
>   white-space: pre-wrap;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/w3c-import.log b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/w3c-import.log
>index f3774926f75aa2ce640d9d68f810a8a350c2a7de..2ebb6b8735c5d84bbaf00d90238466cf2b7b5a58 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/w3c-import.log
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/w3c-import.log
>@@ -56,6 +56,10 @@ List of files:
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-break-word-span-001-expected.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-break-word-span-001.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-break-word-span-002.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-cluster-001-expected.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-cluster-001.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-cluster-002-expected.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-cluster-002.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-min-content-size-001-expected.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-min-content-size-001.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-min-content-size-002-expected.html
>@@ -64,6 +68,8 @@ List of files:
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-min-content-size-003.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-min-content-size-004-expected.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-min-content-size-004.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-normal-keep-all-001-expected.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-normal-keep-all-001.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/word-wrap-001-expected.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/word-wrap-001.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/overflow-wrap/word-wrap-002-expected.html
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/text-transform-valid.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/text-transform-valid.html
>index a40166c37ef69074b00de8662fe250dbd7877693..fead45c8b8e45da70d4c14d12e4eba38348aed41 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/text-transform-valid.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/text-transform-valid.html
>@@ -22,16 +22,20 @@ test_valid_value("text-transform", "full-size-kana");
> test_valid_value("text-transform", "capitalize full-width");
> test_valid_value("text-transform", "uppercase full-size-kana");
> test_valid_value("text-transform", "full-width full-size-kana");
>-test_valid_value("text-transform", "full-width lowercase");
>-test_valid_value("text-transform", "full-size-kana capitalize");
>-test_valid_value("text-transform", "full-size-kana full-width");
>+
>+// serialization canonicalizes the order of values: https://drafts.csswg.org/cssom/#serialize-a-css-value
>+test_valid_value("text-transform", "full-width lowercase", "lowercase full-width");
>+test_valid_value("text-transform", "full-size-kana capitalize", "capitalize full-size-kana");
>+test_valid_value("text-transform", "full-size-kana full-width", "full-width full-size-kana");
> 
> test_valid_value("text-transform", "capitalize full-width full-size-kana");
>-test_valid_value("text-transform", "full-width full-size-kana uppercase");
>-test_valid_value("text-transform", "full-size-kana lowercase full-width");
>-test_valid_value("text-transform", "lowercase full-size-kana full-width");
>-test_valid_value("text-transform", "full-width uppercase full-size-kana");
>-test_valid_value("text-transform", "full-size-kana full-width capitalize");
>+
>+// serialization canonicalizes the order of values
>+test_valid_value("text-transform", "full-width full-size-kana uppercase", "uppercase full-width full-size-kana");
>+test_valid_value("text-transform", "full-size-kana lowercase full-width", "lowercase full-width full-size-kana");
>+test_valid_value("text-transform", "lowercase full-size-kana full-width", "lowercase full-width full-size-kana");
>+test_valid_value("text-transform", "full-width uppercase full-size-kana", "uppercase full-width full-size-kana");
>+test_valid_value("text-transform", "full-size-kana full-width capitalize", "capitalize full-width full-size-kana");
> </script>
> </body>
> </html>
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-invalid-expected.txt b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-invalid-expected.txt
>index f6de160a2bbf7ffe1877a582e41195220ebd5571..5a6c0708ba41739014c633541a589e514cf708b3 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-invalid-expected.txt
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-invalid-expected.txt
>@@ -2,4 +2,9 @@
> PASS e.style['word-break'] = "auto" should not set the property value 
> PASS e.style['word-break'] = "normal keep-all" should not set the property value 
> PASS e.style['word-break'] = "break-all break-all" should not set the property value 
>+PASS e.style['word-break'] = "normal break-word" should not set the property value 
>+PASS e.style['word-break'] = "break-word normal" should not set the property value 
>+PASS e.style['word-break'] = "keep-all break-word" should not set the property value 
>+PASS e.style['word-break'] = "break-all break-word" should not set the property value 
>+PASS e.style['word-break'] = "word-break" should not set the property value 
> 
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-invalid.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-invalid.html
>index 6d544de8bfa2084213ecd46d5cf189a0fc4966b2..46456b677d8d658cf285a005c7b72d10174d450d 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-invalid.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-invalid.html
>@@ -4,7 +4,7 @@
> <meta charset="utf-8">
> <title>CSS Text Module Test: parsing word-break with invalid values</title>
> <link rel="help" href="https://www.w3.org/TR/css-text-3/#propdef-word-break">
>-<meta name="assert" content="word-break supports only the grammar 'normal | keep-all | break-all'.">
>+<meta name="assert" content="word-break supports only the grammar 'normal | keep-all | break-all | break-word'.">
> <script src="/resources/testharness.js"></script>
> <script src="/resources/testharnessreport.js"></script>
> <script src="/css/support/parsing-testcommon.js"></script>
>@@ -14,6 +14,11 @@
> test_invalid_value("word-break", "auto");
> test_invalid_value("word-break", "normal keep-all");
> test_invalid_value("word-break", "break-all break-all");
>+test_invalid_value("word-break", "normal break-word");
>+test_invalid_value("word-break", "break-word normal");
>+test_invalid_value("word-break", "keep-all break-word");
>+test_invalid_value("word-break", "break-all break-word");
>+test_invalid_value("word-break", "word-break");
> </script>
> </body>
> </html>
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-valid-expected.txt b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-valid-expected.txt
>index 5ce6ef810182c79d9ca1237ab0b2656d7bbb4818..13261f01d6475ba932118d65d8aa641c98633870 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-valid-expected.txt
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-valid-expected.txt
>@@ -2,4 +2,5 @@
> PASS e.style['word-break'] = "normal" should set the property value 
> PASS e.style['word-break'] = "keep-all" should set the property value 
> PASS e.style['word-break'] = "break-all" should set the property value 
>+PASS e.style['word-break'] = "break-word" should set the property value 
> 
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-valid.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-valid.html
>index 9cd917b488a0e1f62103ced9e886d9584b134f4b..7f558848fb8bb8a0327c7dd5b6efca9e823dda1f 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-valid.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/parsing/word-break-valid.html
>@@ -4,7 +4,7 @@
> <meta charset="utf-8">
> <title>CSS Text Module Test: parsing word-break with valid values</title>
> <link rel="help" href="https://www.w3.org/TR/css-text-3/#propdef-word-break">
>-<meta name="assert" content="word-break supports the full grammar 'normal | keep-all | break-all'.">
>+<meta name="assert" content="word-break supports the full grammar 'normal | keep-all | break-all | break-word'.">
> <script src="/resources/testharness.js"></script>
> <script src="/resources/testharnessreport.js"></script>
> <script src="/css/support/parsing-testcommon.js"></script>
>@@ -14,6 +14,7 @@
> test_valid_value("word-break", "normal");
> test_valid_value("word-break", "keep-all");
> test_valid_value("word-break", "break-all");
>+test_valid_value("word-break", "break-word");
> </script>
> </body>
> </html>
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/w3c-import.log b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/w3c-import.log
>index 85485df6f3047be05fa57c56a7448995ea44670b..62fff478e2b8ac7259de4afa747a799af4a7701f 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/w3c-import.log
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/w3c-import.log
>@@ -78,6 +78,8 @@ List of files:
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/text-transform-lowercase-001.xht
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/text-transform-lowercase-002-expected.xht
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/text-transform-lowercase-002.xht
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/text-transform-multiple-001-expected.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/text-transform-multiple-001.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/text-transform-none-001-expected.xht
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/text-transform-none-001.xht
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/text-transform/text-transform-shaping-001-expected.html
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-003-expected.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-003-expected.html
>index 3247ae5a881ee513ae0b26821133b702fd473993..dece5f7394470d5bbc393c4318fa412ea25f9b4e 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-003-expected.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-003-expected.html
>@@ -4,7 +4,6 @@
> <link rel="author" title="Javier Fernandez" href="mailto:jfernandez@igalia.com" />
> <style>
> div {
>-  position: relative;
>   width: 100px;
>   height: 100px;
>   background: green;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-003.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-003.html
>index 000388bef5ad9335cc3f22e23fde26f512ede84c..c7a8693fef1978929fccedce8698c008a1dd4367 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-003.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-003.html
>@@ -7,11 +7,12 @@
> <link rel="help" href="https://drafts.csswg.org/css-text-3/#valdef-white-space-break-spaces">
> <link rel="help" href="https://drafts.csswg.org/css-text-3/#valdef-word-break-break-word">
> <meta name="flags" content="ahem">
>-<link rel="match" href="reference/pre-wrap-001-ref.html">
>-<meta name="assert" content="The word is not broken if there are previous breaking opportunities, honoring the 'white-space: break-spaces' value.">
>+<link rel="match" href="reference/white-space-break-spaces-005-ref.html">
>+<meta name="assert" content="break-word + break-spaces do not allow a break
>+between the last character of a word and the first space of a sequence of preserved spaces
>+if there are other wrapping opportunities earlier in the line.">
> <style>
> div {
>-   position: relative;
>    font: 25px/1 Ahem;
> }
> .fail {
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-004.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-004.html
>index 13770b7b8d189a404d44e73dfe19eba1c0e48b58..75b9a0da4e678a32b771a952b1988d77c1f0a19d 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-004.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-004.html
>@@ -11,7 +11,6 @@
> <meta name="assert" content="The word is not broken if there are previous breaking opportunities, honoring the 'white-space: break-spaces' value.">
> <style>
> div {
>-  position: relative;
>   font: 20px/1 Ahem;
> }
> .fail {
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-005-expected.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-005-expected.html
>index 3247ae5a881ee513ae0b26821133b702fd473993..dece5f7394470d5bbc393c4318fa412ea25f9b4e 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-005-expected.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-005-expected.html
>@@ -4,7 +4,6 @@
> <link rel="author" title="Javier Fernandez" href="mailto:jfernandez@igalia.com" />
> <style>
> div {
>-  position: relative;
>   width: 100px;
>   height: 100px;
>   background: green;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-005.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-005.html
>index c7f996fbc7bfc112cb605619736759ca20d65ddb..89bf9cbd97dd205ba2b0069cdb435afe74891d99 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-005.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-005.html
>@@ -9,7 +9,6 @@
> <meta name="assert" content="If 'white-space' is set to 'break-spaces', collapsing preserved white-spaces' advance width is not allowed, so that they can be wrapped honoring the 'white-space' propery.">
> <style>
> div {
>-  position: relative;
>   font: 10px/1 Ahem;
> }
> .fail {
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-006-expected.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-006-expected.html
>index 3247ae5a881ee513ae0b26821133b702fd473993..dece5f7394470d5bbc393c4318fa412ea25f9b4e 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-006-expected.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-006-expected.html
>@@ -4,7 +4,6 @@
> <link rel="author" title="Javier Fernandez" href="mailto:jfernandez@igalia.com" />
> <style>
> div {
>-  position: relative;
>   width: 100px;
>   height: 100px;
>   background: green;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-006.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-006.html
>index f3973169303e00e4853aa70b6bf9f8273f50bf33..b8d12a18440c8926ca2a28bf2339a97938a8b3da 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-006.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-006.html
>@@ -11,7 +11,6 @@
> <meta name="assert" content="White spaces are preserved, honoring the 'white-space: break-spaces', but the words are broken, honring the 'word-beak: break-all' even though there are previous breaking opportunities in the white-spaces.">
> <style>
> div {
>-  position: relative;
>   font: 25px/1 Ahem;
> }
> .fail {
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-007-expected.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-007-expected.html
>index 3247ae5a881ee513ae0b26821133b702fd473993..dece5f7394470d5bbc393c4318fa412ea25f9b4e 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-007-expected.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-007-expected.html
>@@ -4,7 +4,6 @@
> <link rel="author" title="Javier Fernandez" href="mailto:jfernandez@igalia.com" />
> <style>
> div {
>-  position: relative;
>   width: 100px;
>   height: 100px;
>   background: green;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-007.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-007.html
>index d14aae72f517377d28c1828531569a760c1388b7..c7242c27e5ca3e36e87725a05de99345333d8038 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-007.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-007.html
>@@ -8,10 +8,9 @@
> <link rel="help" href="https://drafts.csswg.org/css-text-3/#valdef-word-break-break-all">
> <link rel="match" href="reference/white-space-break-spaces-005-ref.html">
> <meta name="flags" content="ahem">
>-<meta name="assert" content="A single leading white-space should be used, honoring white-space: break-spaces, to avoid overflow; however, a single preserved white-space at the end of the line cannot be wrapped, hence it hangs when breaking after it to move the rest of the text to the next line.">
>+<meta name="assert" content="A single leading white-space should be used, honoring white-space: break-spaces, to avoid overflow; however, a single preserved white-space at the end of the line cannot be wrapped, hence it overflows when breaking after it to move the rest of the text to the next line.">
> <style>
> div {
>-  position: relative;
>   font: 25px/1 Ahem;
> }
> .fail {
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-008-expected.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-008-expected.html
>index 3247ae5a881ee513ae0b26821133b702fd473993..dece5f7394470d5bbc393c4318fa412ea25f9b4e 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-008-expected.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-008-expected.html
>@@ -4,7 +4,6 @@
> <link rel="author" title="Javier Fernandez" href="mailto:jfernandez@igalia.com" />
> <style>
> div {
>-  position: relative;
>   width: 100px;
>   height: 100px;
>   background: green;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-008.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-008.html
>index 7bcb9dd33758baf715060aa288d9a091e9cb0c94..9fdcdb6d2d856dc16212c4159f469652f0579aaf 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-008.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-008.html
>@@ -8,10 +8,9 @@
> <link rel="help" href="https://drafts.csswg.org/css-text-3/#valdef-word-break-break-all">
> <link rel="match" href="reference/white-space-break-spaces-005-ref.html">
> <meta name="flags" content="ahem">
>-<meta name="assert" content="White spaces are preserved, honoring the 'white-space: break-spaces', which may lead to overfow. However, we can break before the first white-space after the word honoring the 'break-all' value.">
>+<meta name="assert" content="White spaces are preserved, honoring the 'white-space: break-spaces', which may lead to overfow. However, we can break before the las letter in the word honoring the 'break-all' value.">
> <style>
> div {
>-  position: relative;
>   font: 25px/1 Ahem;
> }
> .fail {
>@@ -30,6 +29,6 @@ span { color: green; }
> </style>
> <body>
>   <p>Test passes if there is a <strong>filled green square</strong> and <strong>no red</strong>.</p>
>-  <div class="fail">XXXX<br><span>X</span>XX<span>X</span><br><span>XXXX</span><br><span>XXXX</span></div>
>+  <div class="fail">XXX<span>X</span><br>X<span>X</span>XX<br><span>XXXX</span><br><span>XXXX</span></div>
>   <div class="test">XXXX XX</div>
> </body>
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-009-expected.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-009-expected.html
>index 3247ae5a881ee513ae0b26821133b702fd473993..dece5f7394470d5bbc393c4318fa412ea25f9b4e 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-009-expected.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-009-expected.html
>@@ -4,7 +4,6 @@
> <link rel="author" title="Javier Fernandez" href="mailto:jfernandez@igalia.com" />
> <style>
> div {
>-  position: relative;
>   width: 100px;
>   height: 100px;
>   background: green;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-009.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-009.html
>index 9ecaa919ddfd42c920523e123afbeec232bdc920..36aa9dafd6e588c292f598179465233be653ab54 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-009.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-009.html
>@@ -13,7 +13,6 @@ between the last character of a word and the first space of a sequence of preser
> if there are no other wrapping opportunities earlier in the line">
> <style>
> div {
>-  position: relative;
>   font: 25px/1 Ahem;
> }
> .red {
>@@ -27,7 +26,6 @@ div {
>   background: green;
>   color: red;
>   width: 4ch;
>-  z-index: -1;
> 
>   white-space: break-spaces;
>   word-break: break-word;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-008-expected.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-008-expected.html
>index 3247ae5a881ee513ae0b26821133b702fd473993..dece5f7394470d5bbc393c4318fa412ea25f9b4e 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-008-expected.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-008-expected.html
>@@ -4,7 +4,6 @@
> <link rel="author" title="Javier Fernandez" href="mailto:jfernandez@igalia.com" />
> <style>
> div {
>-  position: relative;
>   width: 100px;
>   height: 100px;
>   background: green;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-008.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-008.html
>index 1ddbb7fe356e2cd9af2a35aa848541db2ab8fe23..7e4a24993f14c8e2cbf0e62c474967c61e9ade55 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-008.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-008.html
>@@ -9,7 +9,6 @@
> <meta name="assert" content="The word is not broken despite the 'word-break: break-word' if there are previous breaking opportunities, honoring the white-space: pre-wrap value.">
> <style>
> div {
>-   position: relative;
>    font-size: 20px;
>    font-family: Ahem;
> }
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-015.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-015.html
>index 176259b72ed7e5807b318d848ce1d2b7c9657873..d3f51bc429dad96a6f3a100e5f8296c0b3ab4405 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-015.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-015.html
>@@ -9,7 +9,6 @@
> <meta name="assert" content="The text is broken at the end of the space between the two words, never before, so it hangs and cause an overflow">
> <style>
> div {
>-  position: relative;
>   font: 20px/1 Ahem;
> }
> .ref {
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-016.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-016.html
>index e88c3eda1990aae01756d40d1b4d1e5e8be77e41..18dd834d813dce07092ee8d178fe9fd850126be0 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-016.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/pre-wrap-016.html
>@@ -9,7 +9,6 @@
> <meta name="assert" content="The word is not broken if there are previous breaking opportunities, honoring the white-space: pre-wrap value.">
> <style>
> div {
>-   position: relative;
>    font-size: 20px;
>    font-family: Ahem;
>    line-height: 1em;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/textarea-break-spaces-001-expected.html b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/textarea-break-spaces-001-expected.html
>index 61bc0420488fbf2aaf4271113eca14c950650684..8d3901f3658e01fc8d24171bca5170564e864526 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/textarea-break-spaces-001-expected.html
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/textarea-break-spaces-001-expected.html
>@@ -14,7 +14,6 @@ textarea {
>   word-break: initial;
>   margin: 0;
>   padding: 0;
>-  border-radius: 0px;
>   border: none;
>   outline: none;
>   resize: none;
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/w3c-import.log b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/w3c-import.log
>index a811dbca6fb508b689afddb12b947f780073249f..8d297271228e8fa505613193ae0c36030627e874 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/w3c-import.log
>+++ b/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/w3c-import.log
>@@ -18,6 +18,8 @@ List of files:
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-001.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-002-expected.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-002.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-003-expected.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-003.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-004-expected.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-004.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/break-spaces-005-expected.html
>@@ -50,6 +52,8 @@ List of files:
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/control-chars-008.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/control-chars-00B-expected.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/control-chars-00B.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/control-chars-00C-expected.html
>+/LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/control-chars-00C.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/control-chars-00D-expected.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/control-chars-00D.html
> /LayoutTests/imported/w3c/web-platform-tests/css/css-text/white-space/control-chars-00E-expected.html
>diff --git a/LayoutTests/platform/ios/TestExpectations b/LayoutTests/platform/ios/TestExpectations
>index b5ed869d5c67ba736255ca83a05114fc38365d49..cb9c7fdbf2357ef7092602b01f8cd8282d9dc443 100644
>--- a/LayoutTests/platform/ios/TestExpectations
>+++ b/LayoutTests/platform/ios/TestExpectations
>@@ -3071,6 +3071,8 @@ webkit.org/b/183258 imported/w3c/web-platform-tests/css/css-text/i18n/css3-text-
> webkit.org/b/183258 imported/w3c/web-platform-tests/css/css-text/word-break/word-break-break-all-006.html [ ImageOnlyFailure ]
> webkit.org/b/196169 imported/w3c/web-platform-tests/css/css-text/overflow-wrap/overflow-wrap-break-word-003.html [ ImageOnlyFailure ]
> 
>+imported/w3c/web-platform-tests/css/css-text/white-space/textarea-break-spaces-001.html [ ImageOnlyFailure ]
>+
> # unsupported
> fast/dynamic/mail-autosize-viewport-unit.html [ Failure ]
>

Actions: View | Formatted Diff | Diff

Attachments on bug 197326: 368350 | 368387 | 368412