Attachment 341808 Details for Bug 186212 – Moves the tests

[patch] Moves the tests

bug-186212-20180601170006.patch (text/plain), 17.72 KB, created by Ryosuke Niwa on 2018-06-01 17:00:06 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Ryosuke Niwa

Created: 2018-06-01 17:00:06 PDT

Size: 17.72 KB

patch

obsolete

>Index: LayoutTests/ChangeLog
>===================================================================
>--- LayoutTests/ChangeLog	(revision 232428)
>+++ LayoutTests/ChangeLog	(working copy)
>@@ -1,3 +1,23 @@
>+2018-06-01  Ryosuke Niwa  <rniwa@webkit.org>
>+
>+        Move tests in LayoutTests/editing to appropriate subdirectories
>+        https://bugs.webkit.org/show_bug.cgi?id=186212
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Moved the following four tests in LayoutTests/editing to its subdirectories.
>+
>+        * editing/input/composition-underline-color-expected-mismatch.html: Moved from editing/composition-underline-color-expected-mismatch.html.
>+        * editing/input/composition-underline-color.html: Move from editing/composition-underline-color.html.
>+        * editing/input/marked-text-appearance-expected-mismatch.html: Copied from LayoutTests/editing/marked-text-appearance-expected-mismatch.html.
>+        * editing/input/marked-text-appearance.html: Moved from editing/marked-text-appearance.html.
>+        * editing/selection/rtl-to-ltr-editing-word-move-spin-expected.txt: Moved from editing/rtl-to-ltr-editing-word-move-spin-expected.txt.
>+        * editing/selection/rtl-to-ltr-editing-word-move-spin.html: Moved from editing/rtl-to-ltr-editing-word-move-spin.html.
>+        * editing/selection/simple-line-layout-caret-is-gone-expected.txt: Moved from editing/simple-line-layout-caret-is-gone-expected.txt.
>+        * editing/selection/simple-line-layout-caret-is-gone.html: Moved from editing/simple-line-layout-caret-is-gone.html.
>+        * editing/style/apply-style-iframe-crash-expected.txt: Moved from editing/apply-style-iframe-crash-expected.txt.
>+        * editing/style/apply-style-iframe-crash.html: Moved from editing/apply-style-iframe-crash.html.
>+
> 2018-06-01  David Fenton  <david_fenton@apple.com>
> 
>         Skip fetch/api/redirect/redirect-method.html and fetch/api/redirect/redirect-method-worker.html on macOS debug.
>Index: LayoutTests/editing/apply-style-iframe-crash-expected.txt
>===================================================================
>--- LayoutTests/editing/apply-style-iframe-crash-expected.txt	(revision 232420)
>+++ LayoutTests/editing/apply-style-iframe-crash-expected.txt	(nonexistent)
>@@ -1,4 +0,0 @@
>-
>-
>-PASS
>-WebKit bug #132103: Crash applying editing commands from iframe onload event.
>Index: LayoutTests/editing/apply-style-iframe-crash.html
>===================================================================
>--- LayoutTests/editing/apply-style-iframe-crash.html	(revision 232420)
>+++ LayoutTests/editing/apply-style-iframe-crash.html	(nonexistent)
>@@ -1,36 +0,0 @@
>-<!DOCTYPE html>
>-<script>
>-    function frameLoaded() {
>-        if (document.counter)
>-            document.counter++;
>-        else
>-            document.counter = 1;
>-
>-        if (document.counter <= 16) {
>-            document.designMode='on';
>-            document.execCommand('selectall');
>-            document.execCommand('italic');
>-        }
>-    }
>-
>-    if (window.testRunner) {
>-        testRunner.dumpAsText();
>-        testRunner.waitUntilDone();
>-    }
>-
>-    window.addEventListener("load", function() {
>-        document.getElementById("console").innerText = 'PASS';
>-        if (window.testRunner)
>-            testRunner.notifyDone();
>-    });
>-</script>
>-
>-
>-<h1><button><iframe></iframe></button>
>-</h1><input><iframe onload="frameLoaded()"></iframe>
>-
>-<div id="console">FAIL</div>
>-
>-<div>
>-    WebKit bug #<a href="https://bugs.webkit.org/show_bug.cgi?id=132103">132103</a>: Crash applying editing commands from iframe onload event.
>-</div>
>Index: LayoutTests/editing/composition-underline-color-expected-mismatch.html
>===================================================================
>--- LayoutTests/editing/composition-underline-color-expected-mismatch.html	(revision 232420)
>+++ LayoutTests/editing/composition-underline-color-expected-mismatch.html	(nonexistent)
>@@ -1,33 +0,0 @@
>-<!DOCTYPE html>
>-<style>
>-  div {
>-    position: absolute;
>-    top: 0;
>-    left: 0;
>-  }
>-  #test {
>-    color: magenta;
>-    font: 20px/1 Monospace;
>-    outline: none;
>-  }
>-  #overlapping-top {
>-    background: white;
>-    width: 100px;
>-    height: 15px;
>-  }
>-  #overlapping-right {
>-    background: white;
>-    width: 50px;
>-    height: 50px;
>-    left: 50px;
>-  }
>-</style>
>-<div contenteditable id="test"></div>
>-
>-<div id="overlapping-top"></div>
>-<div id="overlapping-right"></div>
>-<script>
>-  document.getElementById("test").focus();
>-  if (window.textInputController)
>-    textInputController.setMarkedText("^^^^^", 5, 0);
>-</script>
>Index: LayoutTests/editing/composition-underline-color.html
>===================================================================
>--- LayoutTests/editing/composition-underline-color.html	(revision 232420)
>+++ LayoutTests/editing/composition-underline-color.html	(nonexistent)
>@@ -1,33 +0,0 @@
>-<!DOCTYPE html>
>-<style>
>-  div {
>-    position: absolute;
>-    top: 0;
>-    left: 0;
>-  }
>-  #test {
>-    color: lime;
>-    font: 20px/1 Monospace;
>-    outline: none;
>-  }
>-  #overlapping-top {
>-    background: white;
>-    width: 100px;
>-    height: 15px;
>-  }
>-  #overlapping-right {
>-    background: white;
>-    width: 50px;
>-    height: 50px;
>-    left: 50px;
>-  }
>-</style>
>-<div contenteditable id="test"></div>
>-
>-<div id="overlapping-top"></div>
>-<div id="overlapping-right"></div>
>-<script>
>-  document.getElementById("test").focus();
>-  if (window.textInputController)
>-    textInputController.setMarkedText("^^^^^", 5, 0);
>-</script>
>Index: LayoutTests/editing/marked-text-appearance-expected-mismatch.html
>===================================================================
>--- LayoutTests/editing/marked-text-appearance-expected-mismatch.html	(revision 232420)
>+++ LayoutTests/editing/marked-text-appearance-expected-mismatch.html	(nonexistent)
>@@ -1,9 +0,0 @@
>-<!DOCTYPE html>
>-<body contenteditable></body>
>-<script>
>-    document.body.focus();
>-    if (window.textInputController) {
>-        textInputController.setMarkedText("^^^^^", 0, 5, true);
>-        textInputController.insertText("^^^^^");
>-    }
>-</script>
>Index: LayoutTests/editing/marked-text-appearance.html
>===================================================================
>--- LayoutTests/editing/marked-text-appearance.html	(revision 232420)
>+++ LayoutTests/editing/marked-text-appearance.html	(nonexistent)
>@@ -1,7 +0,0 @@
>-<!DOCTYPE html>
>-<body contenteditable></body>
>-<script>
>-    document.body.focus();
>-    if (window.textInputController)
>-        textInputController.setMarkedText("^^^^^", 0, 5, true);
>-</script>
>Index: LayoutTests/editing/rtl-to-ltr-editing-word-move-spin-expected.txt
>===================================================================
>--- LayoutTests/editing/rtl-to-ltr-editing-word-move-spin-expected.txt	(revision 232420)
>+++ LayoutTests/editing/rtl-to-ltr-editing-word-move-spin-expected.txt	(nonexistent)
>@@ -1,4 +0,0 @@
>-Pass if no hang.
>-â« foobar foobarâ«
>-
>-
>Index: LayoutTests/editing/rtl-to-ltr-editing-word-move-spin.html
>===================================================================
>--- LayoutTests/editing/rtl-to-ltr-editing-word-move-spin.html	(revision 232420)
>+++ LayoutTests/editing/rtl-to-ltr-editing-word-move-spin.html	(nonexistent)
>@@ -1,18 +0,0 @@
>-<!DOCTYPE html>
>-<html>
>-<head>
>-<title>This test that we can jump word by word over rtl-ltr content.</title>
>-<meta http-equiv="Content-type" content="text/html; charset=utf-8">
>-</head>
>-<body contenteditable="true";>
>-Pass if no hang.
>-<div style="direction: rtl;">&#x202B foobar foobar&#x202B</div></br>
>-<script>
>-    if (window.testRunner)
>-        testRunner.dumpAsText();
>-    getSelection().setPosition(document.querySelector('div').firstChild, 2);
>-    getSelection().modify('move', 'right', 'word');
>-    getSelection().modify('move', 'right', 'word');
>-</script>
>-</body>
>-</html>
>Index: LayoutTests/editing/simple-line-layout-caret-is-gone-expected.txt
>===================================================================
>--- LayoutTests/editing/simple-line-layout-caret-is-gone-expected.txt	(revision 232420)
>+++ LayoutTests/editing/simple-line-layout-caret-is-gone-expected.txt	(nonexistent)
>@@ -1 +0,0 @@
>-36 0 1 18
>Index: LayoutTests/editing/simple-line-layout-caret-is-gone.html
>===================================================================
>--- LayoutTests/editing/simple-line-layout-caret-is-gone.html	(revision 232420)
>+++ LayoutTests/editing/simple-line-layout-caret-is-gone.html	(nonexistent)
>@@ -1,34 +0,0 @@
>-<!DOCTYPE html>
>-<html>
>-<head>
>-<title>This tests that contenteditable returns the correct caret bounds.</title>
>-<style>
>-body {
>-    margin: 0px;
>-}
>-
>-#editable {
>-    -webkit-nbsp-mode: normal !important;
>-    -webkit-line-break: auto !important;
>-    width: 50px;
>-    height: 50px;
>-}
>-</style>
>-</head>
>-<body>
>-<div id=editable contenteditable=true></div>
>-<script>
>-editable.focus();
>-if (window.testRunner)
>-    testRunner.dumpAsText();
>-if (window.eventSender) {
>-    eventSender.keyDown('\n');
>-    eventSender.keyDown('\n');
>-}
>-if (window.internals) {
>-	var withTextCaretRect = internals.absoluteCaretBounds();
>-    document.body.innerText = withTextCaretRect.top + " " + withTextCaretRect.left + " " + withTextCaretRect.width + " " + withTextCaretRect.height;
>-}
>-</script>
>-</body>
>-</html>
>Index: LayoutTests/editing/input/composition-underline-color-expected-mismatch.html
>===================================================================
>--- LayoutTests/editing/input/composition-underline-color-expected-mismatch.html	(revision 232420)	(from LayoutTests/editing/composition-underline-color-expected-mismatch.html:232420)
>+++ LayoutTests/editing/input/composition-underline-color-expected-mismatch.html	(working copy)
>@@ -0,0 +1,33 @@
>+<!DOCTYPE html>
>+<style>
>+  div {
>+    position: absolute;
>+    top: 0;
>+    left: 0;
>+  }
>+  #test {
>+    color: magenta;
>+    font: 20px/1 Monospace;
>+    outline: none;
>+  }
>+  #overlapping-top {
>+    background: white;
>+    width: 100px;
>+    height: 15px;
>+  }
>+  #overlapping-right {
>+    background: white;
>+    width: 50px;
>+    height: 50px;
>+    left: 50px;
>+  }
>+</style>
>+<div contenteditable id="test"></div>
>+
>+<div id="overlapping-top"></div>
>+<div id="overlapping-right"></div>
>+<script>
>+  document.getElementById("test").focus();
>+  if (window.textInputController)
>+    textInputController.setMarkedText("^^^^^", 5, 0);
>+</script>
>Index: LayoutTests/editing/input/composition-underline-color.html
>===================================================================
>--- LayoutTests/editing/input/composition-underline-color.html	(revision 232420)	(from LayoutTests/editing/composition-underline-color.html:232420)
>+++ LayoutTests/editing/input/composition-underline-color.html	(working copy)
>@@ -0,0 +1,33 @@
>+<!DOCTYPE html>
>+<style>
>+  div {
>+    position: absolute;
>+    top: 0;
>+    left: 0;
>+  }
>+  #test {
>+    color: lime;
>+    font: 20px/1 Monospace;
>+    outline: none;
>+  }
>+  #overlapping-top {
>+    background: white;
>+    width: 100px;
>+    height: 15px;
>+  }
>+  #overlapping-right {
>+    background: white;
>+    width: 50px;
>+    height: 50px;
>+    left: 50px;
>+  }
>+</style>
>+<div contenteditable id="test"></div>
>+
>+<div id="overlapping-top"></div>
>+<div id="overlapping-right"></div>
>+<script>
>+  document.getElementById("test").focus();
>+  if (window.textInputController)
>+    textInputController.setMarkedText("^^^^^", 5, 0);
>+</script>
>Index: LayoutTests/editing/input/marked-text-appearance-expected-mismatch.html
>===================================================================
>--- LayoutTests/editing/input/marked-text-appearance-expected-mismatch.html	(revision 232420)	(from LayoutTests/editing/marked-text-appearance-expected-mismatch.html:232420)
>+++ LayoutTests/editing/input/marked-text-appearance-expected-mismatch.html	(working copy)
>@@ -0,0 +1,9 @@
>+<!DOCTYPE html>
>+<body contenteditable></body>
>+<script>
>+    document.body.focus();
>+    if (window.textInputController) {
>+        textInputController.setMarkedText("^^^^^", 0, 5, true);
>+        textInputController.insertText("^^^^^");
>+    }
>+</script>
>Index: LayoutTests/editing/input/marked-text-appearance.html
>===================================================================
>--- LayoutTests/editing/input/marked-text-appearance.html	(revision 232420)	(from LayoutTests/editing/marked-text-appearance.html:232420)
>+++ LayoutTests/editing/input/marked-text-appearance.html	(working copy)
>@@ -0,0 +1,7 @@
>+<!DOCTYPE html>
>+<body contenteditable></body>
>+<script>
>+    document.body.focus();
>+    if (window.textInputController)
>+        textInputController.setMarkedText("^^^^^", 0, 5, true);
>+</script>
>Index: LayoutTests/editing/selection/rtl-to-ltr-editing-word-move-spin-expected.txt
>===================================================================
>--- LayoutTests/editing/selection/rtl-to-ltr-editing-word-move-spin-expected.txt	(revision 232420)	(from LayoutTests/editing/rtl-to-ltr-editing-word-move-spin-expected.txt:232420)
>+++ LayoutTests/editing/selection/rtl-to-ltr-editing-word-move-spin-expected.txt	(working copy)
>@@ -0,0 +1,4 @@
>+Pass if no hang.
>+â« foobar foobarâ«
>+
>+
>Index: LayoutTests/editing/selection/rtl-to-ltr-editing-word-move-spin.html
>===================================================================
>--- LayoutTests/editing/selection/rtl-to-ltr-editing-word-move-spin.html	(revision 232420)	(from LayoutTests/editing/rtl-to-ltr-editing-word-move-spin.html:232420)
>+++ LayoutTests/editing/selection/rtl-to-ltr-editing-word-move-spin.html	(working copy)
>@@ -0,0 +1,18 @@
>+<!DOCTYPE html>
>+<html>
>+<head>
>+<title>This test that we can jump word by word over rtl-ltr content.</title>
>+<meta http-equiv="Content-type" content="text/html; charset=utf-8">
>+</head>
>+<body contenteditable="true";>
>+Pass if no hang.
>+<div style="direction: rtl;">&#x202B foobar foobar&#x202B</div></br>
>+<script>
>+    if (window.testRunner)
>+        testRunner.dumpAsText();
>+    getSelection().setPosition(document.querySelector('div').firstChild, 2);
>+    getSelection().modify('move', 'right', 'word');
>+    getSelection().modify('move', 'right', 'word');
>+</script>
>+</body>
>+</html>
>Index: LayoutTests/editing/selection/simple-line-layout-caret-is-gone-expected.txt
>===================================================================
>--- LayoutTests/editing/selection/simple-line-layout-caret-is-gone-expected.txt	(revision 232420)	(from LayoutTests/editing/simple-line-layout-caret-is-gone-expected.txt:232420)
>+++ LayoutTests/editing/selection/simple-line-layout-caret-is-gone-expected.txt	(working copy)
>@@ -0,0 +1 @@
>+36 0 1 18
>Index: LayoutTests/editing/selection/simple-line-layout-caret-is-gone.html
>===================================================================
>--- LayoutTests/editing/selection/simple-line-layout-caret-is-gone.html	(revision 232420)	(from LayoutTests/editing/simple-line-layout-caret-is-gone.html:232420)
>+++ LayoutTests/editing/selection/simple-line-layout-caret-is-gone.html	(working copy)
>@@ -0,0 +1,34 @@
>+<!DOCTYPE html>
>+<html>
>+<head>
>+<title>This tests that contenteditable returns the correct caret bounds.</title>
>+<style>
>+body {
>+    margin: 0px;
>+}
>+
>+#editable {
>+    -webkit-nbsp-mode: normal !important;
>+    -webkit-line-break: auto !important;
>+    width: 50px;
>+    height: 50px;
>+}
>+</style>
>+</head>
>+<body>
>+<div id=editable contenteditable=true></div>
>+<script>
>+editable.focus();
>+if (window.testRunner)
>+    testRunner.dumpAsText();
>+if (window.eventSender) {
>+    eventSender.keyDown('\n');
>+    eventSender.keyDown('\n');
>+}
>+if (window.internals) {
>+	var withTextCaretRect = internals.absoluteCaretBounds();
>+    document.body.innerText = withTextCaretRect.top + " " + withTextCaretRect.left + " " + withTextCaretRect.width + " " + withTextCaretRect.height;
>+}
>+</script>
>+</body>
>+</html>
>Index: LayoutTests/editing/style/apply-style-iframe-crash-expected.txt
>===================================================================
>--- LayoutTests/editing/style/apply-style-iframe-crash-expected.txt	(revision 232420)	(from LayoutTests/editing/apply-style-iframe-crash-expected.txt:232420)
>+++ LayoutTests/editing/style/apply-style-iframe-crash-expected.txt	(working copy)
>@@ -0,0 +1,4 @@
>+
>+
>+PASS
>+WebKit bug #132103: Crash applying editing commands from iframe onload event.
>Index: LayoutTests/editing/style/apply-style-iframe-crash.html
>===================================================================
>--- LayoutTests/editing/style/apply-style-iframe-crash.html	(revision 232420)	(from LayoutTests/editing/apply-style-iframe-crash.html:232420)
>+++ LayoutTests/editing/style/apply-style-iframe-crash.html	(working copy)
>@@ -0,0 +1,36 @@
>+<!DOCTYPE html>
>+<script>
>+    function frameLoaded() {
>+        if (document.counter)
>+            document.counter++;
>+        else
>+            document.counter = 1;
>+
>+        if (document.counter <= 16) {
>+            document.designMode='on';
>+            document.execCommand('selectall');
>+            document.execCommand('italic');
>+        }
>+    }
>+
>+    if (window.testRunner) {
>+        testRunner.dumpAsText();
>+        testRunner.waitUntilDone();
>+    }
>+
>+    window.addEventListener("load", function() {
>+        document.getElementById("console").innerText = 'PASS';
>+        if (window.testRunner)
>+            testRunner.notifyDone();
>+    });
>+</script>
>+
>+
>+<h1><button><iframe></iframe></button>
>+</h1><input><iframe onload="frameLoaded()"></iframe>
>+
>+<div id="console">FAIL</div>
>+
>+<div>
>+    WebKit bug #<a href="https://bugs.webkit.org/show_bug.cgi?id=132103">132103</a>: Crash applying editing commands from iframe onload event.
>+</div>

Flags:

wenson_hsieh: review+

Actions: View | Formatted Diff | Diff

Attachments on bug 186212: 341808