Unreviewed, rolling out r175813.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Nov 2014 20:33:33 +0000 (20:33 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 10 Nov 2014 20:33:33 +0000 (20:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=138573

Breaks complex text layout spacing (Requested by litherum on
#webkit).

Reverted changeset:

"[OSX] Some words are placed on top of each other in complex
text layout"
https://bugs.webkit.org/show_bug.cgi?id=138348
http://trac.webkit.org/changeset/175813

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@175824 268f45cc-cd09-0410-ab3c-d52691b4dbfc

16 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/text/large-space-width-complex-expected.html [deleted file]
LayoutTests/fast/text/large-space-width-complex.html [deleted file]
LayoutTests/fast/text/resources/NotoNastaliqUrduDraft.ttf [deleted file]
LayoutTests/platform/mac-mavericks/fast/text/emphasis-expected.png [deleted file]
LayoutTests/platform/mac-mavericks/fast/text/emphasis-expected.txt
LayoutTests/platform/mac-mountainlion/fast/text/emphasis-expected.png [deleted file]
LayoutTests/platform/mac-mountainlion/fast/text/emphasis-expected.txt [deleted file]
LayoutTests/platform/mac/fast/text/emphasis-expected.png
LayoutTests/platform/mac/fast/text/emphasis-expected.txt
LayoutTests/platform/mac/fast/text/sticky-typesetting-features-expected.png
LayoutTests/platform/mac/fast/text/sticky-typesetting-features-expected.txt
LayoutTests/platform/mac/fast/writing-mode/text-orientation-basic-expected.png
LayoutTests/platform/mac/fast/writing-mode/text-orientation-basic-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/mac/ComplexTextController.cpp

index d776aba..09a96e1 100644 (file)
@@ -1,3 +1,18 @@
+2014-11-10  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r175813.
+        https://bugs.webkit.org/show_bug.cgi?id=138573
+
+        Breaks complex text layout spacing (Requested by litherum on
+        #webkit).
+
+        Reverted changeset:
+
+        "[OSX] Some words are placed on top of each other in complex
+        text layout"
+        https://bugs.webkit.org/show_bug.cgi?id=138348
+        http://trac.webkit.org/changeset/175813
+
 2014-11-10  Myles C. Maxfield  <mmaxfield@apple.com>
 
         [OSX] Some words are placed on top of each other in complex text layout
diff --git a/LayoutTests/fast/text/large-space-width-complex-expected.html b/LayoutTests/fast/text/large-space-width-complex-expected.html
deleted file mode 100644 (file)
index 0d4ad73..0000000
+++ /dev/null
@@ -1,15 +0,0 @@
-<html dir="rtl" xml:lang="ur">
-<head>
-  <style>
-    @font-face {
-      font-family: NotoNastaliqUrduDraft;
-      src: url("resources/NotoNastaliqUrduDraft.ttf");
-    }
-  </style>
-</head>
-<body>
-<div style="font: 64px NotoNastaliqUrduDraft;">
-<span>&#x644;&#x6d2;</span> &#x622;
-</div>
-</body>
-</html>
diff --git a/LayoutTests/fast/text/large-space-width-complex.html b/LayoutTests/fast/text/large-space-width-complex.html
deleted file mode 100644 (file)
index 2d88ec5..0000000
+++ /dev/null
@@ -1,15 +0,0 @@
-<html dir="rtl" xml:lang="ur">
-<head>
-  <style>
-    @font-face {
-      font-family: NotoNastaliqUrduDraft;
-      src: url("resources/NotoNastaliqUrduDraft.ttf");
-    }
-  </style>
-</head>
-<body>
-<div style="font: 64px NotoNastaliqUrduDraft;">
-&#x644;&#x6d2; &#x622;
-</div>
-</body>
-</html>
diff --git a/LayoutTests/fast/text/resources/NotoNastaliqUrduDraft.ttf b/LayoutTests/fast/text/resources/NotoNastaliqUrduDraft.ttf
deleted file mode 100644 (file)
index e313116..0000000
Binary files a/LayoutTests/fast/text/resources/NotoNastaliqUrduDraft.ttf and /dev/null differ
diff --git a/LayoutTests/platform/mac-mavericks/fast/text/emphasis-expected.png b/LayoutTests/platform/mac-mavericks/fast/text/emphasis-expected.png
deleted file mode 100644 (file)
index 1528711..0000000
Binary files a/LayoutTests/platform/mac-mavericks/fast/text/emphasis-expected.png and /dev/null differ
index 7844228..f7adaf8 100644 (file)
@@ -25,9 +25,9 @@ layer at (0,0) size 800x562
         RenderInline {SPAN} at (0,0) size 217x28
           RenderText {#text} at (3,45) size 217x28
             text run at (3,45) width 217: "consectetur adipiscing"
-        RenderText {#text} at (220,45) size 134x28
+        RenderText {#text} at (220,45) size 135x28
           text run at (220,45) width 6: " "
-          text run at (226,45) width 128: "elit. Aliquam"
+          text run at (226,45) width 129: "elit. Aliquam"
         RenderInline {SPAN} at (0,0) size 111x28
           RenderText {#text} at (3,73) size 111x28
             text run at (3,73) width 111: "odio sa\x{300}pien"
diff --git a/LayoutTests/platform/mac-mountainlion/fast/text/emphasis-expected.png b/LayoutTests/platform/mac-mountainlion/fast/text/emphasis-expected.png
deleted file mode 100644 (file)
index 1528711..0000000
Binary files a/LayoutTests/platform/mac-mountainlion/fast/text/emphasis-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/mac-mountainlion/fast/text/emphasis-expected.txt b/LayoutTests/platform/mac-mountainlion/fast/text/emphasis-expected.txt
deleted file mode 100644 (file)
index 7844228..0000000
+++ /dev/null
@@ -1,191 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x562
-  RenderBlock {HTML} at (0,0) size 800x562
-    RenderBody {BODY} at (8,8) size 784x0
-      RenderBlock (floating) {DIV} at (8,8) size 366x146 [border: (3px solid #000000)]
-        RenderText {#text} at (3,3) size 276x28
-          text run at (3,3) width 276: "Lorem ipsum dolor sit amet,"
-        RenderInline {SPAN} at (0,0) size 217x28
-          RenderText {#text} at (3,45) size 217x28
-            text run at (3,45) width 217: "consectetur adipiscing"
-        RenderText {#text} at (220,45) size 135x28
-          text run at (220,45) width 6: " "
-          text run at (226,45) width 129: "elit. Aliquam"
-        RenderInline {SPAN} at (0,0) size 111x28
-          RenderText {#text} at (3,73) size 111x28
-            text run at (3,73) width 111: "odio sapien"
-        RenderText {#text} at (114,73) size 338x70
-          text run at (114,73) width 12: ", "
-          text run at (126,73) width 215: "lobortis eu iaculis vel,"
-          text run at (3,115) width 211: "scelerisque nec dolor."
-      RenderBlock (floating) {DIV} at (390,8) size 366x146 [border: (3px solid #000000)]
-        RenderText {#text} at (3,3) size 276x28
-          text run at (3,3) width 276: "Lorem ipsum dolor sit amet,"
-        RenderInline {SPAN} at (0,0) size 217x28
-          RenderText {#text} at (3,45) size 217x28
-            text run at (3,45) width 217: "consectetur adipiscing"
-        RenderText {#text} at (220,45) size 134x28
-          text run at (220,45) width 6: " "
-          text run at (226,45) width 128: "elit. Aliquam"
-        RenderInline {SPAN} at (0,0) size 111x28
-          RenderText {#text} at (3,73) size 111x28
-            text run at (3,73) width 111: "odio sa\x{300}pien"
-        RenderText {#text} at (114,73) size 338x70
-          text run at (114,73) width 12: ", "
-          text run at (126,73) width 215: "lobortis eu iaculis vel,"
-          text run at (3,115) width 210: "scelerisque nec dolor."
-      RenderBlock (floating) {DIV} at (8,170) size 366x162 [border: (3px solid #000000)]
-        RenderText {#text} at (3,18) size 71x28
-          text run at (3,18) width 71: "Lorem "
-        RenderInline {SPAN} at (0,0) size 59x28
-          RenderText {#text} at (74,18) size 59x28
-            text run at (74,18) width 59: "ipsum"
-        RenderText {#text} at (133,18) size 6x28
-          text run at (133,18) width 6: " "
-        RenderInline {SPAN} at (0,0) size 51x28
-          RenderText {#text} at (139,18) size 51x28
-            text run at (139,18) width 51: "dolor"
-        RenderText {#text} at (190,18) size 6x28
-          text run at (190,18) width 6: " "
-        RenderInline {SPAN} at (0,0) size 23x28
-          RenderText {#text} at (196,18) size 23x28
-            text run at (196,18) width 23: "sit"
-        RenderText {#text} at (219,18) size 6x28
-          text run at (219,18) width 6: " "
-        RenderInline {SPAN} at (0,0) size 48x28
-          RenderText {#text} at (225,18) size 48x28
-            text run at (225,18) width 48: "amet"
-        RenderText {#text} at (273,18) size 6x28
-          text run at (273,18) width 6: ","
-        RenderInline {SPAN} at (0,0) size 111x28
-          RenderText {#text} at (3,61) size 111x28
-            text run at (3,61) width 111: "consectetur"
-        RenderText {#text} at (114,61) size 6x28
-          text run at (114,61) width 6: " "
-        RenderInline {SPAN} at (0,0) size 100x28
-          RenderText {#text} at (120,61) size 100x28
-            text run at (120,61) width 100: "adipiscing"
-        RenderText {#text} at (220,61) size 6x28
-          text run at (220,61) width 6: " "
-        RenderInline {SPAN} at (0,0) size 32x28
-          RenderText {#text} at (226,61) size 32x28
-            text run at (226,61) width 32: "elit"
-        RenderText {#text} at (258,61) size 12x28
-          text run at (258,61) width 12: ". "
-        RenderInline {SPAN} at (0,0) size 85x28
-          RenderText {#text} at (270,61) size 85x28
-            text run at (270,61) width 85: "Aliquam"
-        RenderText {#text} at (355,61) size 6x28
-          text run at (355,61) width 6: ","
-        RenderInline {SPAN} at (0,0) size 43x28
-          RenderText {#text} at (3,103) size 43x28
-            text run at (3,103) width 43: "odio"
-        RenderText {#text} at (46,103) size 6x28
-          text run at (46,103) width 6: " "
-        RenderInline {SPAN} at (0,0) size 62x28
-          RenderText {#text} at (52,103) size 62x28
-            text run at (52,103) width 62: "sapien"
-        RenderText {#text} at (114,103) size 12x28
-          text run at (114,103) width 12: ", "
-        RenderInline {SPAN} at (0,0) size 74x28
-          RenderText {#text} at (126,103) size 74x28
-            text run at (126,103) width 74: "lobortis"
-        RenderText {#text} at (200,103) size 338x56
-          text run at (200,103) width 6: " "
-          text run at (206,103) width 135: "eu iaculis vel,"
-          text run at (3,131) width 211: "scelerisque nec dolor."
-      RenderBlock (floating) {DIV} at (390,170) size 366x162 [border: (3px solid #000000)]
-        RenderText {#text} at (3,3) size 71x28
-          text run at (3,3) width 71: "Lorem "
-        RenderInline {SPAN} at (0,0) size 59x28
-          RenderText {#text} at (74,3) size 59x28
-            text run at (74,3) width 59: "ipsum"
-        RenderText {#text} at (133,3) size 6x28
-          text run at (133,3) width 6: " "
-        RenderInline {SPAN} at (0,0) size 51x28
-          RenderText {#text} at (139,3) size 51x28
-            text run at (139,3) width 51: "dolor"
-        RenderText {#text} at (190,3) size 6x28
-          text run at (190,3) width 6: " "
-        RenderInline {SPAN} at (0,0) size 23x28
-          RenderText {#text} at (196,3) size 23x28
-            text run at (196,3) width 23: "sit"
-        RenderText {#text} at (219,3) size 6x28
-          text run at (219,3) width 6: " "
-        RenderInline {SPAN} at (0,0) size 48x28
-          RenderText {#text} at (225,3) size 48x28
-            text run at (225,3) width 48: "amet"
-        RenderText {#text} at (273,3) size 6x28
-          text run at (273,3) width 6: ","
-        RenderInline {SPAN} at (0,0) size 111x28
-          RenderText {#text} at (3,46) size 111x28
-            text run at (3,46) width 111: "consectetur"
-        RenderText {#text} at (114,46) size 6x28
-          text run at (114,46) width 6: " "
-        RenderInline {SPAN} at (0,0) size 100x28
-          RenderText {#text} at (120,46) size 100x28
-            text run at (120,46) width 100: "adipiscing"
-        RenderText {#text} at (220,46) size 6x28
-          text run at (220,46) width 6: " "
-        RenderInline {SPAN} at (0,0) size 32x28
-          RenderText {#text} at (226,46) size 32x28
-            text run at (226,46) width 32: "elit"
-        RenderText {#text} at (258,46) size 12x28
-          text run at (258,46) width 12: ". "
-        RenderInline {SPAN} at (0,0) size 85x28
-          RenderText {#text} at (270,46) size 85x28
-            text run at (270,46) width 85: "Aliquam"
-        RenderText {#text} at (355,46) size 6x28
-          text run at (355,46) width 6: ","
-        RenderInline {SPAN} at (0,0) size 43x28
-          RenderText {#text} at (3,89) size 43x28
-            text run at (3,89) width 43: "odio"
-        RenderText {#text} at (46,89) size 6x28
-          text run at (46,89) width 6: " "
-        RenderInline {SPAN} at (0,0) size 62x28
-          RenderText {#text} at (52,89) size 62x28
-            text run at (52,89) width 62: "sapien"
-        RenderText {#text} at (114,89) size 12x28
-          text run at (114,89) width 12: ", "
-        RenderInline {SPAN} at (0,0) size 74x28
-          RenderText {#text} at (126,89) size 74x28
-            text run at (126,89) width 74: "lobortis"
-        RenderText {#text} at (200,89) size 338x70
-          text run at (200,89) width 6: " "
-          text run at (206,89) width 135: "eu iaculis vel,"
-          text run at (3,131) width 211: "scelerisque nec dolor."
-      RenderBlock (floating) {DIV} at (8,348) size 366x198 [border: (3px solid #000000)]
-        RenderText {#text} at (3,13) size 276x28
-          text run at (3,13) width 276: "Lorem ipsum dolor sit amet,"
-        RenderInline {SPAN} at (0,0) size 217x28
-          RenderText {#text} at (3,61) size 217x28
-            text run at (3,61) width 217: "consectetur adipiscing"
-        RenderText {#text} at (220,61) size 135x28
-          text run at (220,61) width 6: " "
-          text run at (226,61) width 129: "elit. Aliquam"
-        RenderInline {SPAN} at (0,0) size 111x28
-          RenderText {#text} at (3,109) size 111x28
-            text run at (3,109) width 111: "odio sapien"
-        RenderText {#text} at (114,109) size 338x76
-          text run at (114,109) width 12: ", "
-          text run at (126,109) width 215: "lobortis eu iaculis vel,"
-          text run at (3,157) width 211: "scelerisque nec dolor."
-      RenderBlock (floating) {DIV} at (390,348) size 366x146 [border: (3px solid #000000)]
-        RenderText {#text} at (3,3) size 276x28
-          text run at (3,3) width 276: "Lorem ipsum dolor sit amet,"
-        RenderInline {SPAN} at (0,0) size 217x28
-          RenderText {#text} at (3,45) size 217x28
-            text run at (3,45) width 217: "consectetur adipiscing"
-        RenderText {#text} at (220,45) size 135x28
-          text run at (220,45) width 6: " "
-          text run at (226,45) width 129: "elit. Aliquam"
-        RenderInline {SPAN} at (0,0) size 111x28
-          RenderText {#text} at (3,73) size 111x28
-            text run at (3,73) width 111: "odio sapien"
-        RenderText {#text} at (114,73) size 338x70
-          text run at (114,73) width 12: ", "
-          text run at (126,73) width 215: "lobortis eu iaculis vel,"
-          text run at (3,115) width 211: "scelerisque nec dolor."
-selection start: position 10 of child 0 {#text} of child 1 {SPAN} of child 10 {DIV} of body
-selection end:   position 7 of child 0 {#text} of child 3 {SPAN} of child 10 {DIV} of body
index 2f8ecff..879ab26 100644 (file)
Binary files a/LayoutTests/platform/mac/fast/text/emphasis-expected.png and b/LayoutTests/platform/mac/fast/text/emphasis-expected.png differ
index 2eb213f..034c9b7 100644 (file)
@@ -25,9 +25,9 @@ layer at (0,0) size 800x567
         RenderInline {SPAN} at (0,0) size 217x28
           RenderText {#text} at (3,45) size 217x28
             text run at (3,45) width 217: "consectetur adipiscing"
-        RenderText {#text} at (220,45) size 134x28
+        RenderText {#text} at (220,45) size 135x28
           text run at (220,45) width 6: " "
-          text run at (226,45) width 128: "elit. Aliquam"
+          text run at (226,45) width 129: "elit. Aliquam"
         RenderInline {SPAN} at (0,0) size 111x28
           RenderText {#text} at (3,73) size 111x28
             text run at (3,73) width 111: "odio sa\x{300}pien"
index dfe2da2..17c48e0 100644 (file)
Binary files a/LayoutTests/platform/mac/fast/text/sticky-typesetting-features-expected.png and b/LayoutTests/platform/mac/fast/text/sticky-typesetting-features-expected.png differ
index 3a1b8aa..5a406a4 100644 (file)
@@ -13,5 +13,5 @@ layer at (0,0) size 800x600
         RenderText {#text} at (0,0) size 324x18
           text run at (0,0) width 324: "This should be rendered with kerning and ligatures:"
       RenderBlock {DIV} at (0,250) size 784x166
-        RenderText {#text} at (0,0) size 751x166
-          text run at (0,0) width 751: "Office Today"
+        RenderText {#text} at (0,0) size 754x166
+          text run at (0,0) width 754: "Office Today"
index 20acbe2..5936c77 100644 (file)
Binary files a/LayoutTests/platform/mac/fast/writing-mode/text-orientation-basic-expected.png and b/LayoutTests/platform/mac/fast/writing-mode/text-orientation-basic-expected.png differ
index 22aa99d..b10705d 100644 (file)
@@ -49,8 +49,8 @@ layer at (0,0) size 785x704
             RenderText {#text} at (94,1) size 36x197
               text run at (94,1) width 197: "Hello world"
           RenderBR {BR} at (112,198) size 0x0
-          RenderInline {SPAN} at (0,0) size 36x197
-            RenderText {#text} at (148,1) size 36x197
-              text run at (148,1) width 197: "Hello world"
-          RenderBR {BR} at (166,198) size 0x0
+          RenderInline {SPAN} at (0,0) size 36x221
+            RenderText {#text} at (148,1) size 36x221
+              text run at (148,1) width 221: "Hello world"
+          RenderBR {BR} at (166,222) size 0x0
         RenderText {#text} at (0,0) size 0x0
index 3264363..fdcc99b 100644 (file)
@@ -1,3 +1,18 @@
+2014-11-10  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r175813.
+        https://bugs.webkit.org/show_bug.cgi?id=138573
+
+        Breaks complex text layout spacing (Requested by litherum on
+        #webkit).
+
+        Reverted changeset:
+
+        "[OSX] Some words are placed on top of each other in complex
+        text layout"
+        https://bugs.webkit.org/show_bug.cgi?id=138348
+        http://trac.webkit.org/changeset/175813
+
 2014-11-10  Beth Dakin  <bdakin@apple.com>
 
         Rolling out failed build fix 
index 5982439..5ebd43e 100644 (file)
@@ -617,7 +617,7 @@ void ComplexTextController::adjustGlyphsAndAdvances()
 
             bool treatAsSpace = Font::treatAsSpace(ch);
             CGGlyph glyph = treatAsSpace ? fontData->spaceGlyph() : glyphs[i];
-            CGSize advance = advances[i];
+            CGSize advance = treatAsSpace ? CGSizeMake(spaceWidth, advances[i].height) : advances[i];
 #if PLATFORM(IOS)
             if (isEmoji && advance.width)
                 advance.width = fontData->widthForGlyph(glyph);