Bug 87890 - [Win] Many editing tests failing
Summary: [Win] Many editing tests failing
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Windows 7
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar, LayoutTestFailure, Regression
Depends on:
Blocks:
 
Reported: 2012-05-30 13:18 PDT by Jessie Berlin
Modified: 2012-05-30 14:05 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jessie Berlin 2012-05-30 13:18:33 PDT
http://build.webkit.org/results/Windows%207%20Release%20(Tests)/r118945%20(24273)/results.html

Some examples:

--- /home/buildbot/slave/win-release-tests/build/layout-test-results/editing/deleting/delete-start-block-expected.txt	2012-05-30 12:01:55.501993100 -0700
+++ /home/buildbot/slave/win-release-tests/build/layout-test-results/editing/deleting/delete-start-block-actual.txt	2012-05-30 12:01:55.500993100 -0700
@@ -1,5 +1,25 @@
 This tests removing the start block. WebKit should not crash.
+| <progress>
+|   <script>
+|     src="../../resources/dump-as-markup.js"
+|   <script>
+|     "
+
+if (window.layoutTestController)
+    layoutTestController.dumpAsText();
+
+document.designMode="on";
+document.execCommand("selectall");
+document.execCommand("justifycenter",false);
+document.body.removeChild(document.body.firstElementChild);
+document.execCommand("insertparagraph");
+
+Markup.description("This tests removing the start block. WebKit should not crash.");
+Markup.dump(document.body);
+
+"
 | <div>
 |   <br>
-| <br>
-| <br>
+| <div>
+|   <#selection-caret>
+|   <br>



--- /home/buildbot/slave/win-release-tests/build/layout-test-results/editing/input/reveal-caret-of-multiline-contenteditable-expected.txt	2012-05-30 12:02:06.636630000 -0700
+++ /home/buildbot/slave/win-release-tests/build/layout-test-results/editing/input/reveal-caret-of-multiline-contenteditable-actual.txt	2012-05-30 12:02:06.635630000 -0700
@@ -62,9 +62,9 @@
     RenderText {#text} at (1,307) size 16x18
       text run at (1,307) width 16: "17"
     RenderBR {BR} at (17,307) size 0x18
-    RenderText {#text} at (1,325) size 25x18
-      text run at (1,325) width 25: ">18"
-    RenderBR {BR} at (26,325) size 0x18
+    RenderText {#text} at (1,325) size 20x18
+      text run at (1,325) width 20: ".18"
+    RenderBR {BR} at (21,325) size 0x18
     RenderText {#text} at (1,343) size 16x18
       text run at (1,343) width 16: "19"
     RenderBR {BR} at (17,343) size 0x18


--- /home/buildbot/slave/win-release-tests/build/layout-test-results/editing/selection/caret-ltr-right-expected.txt	2012-05-30 12:02:38.180024700 -0700
+++ /home/buildbot/slave/win-release-tests/build/layout-test-results/editing/selection/caret-ltr-right-actual.txt	2012-05-30 12:02:38.179024600 -0700
@@ -14,5 +14,6 @@
         RenderText {#text} at (0,0) size 186x18
           text run at (0,0) width 186: "FAIL: selection not collapsed"
         RenderBR {BR} at (186,0) size 0x18
+        RenderText {#text} at (0,0) size 0x0
 selection start: position 0 of child 0 {#text} of child 3 {DIV} of body
 selection end:   position 7 of child 0 {#text} of child 3 {DIV} of body


Unfortunately, due to the state of the Windows bots it is incredibly difficult to figure out when this started happening. I am going to land expected (failing?) results for the editing tests in order to get the bots green.
Comment 1 Radar WebKit Bug Importer 2012-05-30 13:19:41 PDT
<rdar://problem/11559276>
Comment 2 Jessie Berlin 2012-05-30 13:59:00 PDT
Added expected (failing?) results in http://trac.webkit.org/changeset/118964
Comment 3 Ryosuke Niwa 2012-05-30 14:05:59 PDT
(In reply to comment #0)
> --- /home/buildbot/slave/win-release-tests/build/layout-test-results/editing/deleting/delete-start-block-expected.txt    2012-05-30 12:01:55.501993100 -0700
> +++ /home/buildbot/slave/win-release-tests/build/layout-test-results/editing/deleting/delete-
start-block-actual.txt    2012-05-30 12:01:55.500993100 -0700
> @@ -1,5 +1,25 @@
>  This tests removing the start block. WebKit should not crash.
> +| <progress>
> +|   <script>
> +|     src="../../resources/dump-as-markup.js"
> +|   <script>
> +|     "
> +
> +if (window.layoutTestController)
> +    layoutTestController.dumpAsText();
> +
> +document.designMode="on";
> +document.execCommand("selectall");
> +document.execCommand("justifycenter",false);
> +document.body.removeChild(document.body.firstElementChild);
> +document.execCommand("insertparagraph");
> +
> +Markup.description("This tests removing the start block. WebKit should not crash.");
> +Markup.dump(document.body);
> +
> +"
>  | <div>
>  |   <br>
> -| <br>
> -| <br>
> +| <div>
> +|   <#selection-caret>
> +|   <br>

This definitely looks like a regression :(