Unreviewed, rolling out r205373.
authorryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 2 Sep 2016 23:18:48 +0000 (23:18 +0000)
committerryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 2 Sep 2016 23:18:48 +0000 (23:18 +0000)
This change causes LayoutTest crashes under GuardMalloc

Reverted changeset:

"[Cocoa] Distinguish between paint advances and base advances"
https://bugs.webkit.org/show_bug.cgi?id=160892
http://trac.webkit.org/changeset/205373

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

20 files changed:
LayoutTests/ChangeLog
LayoutTests/platform/mac-elcapitan/css2.1/t051202-c26-psudo-nest-00-c-expected.png [deleted file]
LayoutTests/platform/mac-elcapitan/css2.1/t051202-c26-psudo-nest-00-c-expected.txt [deleted file]
LayoutTests/platform/mac-elcapitan/css2.1/t1508-c527-font-00-b-expected.png [deleted file]
LayoutTests/platform/mac-elcapitan/css2.1/t1508-c527-font-00-b-expected.txt [deleted file]
LayoutTests/platform/mac-elcapitan/fast/inline/absolute-positioned-inline-in-centred-block-expected.png [deleted file]
LayoutTests/platform/mac-elcapitan/fast/inline/absolute-positioned-inline-in-centred-block-expected.txt [deleted file]
LayoutTests/platform/mac/css2.1/t051202-c26-psudo-nest-00-c-expected.png
LayoutTests/platform/mac/css2.1/t051202-c26-psudo-nest-00-c-expected.txt
LayoutTests/platform/mac/css2.1/t1508-c527-font-00-b-expected.png
LayoutTests/platform/mac/css2.1/t1508-c527-font-00-b-expected.txt [deleted file]
LayoutTests/platform/mac/fast/inline/absolute-positioned-inline-in-centred-block-expected.png
LayoutTests/platform/mac/fast/inline/absolute-positioned-inline-in-centred-block-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/GlyphBuffer.h
Source/WebCore/platform/graphics/cocoa/FontCascadeCocoa.mm
Source/WebCore/platform/graphics/mac/ComplexTextController.cpp
Source/WebCore/platform/graphics/mac/ComplexTextController.h
Source/WebCore/platform/graphics/mac/ComplexTextControllerCoreText.mm
Source/WebCore/platform/spi/cocoa/CoreTextSPI.h

index 61a5a25..779e15d 100644 (file)
@@ -1,3 +1,15 @@
+2016-09-02  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Unreviewed, rolling out r205373.
+
+        This change causes LayoutTest crashes under GuardMalloc
+
+        Reverted changeset:
+
+        "[Cocoa] Distinguish between paint advances and base advances"
+        https://bugs.webkit.org/show_bug.cgi?id=160892
+        http://trac.webkit.org/changeset/205373
+
 2016-09-02  Beth Dakin  <bdakin@apple.com>
 
         Need to updateEditorState if an element change edit-ability without changing 
diff --git a/LayoutTests/platform/mac-elcapitan/css2.1/t051202-c26-psudo-nest-00-c-expected.png b/LayoutTests/platform/mac-elcapitan/css2.1/t051202-c26-psudo-nest-00-c-expected.png
deleted file mode 100644 (file)
index d8e7382..0000000
Binary files a/LayoutTests/platform/mac-elcapitan/css2.1/t051202-c26-psudo-nest-00-c-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/mac-elcapitan/css2.1/t051202-c26-psudo-nest-00-c-expected.txt b/LayoutTests/platform/mac-elcapitan/css2.1/t051202-c26-psudo-nest-00-c-expected.txt
deleted file mode 100644 (file)
index 7918bf3..0000000
+++ /dev/null
@@ -1,137 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x455
-  RenderBlock {HTML} at (0,0) size 800x455
-    RenderBody {BODY} at (8,16) size 784x431 [color=#0000FF]
-      RenderBlock {P} at (0,0) size 784x238
-        RenderInline (generated) at (0,0) size 88x166 [color=#00FFFF]
-          RenderText {#text} at (0,0) size 88x166
-            text run at (0,0) width 88: "T"
-        RenderText {#text} at (87,87) size 59x55
-          text run at (87,87) width 59: "he "
-        RenderInline {STRONG} at (0,0) size 205x55
-          RenderText {#text} at (145,87) size 205x55
-            text run at (145,87) width 205: "first letter"
-        RenderText {#text} at (349,87) size 780x97
-          text run at (349,87) width 431: " of this paragraph, and"
-          text run at (0,166) width 61: "only that "
-          text run at (60,166) width 98: "one, should be "
-        RenderInline {STRONG} at (0,0) size 88x18
-          RenderText {#text} at (157,166) size 88x18
-            text run at (157,166) width 88: "600% bigger"
-        RenderText {#text} at (244,166) size 527x18
-          text run at (244,166) width 138: " than the normal text "
-          text run at (381,166) width 390: "(300% bigger than the rest of first line of this paragraph) and"
-        RenderInline {STRONG} at (0,0) size 34x18
-          RenderText {#text} at (0,184) size 34x18
-            text run at (0,184) width 34: "aqua"
-        RenderText {#text} at (33,184) size 113x18
-          text run at (33,184) width 113: ", while the entire "
-        RenderInline {STRONG} at (0,0) size 57x18
-          RenderText {#text} at (145,184) size 57x18
-            text run at (145,184) width 57: "first line"
-        RenderText {#text} at (201,184) size 71x18
-          text run at (201,184) width 5: " "
-          text run at (205,184) width 67: "should be "
-        RenderInline {STRONG} at (0,0) size 89x18
-          RenderText {#text} at (271,184) size 89x18
-            text run at (271,184) width 89: "300% bigger"
-        RenderText {#text} at (359,184) size 113x18
-          text run at (359,184) width 113: " than normal and "
-        RenderInline {STRONG} at (0,0) size 25x18
-          RenderText {#text} at (471,184) size 25x18
-            text run at (471,184) width 25: "teal"
-        RenderText {#text} at (495,184) size 778x54
-          text run at (495,184) width 281: ". If this precise combination does not occur,"
-          text run at (0,202) width 421: "then the user agent has failed this test. Remember that in order to "
-          text run at (420,202) width 358: "ensure a complete test, the paragraph must be displayed"
-          text run at (0,220) width 57: "on more "
-          text run at (56,220) width 149: "than one line. (TEST1)"
-      RenderBlock {P} at (0,254) size 784x73
-        RenderInline (generated) at (0,0) size 33x37 [color=#00FFFF]
-          RenderText {#text} at (0,0) size 33x37
-            text run at (0,0) width 33: "\"T"
-        RenderText {#text} at (32,15) size 99x18
-          text run at (32,15) width 99: "est\": The first "
-        RenderInline {STRONG} at (0,0) size 111x18
-          RenderText {#text} at (130,15) size 111x18
-            text run at (130,15) width 111: "two characters"
-        RenderText {#text} at (240,15) size 452x18
-          text run at (240,15) width 121: " in this paragraph "
-          text run at (360,15) width 332: "(a double-quote mark and a capital 'T') should be "
-        RenderInline {STRONG} at (0,0) size 89x18
-          RenderText {#text} at (691,15) size 89x18
-            text run at (691,15) width 45: "200% "
-            text run at (735,15) width 45: "bigger"
-        RenderText {#text} at (0,37) size 223x18
-          text run at (0,37) width 223: "than the rest of the paragraph, and "
-        RenderInline {STRONG} at (0,0) size 34x18
-          RenderText {#text} at (222,37) size 34x18
-            text run at (222,37) width 34: "aqua"
-        RenderText {#text} at (255,37) size 151x18
-          text run at (255,37) width 151: ". In addition, the entire "
-        RenderInline {STRONG} at (0,0) size 58x18
-          RenderText {#text} at (405,37) size 58x18
-            text run at (405,37) width 33: "first "
-            text run at (437,37) width 26: "line"
-        RenderText {#text} at (462,37) size 98x18
-          text run at (462,37) width 98: " should be in a "
-        RenderInline {STRONG} at (0,0) size 163x18
-          RenderText {#text} at (559,37) size 163x18
-            text run at (559,37) width 138: "small-caps font and "
-            text run at (696,37) width 26: "teal"
-        RenderText {#text} at (721,37) size 738x36
-          text run at (721,37) width 5: "."
-          text run at (0,55) width 323: "Remember that in order to ensure a complete test, "
-          text run at (322,55) width 416: "the paragraph must be displayed on more than one line. (TEST2)"
-      RenderBlock {P} at (0,343) size 784x54
-        RenderInline (generated) at (0,0) size 10x18 [color=#00FFFF]
-          RenderText {#text} at (0,0) size 10x18
-            text run at (0,0) width 10: "T"
-        RenderText {#text} at (9,0) size 20x18
-          text run at (9,0) width 20: "he "
-        RenderInline {STRONG} at (0,0) size 69x18
-          RenderText {#text} at (28,0) size 69x18
-            text run at (28,0) width 69: "first letter"
-        RenderText {#text} at (96,0) size 306x18
-          text run at (96,0) width 209: " of this paragraph, and only that "
-          text run at (304,0) width 98: "one, should be "
-        RenderInline {STRONG} at (0,0) size 35x18
-          RenderText {#text} at (401,0) size 35x18
-            text run at (401,0) width 35: "aqua"
-        RenderText {#text} at (435,0) size 112x18
-          text run at (435,0) width 112: ", while the entire "
-        RenderInline {STRONG} at (0,0) size 58x18
-          RenderText {#text} at (546,0) size 58x18
-            text run at (546,0) width 58: "first line"
-        RenderText {#text} at (603,0) size 71x18
-          text run at (603,0) width 71: " should be "
-        RenderInline {STRONG} at (0,0) size 26x18
-          RenderText {#text} at (673,0) size 26x18
-            text run at (673,0) width 26: "teal"
-        RenderText {#text} at (698,0) size 770x54
-          text run at (698,0) width 23: ". If "
-          text run at (720,0) width 24: "this"
-          text run at (0,18) width 384: "precise combination does not occur, then the user agent has "
-          text run at (383,18) width 387: "failed this test. Remember that in order to ensure a complete"
-          text run at (0,36) width 31: "test, "
-          text run at (30,36) width 416: "the paragraph must be displayed on more than one line. (TEST3)"
-      RenderBlock {DIV} at (0,413) size 784x18
-        RenderText {#text} at (0,0) size 168x18
-          text run at (0,0) width 168: "You should see the words "
-        RenderInline {STRONG} at (0,0) size 68x18
-          RenderText {#text} at (167,0) size 68x18
-            text run at (167,0) width 68: "\"TEST1\""
-        RenderText {#text} at (234,0) size 9x18
-          text run at (234,0) width 9: ", "
-        RenderInline {STRONG} at (0,0) size 68x18
-          RenderText {#text} at (242,0) size 68x18
-            text run at (242,0) width 68: "\"TEST2\""
-        RenderText {#text} at (309,0) size 36x18
-          text run at (309,0) width 36: ", and "
-        RenderInline {STRONG} at (0,0) size 67x18
-          RenderText {#text} at (344,0) size 67x18
-            text run at (344,0) width 67: "\"TEST3\""
-        RenderText {#text} at (410,0) size 241x18
-          text run at (410,0) width 89: " at the end of "
-          text run at (498,0) width 153: "three paragraphs above."
diff --git a/LayoutTests/platform/mac-elcapitan/css2.1/t1508-c527-font-00-b-expected.png b/LayoutTests/platform/mac-elcapitan/css2.1/t1508-c527-font-00-b-expected.png
deleted file mode 100644 (file)
index 4461a74..0000000
Binary files a/LayoutTests/platform/mac-elcapitan/css2.1/t1508-c527-font-00-b-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/mac-elcapitan/css2.1/t1508-c527-font-00-b-expected.txt b/LayoutTests/platform/mac-elcapitan/css2.1/t1508-c527-font-00-b-expected.txt
deleted file mode 100644 (file)
index 91dceab..0000000
+++ /dev/null
@@ -1,23 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x101
-  RenderBlock {HTML} at (0,0) size 800x101
-    RenderBody {BODY} at (8,13) size 784x75 [color=#000080]
-      RenderBlock {P} at (0,0) size 784x75
-        RenderText {#text} at (0,0) size 617x15
-          text run at (0,0) width 617: "This text should be 13px Helvetica in small-caps and italicized. There should be a small gap between each line. "
-        RenderInline {SPAN} at (0,0) size 781x75 [color=#C0C0C0]
-          RenderText {#text} at (616,0) size 781x75
-            text run at (616,0) width 165: "dummy text dummy text dummy"
-            text run at (0,15) width 195: "text dummy text dummy text dummy "
-            text run at (194,15) width 390: "text dummy text dummy text dummy text dummy text dummy text dummy "
-            text run at (583,15) width 192: "text dummy text dummy text dummy"
-            text run at (0,30) width 195: "text dummy text dummy text dummy "
-            text run at (194,30) width 390: "text dummy text dummy text dummy text dummy text dummy text dummy "
-            text run at (583,30) width 192: "text dummy text dummy text dummy"
-            text run at (0,45) width 195: "text dummy text dummy text dummy "
-            text run at (194,45) width 390: "text dummy text dummy text dummy text dummy text dummy text dummy "
-            text run at (583,45) width 192: "text dummy text dummy text dummy"
-            text run at (0,60) width 195: "text dummy text dummy text dummy "
-            text run at (194,60) width 349: "text dummy text dummy text dummy text dummy text dummy text"
-        RenderText {#text} at (0,0) size 0x0
diff --git a/LayoutTests/platform/mac-elcapitan/fast/inline/absolute-positioned-inline-in-centred-block-expected.png b/LayoutTests/platform/mac-elcapitan/fast/inline/absolute-positioned-inline-in-centred-block-expected.png
deleted file mode 100644 (file)
index 3ab7434..0000000
Binary files a/LayoutTests/platform/mac-elcapitan/fast/inline/absolute-positioned-inline-in-centred-block-expected.png and /dev/null differ
diff --git a/LayoutTests/platform/mac-elcapitan/fast/inline/absolute-positioned-inline-in-centred-block-expected.txt b/LayoutTests/platform/mac-elcapitan/fast/inline/absolute-positioned-inline-in-centred-block-expected.txt
deleted file mode 100644 (file)
index ece7b83..0000000
+++ /dev/null
@@ -1,10 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x600
-  RenderBlock {HTML} at (0,0) size 800x600
-    RenderBody {BODY} at (8,8) size 784x584
-      RenderBlock {DIV} at (0,0) size 784x0
-layer at (250,562) size 303x28
-  RenderBlock (positioned) {FONT} at (250,562) size 303x28
-    RenderText {#text} at (0,0) size 303x28
-      text run at (0,0) width 303: "Hello World, And Stuff!"
index 04c7918..d8e7382 100644 (file)
Binary files a/LayoutTests/platform/mac/css2.1/t051202-c26-psudo-nest-00-c-expected.png and b/LayoutTests/platform/mac/css2.1/t051202-c26-psudo-nest-00-c-expected.png differ
index fe9c868..7918bf3 100644 (file)
@@ -56,13 +56,13 @@ layer at (0,0) size 800x455
         RenderInline {STRONG} at (0,0) size 111x18
           RenderText {#text} at (130,15) size 111x18
             text run at (130,15) width 111: "two characters"
-        RenderText {#text} at (240,15) size 451x18
-          text run at (240,15) width 120: " in this paragraph "
-          text run at (359,15) width 332: "(a double-quote mark and a capital 'T') should be "
-        RenderInline {STRONG} at (0,0) size 90x18
-          RenderText {#text} at (690,15) size 90x18
-            text run at (690,15) width 45: "200% "
-            text run at (734,15) width 46: "bigger"
+        RenderText {#text} at (240,15) size 452x18
+          text run at (240,15) width 121: " in this paragraph "
+          text run at (360,15) width 332: "(a double-quote mark and a capital 'T') should be "
+        RenderInline {STRONG} at (0,0) size 89x18
+          RenderText {#text} at (691,15) size 89x18
+            text run at (691,15) width 45: "200% "
+            text run at (735,15) width 45: "bigger"
         RenderText {#text} at (0,37) size 223x18
           text run at (0,37) width 223: "than the rest of the paragraph, and "
         RenderInline {STRONG} at (0,0) size 34x18
index 5eb99c7..4461a74 100644 (file)
Binary files a/LayoutTests/platform/mac/css2.1/t1508-c527-font-00-b-expected.png and b/LayoutTests/platform/mac/css2.1/t1508-c527-font-00-b-expected.png differ
diff --git a/LayoutTests/platform/mac/css2.1/t1508-c527-font-00-b-expected.txt b/LayoutTests/platform/mac/css2.1/t1508-c527-font-00-b-expected.txt
deleted file mode 100644 (file)
index e8cd856..0000000
+++ /dev/null
@@ -1,23 +0,0 @@
-layer at (0,0) size 800x600
-  RenderView at (0,0) size 800x600
-layer at (0,0) size 800x101
-  RenderBlock {HTML} at (0,0) size 800x101
-    RenderBody {BODY} at (8,13) size 784x75 [color=#000080]
-      RenderBlock {P} at (0,0) size 784x75
-        RenderText {#text} at (0,0) size 616x15
-          text run at (0,0) width 616: "This text should be 13px Helvetica in small-caps and italicized. There should be a small gap between each line. "
-        RenderInline {SPAN} at (0,0) size 781x75 [color=#C0C0C0]
-          RenderText {#text} at (615,0) size 781x75
-            text run at (615,0) width 166: "dummy text dummy text dummy"
-            text run at (0,15) width 195: "text dummy text dummy text dummy "
-            text run at (194,15) width 390: "text dummy text dummy text dummy text dummy text dummy text dummy "
-            text run at (583,15) width 192: "text dummy text dummy text dummy"
-            text run at (0,30) width 195: "text dummy text dummy text dummy "
-            text run at (194,30) width 390: "text dummy text dummy text dummy text dummy text dummy text dummy "
-            text run at (583,30) width 192: "text dummy text dummy text dummy"
-            text run at (0,45) width 195: "text dummy text dummy text dummy "
-            text run at (194,45) width 390: "text dummy text dummy text dummy text dummy text dummy text dummy "
-            text run at (583,45) width 192: "text dummy text dummy text dummy"
-            text run at (0,60) width 195: "text dummy text dummy text dummy "
-            text run at (194,60) width 349: "text dummy text dummy text dummy text dummy text dummy text"
-        RenderText {#text} at (0,0) size 0x0
index 9a9b5e8..3ab7434 100644 (file)
Binary files a/LayoutTests/platform/mac/fast/inline/absolute-positioned-inline-in-centred-block-expected.png and b/LayoutTests/platform/mac/fast/inline/absolute-positioned-inline-in-centred-block-expected.png differ
index 6aebd29..ece7b83 100644 (file)
@@ -6,5 +6,5 @@ layer at (0,0) size 800x600
       RenderBlock {DIV} at (0,0) size 784x0
 layer at (250,562) size 303x28
   RenderBlock (positioned) {FONT} at (250,562) size 303x28
-    RenderText {#text} at (0,0) size 302x28
-      text run at (0,0) width 302: "Hello World, And Stuff!"
+    RenderText {#text} at (0,0) size 303x28
+      text run at (0,0) width 303: "Hello World, And Stuff!"
index 5ba8099..35ebc30 100644 (file)
@@ -1,3 +1,14 @@
+2016-09-02  Ryan Haddad  <ryanhaddad@apple.com>
+        Unreviewed, rolling out r205373.
+
+        This change causes LayoutTest crashes under GuardMalloc
+
+        Reverted changeset:
+
+        "[Cocoa] Distinguish between paint advances and base advances"
+        https://bugs.webkit.org/show_bug.cgi?id=160892
+        http://trac.webkit.org/changeset/205373
+
 2016-09-02  Beth Dakin  <bdakin@apple.com>
 
         Need to updateEditorState if an element change edit-ability without changing 
index 0b243b2..563540c 100644 (file)
@@ -67,7 +67,6 @@ public:
     }
 
     void setWidth(CGFloat width) { this->CGSize::width = width; }
-    void setHeight(CGFloat height) { this->CGSize::height = height; }
     CGFloat width() const { return this->CGSize::width; }
     CGFloat height() const { return this->CGSize::height; }
 };
index 3b03f36..7e04cc7 100644 (file)
@@ -514,7 +514,7 @@ float FontCascade::getGlyphsAndAdvancesForComplexText(const TextRun& run, unsign
     float afterWidth = controller.runWidthSoFar();
 
     if (run.rtl()) {
-        initialAdvance = controller.totalWidth() - afterWidth + controller.leadingExpansion();
+        initialAdvance = controller.totalWidth() + controller.finalRoundingWidth() - afterWidth + controller.leadingExpansion();
         glyphBuffer.reverse(0, glyphBuffer.size());
     } else
         initialAdvance = beforeWidth;
index a29a092..b1369cb 100644 (file)
@@ -107,13 +107,28 @@ float FontCascade::width(TextLayout& layout, unsigned from, unsigned len, HashSe
 }
 
 ComplexTextController::ComplexTextController(const FontCascade& font, const TextRun& run, bool mayUseNaturalWritingDirection, HashSet<const Font*>* fallbackFonts, bool forTextEmphasis)
-    : m_fallbackFonts(fallbackFonts)
-    , m_font(font)
+    : m_font(font)
     , m_run(run)
-    , m_end(run.length())
-    , m_expansion(run.expansion())
+    , m_isLTROnly(true)
     , m_mayUseNaturalWritingDirection(mayUseNaturalWritingDirection)
     , m_forTextEmphasis(forTextEmphasis)
+    , m_currentCharacter(0)
+    , m_end(run.length())
+    , m_totalWidth(0)
+    , m_runWidthSoFar(0)
+    , m_numGlyphsSoFar(0)
+    , m_currentRun(0)
+    , m_glyphInCurrentRun(0)
+    , m_characterInCurrentGlyph(0)
+    , m_finalRoundingWidth(0)
+    , m_expansion(run.expansion())
+    , m_leadingExpansion(0)
+    , m_fallbackFonts(fallbackFonts)
+    , m_minGlyphBoundingBoxX(std::numeric_limits<float>::max())
+    , m_maxGlyphBoundingBoxX(std::numeric_limits<float>::min())
+    , m_minGlyphBoundingBoxY(std::numeric_limits<float>::max())
+    , m_maxGlyphBoundingBoxY(std::numeric_limits<float>::min())
+    , m_lastRoundingGlyph(0)
 {
     if (!m_expansion)
         m_expansionPerOpportunity = 0;
@@ -143,7 +158,7 @@ ComplexTextController::ComplexTextController(const FontCascade& font, const Text
     m_runWidthSoFar = m_leadingExpansion;
 }
 
-unsigned ComplexTextController::offsetForPosition(float h, bool includePartialGlyphs)
+int ComplexTextController::offsetForPosition(float h, bool includePartialGlyphs)
 {
     if (h >= m_totalWidth)
         return m_run.ltr() ? m_end : 0;
@@ -161,7 +176,7 @@ unsigned ComplexTextController::offsetForPosition(float h, bool includePartialGl
         const ComplexTextRun& complexTextRun = *m_complexTextRuns[r];
         for (unsigned j = 0; j < complexTextRun.glyphCount(); ++j) {
             size_t index = offsetIntoAdjustedGlyphs + j;
-            CGFloat adjustedAdvance = m_adjustedBaseAdvances[index].width;
+            CGFloat adjustedAdvance = m_adjustedAdvances[index].width;
             if (!index)
                 adjustedAdvance += complexTextRun.initialAdvance().width;
             if (x < adjustedAdvance) {
@@ -173,7 +188,8 @@ unsigned ComplexTextController::offsetForPosition(float h, bool includePartialGl
                     hitGlyphEnd = std::max<CFIndex>(hitGlyphStart, j > 0 ? complexTextRun.indexAt(j - 1) : static_cast<CFIndex>(complexTextRun.indexEnd()));
 
                 // FIXME: Instead of dividing the glyph's advance equally between the characters, this
-                // could use the glyph's "ligature carets". This is available in CoreText via CTFontGetLigatureCaretPositions().
+                // could use the glyph's "ligature carets". However, there is no Core Text API to get the
+                // ligature carets.
                 CFIndex hitIndex = hitGlyphStart + (hitGlyphEnd - hitGlyphStart) * (m_run.ltr() ? x / adjustedAdvance : 1 - x / adjustedAdvance);
                 int stringLength = complexTextRun.stringLength();
                 TextBreakIterator* cursorPositionIterator = cursorMovementIterator(StringView(complexTextRun.characters(), stringLength));
@@ -201,14 +217,14 @@ unsigned ComplexTextController::offsetForPosition(float h, bool includePartialGl
                     clusterWidth = adjustedAdvance;
                     int firstGlyphBeforeCluster = j - 1;
                     while (firstGlyphBeforeCluster >= 0 && complexTextRun.indexAt(firstGlyphBeforeCluster) >= clusterStart && complexTextRun.indexAt(firstGlyphBeforeCluster) < clusterEnd) {
-                        CGFloat width = m_adjustedBaseAdvances[offsetIntoAdjustedGlyphs + firstGlyphBeforeCluster].width;
+                        CGFloat width = m_adjustedAdvances[offsetIntoAdjustedGlyphs + firstGlyphBeforeCluster].width;
                         clusterWidth += width;
                         x += width;
                         firstGlyphBeforeCluster--;
                     }
                     unsigned firstGlyphAfterCluster = j + 1;
                     while (firstGlyphAfterCluster < complexTextRun.glyphCount() && complexTextRun.indexAt(firstGlyphAfterCluster) >= clusterStart && complexTextRun.indexAt(firstGlyphAfterCluster) < clusterEnd) {
-                        clusterWidth += m_adjustedBaseAdvances[offsetIntoAdjustedGlyphs + firstGlyphAfterCluster].width;
+                        clusterWidth += m_adjustedAdvances[offsetIntoAdjustedGlyphs + firstGlyphAfterCluster].width;
                         firstGlyphAfterCluster++;
                     }
                 } else {
@@ -307,11 +323,10 @@ void ComplexTextController::collectComplexTextRuns()
     // We break up glyph run generation for the string by Font.
     const UChar* cp;
 
-    Vector<String> stringsFor8BitRuns;
     if (m_run.is8Bit()) {
         String stringFor8BitRun = String::make16BitFrom8BitSource(m_run.characters8(), m_run.length());
-        stringsFor8BitRuns.append(WTFMove(stringFor8BitRun));
-        cp = stringsFor8BitRuns.last().characters16();
+        cp = stringFor8BitRun.characters16();
+        m_stringsFor8BitRuns.append(stringFor8BitRun);
     } else
         cp = m_run.characters16();
 
@@ -413,7 +428,6 @@ void ComplexTextController::collectComplexTextRuns()
         }
     }
 
-    ASSERT(m_end >= indexOfFontTransition);
     unsigned itemLength = m_end - indexOfFontTransition;
     if (itemLength) {
         unsigned itemStart = indexOfFontTransition;
@@ -449,7 +463,7 @@ void ComplexTextController::ComplexTextRun::setIsNonMonotonic()
     }
 
     m_glyphEndOffsets.grow(m_glyphCount);
-    for (unsigned i = 0; i < m_glyphCount; ++i) {
+    for (size_t i = 0; i < m_glyphCount; ++i) {
         CFIndex nextMappedIndex = m_indexEnd;
         for (size_t j = indexAt(i) + 1; j < m_stringLength; ++j) {
             if (mappedIndices[j]) {
@@ -516,28 +530,9 @@ unsigned ComplexTextController::incrementCurrentRun(unsigned& leftmostGlyph)
     return indexOfCurrentRun(leftmostGlyph);
 }
 
-float ComplexTextController::runWidthSoFarFraction(unsigned glyphStartOffset, unsigned glyphEndOffset, unsigned oldCharacterInCurrentGlyph, GlyphIterationStyle iterationStyle) const
-{
-    // FIXME: Instead of dividing the glyph's advance equally between the characters, this
-    // could use the glyph's "ligature carets". This is available in CoreText via CTFontGetLigatureCaretPositions().
-    if (glyphStartOffset == glyphEndOffset) {
-        // When there are multiple glyphs per character we need to advance by the full width of the glyph.
-        ASSERT(m_characterInCurrentGlyph == oldCharacterInCurrentGlyph);
-        return 1;
-    }
-
-    if (iterationStyle == ByWholeGlyphs) {
-        if (!oldCharacterInCurrentGlyph)
-            return 1;
-        return 0;
-    }
-
-    return static_cast<float>(m_characterInCurrentGlyph - oldCharacterInCurrentGlyph) / (glyphEndOffset - glyphStartOffset);
-}
-
 void ComplexTextController::advance(unsigned offset, GlyphBuffer* glyphBuffer, GlyphIterationStyle iterationStyle, HashSet<const Font*>* fallbackFonts)
 {
-    if (offset > m_end)
+    if (static_cast<int>(offset) > m_end)
         offset = m_end;
 
     if (offset <= m_currentCharacter) {
@@ -567,14 +562,7 @@ void ComplexTextController::advance(unsigned offset, GlyphBuffer* glyphBuffer, G
         // When leftmostGlyph is 0, it represents the first glyph to draw, taking into
         // account the text direction.
         if (glyphBuffer && !leftmostGlyph) {
-            CGSize initialAdvance = complexTextRun.initialAdvance();
-#if USE_LAYOUT_SPECIFIC_ADVANCES
-            unsigned index = ltr ? 0 : m_glyphOrigins.size() - 1;
-            initialAdvance.width += m_glyphOrigins[index].x;
-            initialAdvance.height += m_glyphOrigins[index].y;
-#endif
-            glyphBuffer->setInitialAdvance(initialAdvance);
-
+            glyphBuffer->setInitialAdvance(complexTextRun.initialAdvance());
             glyphBuffer->setLeadingExpansion(m_leadingExpansion);
         }
 
@@ -589,25 +577,28 @@ void ComplexTextController::advance(unsigned offset, GlyphBuffer* glyphBuffer, G
             } else
                 glyphEndOffset = complexTextRun.endOffsetAt(g);
 
-            CGSize adjustedBaseAdvance = m_adjustedBaseAdvances[k];
+            CGSize adjustedAdvance = m_adjustedAdvances[k];
 
             if (glyphStartOffset + complexTextRun.stringLocation() >= m_currentCharacter)
                 return;
 
-            if (glyphBuffer && !m_characterInCurrentGlyph) {
-                GlyphBufferAdvance paintAdvance = adjustedBaseAdvance;
-#if USE_LAYOUT_SPECIFIC_ADVANCES
-                if (k + 1 < m_adjustedBaseAdvances.size()) {
-                    paintAdvance.setWidth(paintAdvance.width() + m_glyphOrigins[k + 1].x - m_glyphOrigins[k].x);
-                    paintAdvance.setHeight(paintAdvance.height() - m_glyphOrigins[k + 1].y + m_glyphOrigins[k].y);
-                }
-#endif
-                glyphBuffer->add(m_adjustedGlyphs[k], &complexTextRun.font(), paintAdvance, complexTextRun.indexAt(m_glyphInCurrentRun));
-            }
+            if (glyphBuffer && !m_characterInCurrentGlyph)
+                glyphBuffer->add(m_adjustedGlyphs[k], &complexTextRun.font(), adjustedAdvance, complexTextRun.indexAt(m_glyphInCurrentRun));
 
             unsigned oldCharacterInCurrentGlyph = m_characterInCurrentGlyph;
             m_characterInCurrentGlyph = std::min(m_currentCharacter - complexTextRun.stringLocation(), glyphEndOffset) - glyphStartOffset;
-            m_runWidthSoFar += adjustedBaseAdvance.width * runWidthSoFarFraction(glyphStartOffset, glyphEndOffset, oldCharacterInCurrentGlyph, iterationStyle);
+            // FIXME: Instead of dividing the glyph's advance equally between the characters, this
+            // could use the glyph's "ligature carets". However, there is no Core Text API to get the
+            // ligature carets.
+            if (glyphStartOffset == glyphEndOffset) {
+                // When there are multiple glyphs per character we need to advance by the full width of the glyph.
+                ASSERT(m_characterInCurrentGlyph == oldCharacterInCurrentGlyph);
+                m_runWidthSoFar += adjustedAdvance.width;
+            } else if (iterationStyle == ByWholeGlyphs) {
+                if (!oldCharacterInCurrentGlyph)
+                    m_runWidthSoFar += adjustedAdvance.width;
+            } else
+                m_runWidthSoFar += adjustedAdvance.width * (m_characterInCurrentGlyph - oldCharacterInCurrentGlyph) / (glyphEndOffset - glyphStartOffset);
 
             if (glyphEndOffset + complexTextRun.stringLocation() > m_currentCharacter)
                 return;
@@ -626,6 +617,8 @@ void ComplexTextController::advance(unsigned offset, GlyphBuffer* glyphBuffer, G
         currentRunIndex = incrementCurrentRun(leftmostGlyph);
         m_glyphInCurrentRun = 0;
     }
+    if (!m_run.ltr() && m_numGlyphsSoFar == m_adjustedAdvances.size())
+        m_runWidthSoFar += m_finalRoundingWidth;
 }
 
 static inline std::pair<bool, bool> expansionLocation(bool ideograph, bool treatAsSpace, bool ltr, bool isAfterExpansion, bool forbidLeadingExpansion, bool forbidTrailingExpansion, bool forceLeadingExpansion, bool forceTrailingExpansion)
@@ -663,7 +656,6 @@ void ComplexTextController::adjustGlyphsAndAdvances()
     bool runForcesTrailingExpansion = (m_run.expansionBehavior() & TrailingExpansionMask) == ForceTrailingExpansion;
     bool runForbidsLeadingExpansion = (m_run.expansionBehavior() & LeadingExpansionMask) == ForbidLeadingExpansion;
     bool runForbidsTrailingExpansion = (m_run.expansionBehavior() & TrailingExpansionMask) == ForbidTrailingExpansion;
-
     // We are iterating in glyph order, not string order. Compare this to WidthIterator::advanceInternal()
     for (size_t r = 0; r < runCount; ++r) {
         ComplexTextRun& complexTextRun = *m_complexTextRuns[r];
@@ -672,11 +664,11 @@ void ComplexTextController::adjustGlyphsAndAdvances()
 
         // Represent the initial advance for a text run by adjusting the advance
         // of the last glyph of the previous text run in the glyph buffer.
-        if (r && m_adjustedBaseAdvances.size()) {
-            CGSize previousAdvance = m_adjustedBaseAdvances.last();
+        if (r && m_adjustedAdvances.size()) {
+            CGSize previousAdvance = m_adjustedAdvances.last();
             previousAdvance.width += complexTextRun.initialAdvance().width;
             previousAdvance.height -= complexTextRun.initialAdvance().height;
-            m_adjustedBaseAdvances[m_adjustedBaseAdvances.size() - 1] = previousAdvance;
+            m_adjustedAdvances[m_adjustedAdvances.size() - 1] = previousAdvance;
         }
         widthSinceLastCommit += complexTextRun.initialAdvance().width;
 
@@ -684,7 +676,7 @@ void ComplexTextController::adjustGlyphsAndAdvances()
             m_isLTROnly = false;
 
         const CGGlyph* glyphs = complexTextRun.glyphs();
-        const CGSize* advances = complexTextRun.baseAdvances();
+        const CGSize* advances = complexTextRun.advances();
 
         bool lastRun = r + 1 == runCount;
         float spaceWidth = font.spaceWidth() - font.syntheticBoldOffset();
@@ -758,10 +750,10 @@ void ComplexTextController::adjustGlyphsAndAdvances()
                             // Increase previous width
                             m_expansion -= m_expansionPerOpportunity;
                             m_totalWidth += m_expansionPerOpportunity;
-                            if (m_adjustedBaseAdvances.isEmpty())
+                            if (m_adjustedAdvances.isEmpty())
                                 m_leadingExpansion = m_expansionPerOpportunity;
                             else
-                                m_adjustedBaseAdvances.last().width += m_expansionPerOpportunity;
+                                m_adjustedAdvances.last().width += m_expansionPerOpportunity;
                         }
                         if (expandRight) {
                             m_expansion -= m_expansionPerOpportunity;
@@ -785,10 +777,7 @@ void ComplexTextController::adjustGlyphsAndAdvances()
                 glyph = 0;
 
             advance.height *= -1;
-            m_adjustedBaseAdvances.append(advance);
-#if USE_LAYOUT_SPECIFIC_ADVANCES
-            m_glyphOrigins.append(complexTextRun.glyphOrigins()[i]);
-#endif
+            m_adjustedAdvances.append(advance);
             m_adjustedGlyphs.append(glyph);
             
             FloatRect glyphBounds = font.boundsForGlyph(glyph);
index 287abfb..8148a04 100644 (file)
@@ -32,8 +32,6 @@
 #include <wtf/Vector.h>
 #include <wtf/text/WTFString.h>
 
-#define USE_LAYOUT_SPECIFIC_ADVANCES ((PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED >= 101100) || (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED >= 90000))
-
 typedef unsigned short CGGlyph;
 
 typedef const struct __CTRun * CTRunRef;
@@ -58,13 +56,15 @@ public:
     void advance(unsigned to, GlyphBuffer* = nullptr, GlyphIterationStyle = IncludePartialGlyphs, HashSet<const Font*>* fallbackFonts = nullptr);
 
     // Compute the character offset for a given x coordinate.
-    unsigned offsetForPosition(float x, bool includePartialGlyphs);
+    int offsetForPosition(float x, bool includePartialGlyphs);
 
     // Returns the width of everything we've consumed so far.
     float runWidthSoFar() const { return m_runWidthSoFar; }
 
     float totalWidth() const { return m_totalWidth; }
 
+    float finalRoundingWidth() const { return m_finalRoundingWidth; }
+
     float minGlyphBoundingBoxX() const { return m_minGlyphBoundingBoxX; }
     float maxGlyphBoundingBoxX() const { return m_maxGlyphBoundingBoxX; }
     float minGlyphBoundingBoxY() const { return m_minGlyphBoundingBoxY; }
@@ -95,20 +95,8 @@ private:
         CFIndex indexEnd() const { return m_indexEnd; }
         CFIndex endOffsetAt(size_t i) const { ASSERT(!m_isMonotonic); return m_glyphEndOffsets[i]; }
         const CGGlyph* glyphs() const { return m_glyphs; }
-
-        /*
-         *                                              X (Paint glyph position)   X (Paint glyph position)   X (Paint glyph position)
-         *                                             7                          7                          7
-         *                                            /                          /                          /
-         *                                           / (Glyph origin)           / (Glyph origin)           / (Glyph origin)
-         *                                          /                          /                          /
-         *                                         /                          /                          /
-         *                X-----------------------X--------------------------X--------------------------X---->...
-         * (text position ^)  (initial advance)          (base advance)             (base advance)
-         */
         CGSize initialAdvance() const { return m_initialAdvance; }
-        const CGSize* baseAdvances() const { return m_baseAdvances; }
-        const CGPoint* glyphOrigins() const { return m_glyphOrigins.data(); }
+        const CGSize* advances() const { return m_advances; }
         bool isLTR() const { return m_isLTR; }
         bool isMonotonic() const { return m_isMonotonic; }
         void setIsNonMonotonic();
@@ -117,22 +105,21 @@ private:
         ComplexTextRun(CTRunRef, const Font&, const UChar* characters, unsigned stringLocation, size_t stringLength, CFRange runRange);
         ComplexTextRun(const Font&, const UChar* characters, unsigned stringLocation, size_t stringLength, bool ltr);
 
-        Vector<CGSize, 64> m_baseAdvancesVector;
-        Vector<CGPoint, 64> m_glyphOrigins;
-        Vector<CGGlyph, 64> m_glyphsVector;
-        Vector<CFIndex, 64> m_glyphEndOffsets;
-        Vector<CFIndex, 64> m_coreTextIndicesVector;
-        CGSize m_initialAdvance;
+        unsigned m_glyphCount;
         const Font& m_font;
         const UChar* m_characters;
+        unsigned m_stringLocation;
         size_t m_stringLength;
+        Vector<CFIndex, 64> m_coreTextIndicesVector;
         const CFIndex* m_coreTextIndices;
-        const CGGlyph* m_glyphs;
-        const CGSize* m_baseAdvances;
         CFIndex m_indexBegin;
         CFIndex m_indexEnd;
-        unsigned m_glyphCount;
-        unsigned m_stringLocation;
+        Vector<CFIndex, 64> m_glyphEndOffsets;
+        Vector<CGGlyph, 64> m_glyphsVector;
+        const CGGlyph* m_glyphs;
+        CGSize m_initialAdvance;
+        Vector<CGSize, 64> m_advancesVector;
+        const CGSize* m_advances;
         bool m_isLTR;
         bool m_isMonotonic;
     };
@@ -148,12 +135,18 @@ private:
     unsigned indexOfCurrentRun(unsigned& leftmostGlyph);
     unsigned incrementCurrentRun(unsigned& leftmostGlyph);
 
-    float runWidthSoFarFraction(unsigned glyphStartOffset, unsigned glyphEndOffset, unsigned oldCharacterInCurrentGlyph, GlyphIterationStyle) const;
+    // The initial capacity of these vectors was selected as being the smallest power of two greater than
+    // the average (3.5) plus one standard deviation (7.5) of nonzero sizes used on Arabic Wikipedia.
+    Vector<unsigned, 16> m_runIndices;
+    Vector<unsigned, 16> m_glyphCountFromStartToIndex;
 
-    Vector<CGSize, 256> m_adjustedBaseAdvances;
-    Vector<CGPoint, 256> m_glyphOrigins;
-    Vector<CGGlyph, 256> m_adjustedGlyphs;
+    const FontCascade& m_font;
+    const TextRun& m_run;
+    bool m_isLTROnly;
+    bool m_mayUseNaturalWritingDirection;
+    bool m_forTextEmphasis;
 
+    Vector<String> m_stringsFor8BitRuns;
     Vector<UChar, 256> m_smallCapsBuffer;
 
     // There is a 3-level hierarchy here. At the top, we are interested in m_run.string(). We partition that string
@@ -163,41 +156,34 @@ private:
     // relative to m_run.string(). ComplexTextRun::indexAt() returns to the offset of a codepoint relative to
     // its Line. ComplexTextRun::glyphs() and ComplexTextRun::advances() refer to glyphs relative to the ComplexTextRun.
     // The length of the entire TextRun is m_run.length()
+    Vector<RetainPtr<CTLineRef>> m_coreTextLines;
     Vector<RefPtr<ComplexTextRun>, 16> m_complexTextRuns;
+    Vector<CGSize, 256> m_adjustedAdvances;
+    Vector<CGGlyph, 256> m_adjustedGlyphs;
+    unsigned m_currentCharacter;
+    int m_end;
 
-    // The initial capacity of these vectors was selected as being the smallest power of two greater than
-    // the average (3.5) plus one standard deviation (7.5) of nonzero sizes used on Arabic Wikipedia.
-    Vector<unsigned, 16> m_runIndices;
-    Vector<unsigned, 16> m_glyphCountFromStartToIndex;
+    CGFloat m_totalWidth;
 
-    Vector<RetainPtr<CTLineRef>> m_coreTextLines;
+    float m_runWidthSoFar;
+    unsigned m_numGlyphsSoFar;
+    size_t m_currentRun;
+    unsigned m_glyphInCurrentRun;
+    unsigned m_characterInCurrentGlyph;
+    float m_finalRoundingWidth;
+    float m_expansion;
+    float m_expansionPerOpportunity;
+    float m_leadingExpansion;
 
-    HashSet<const Font*>* m_fallbackFonts { nullptr };
+    HashSet<const Font*>* m_fallbackFonts;
 
-    const FontCascade& m_font;
-    const TextRun& m_run;
-    unsigned m_currentCharacter { 0 };
-    unsigned m_end { 0 };
-
-    float m_totalWidth { 0 };
-    float m_runWidthSoFar { 0 };
-    unsigned m_numGlyphsSoFar { 0 };
-    unsigned m_currentRun { 0 };
-    unsigned m_glyphInCurrentRun { 0 };
-    unsigned m_characterInCurrentGlyph { 0 };
-    float m_expansion { 0 };
-    float m_expansionPerOpportunity { 0 };
-    float m_leadingExpansion { 0 };
-
-    float m_minGlyphBoundingBoxX { std::numeric_limits<float>::max() };
-    float m_maxGlyphBoundingBoxX { std::numeric_limits<float>::min() };
-    float m_minGlyphBoundingBoxY { std::numeric_limits<float>::max() };
-    float m_maxGlyphBoundingBoxY { std::numeric_limits<float>::min() };
-
-    bool m_isLTROnly { true };
-    bool m_mayUseNaturalWritingDirection { false };
-    bool m_forTextEmphasis { false };
+    float m_minGlyphBoundingBoxX;
+    float m_maxGlyphBoundingBoxX;
+    float m_minGlyphBoundingBoxY;
+    float m_maxGlyphBoundingBoxY;
+
+    unsigned m_lastRoundingGlyph;
 };
 
 } // namespace WebCore
index 5a39569..4c03cac 100644 (file)
@@ -29,7 +29,6 @@
 #include "CoreTextSPI.h"
 #include "FontCache.h"
 #include "FontCascade.h"
-#include "SoftLinking.h"
 #include "TextRun.h"
 #include "WebCoreSystemInterface.h"
 #include <wtf/WeakPtr.h>
 #include <ApplicationServices/ApplicationServices.h>
 #endif
 
-#if (PLATFORM(MAC) && __MAC_OS_X_VERSION_MIN_REQUIRED < 101100) || (PLATFORM(IOS) && __IPHONE_OS_VERSION_MIN_REQUIRED < 90000)
-SOFT_LINK_FRAMEWORK(CoreText);
-SOFT_LINK(CoreText, CTRunGetBaseAdvancesAndOrigins, void, (CTRunRef run, CFRange range, CGSize baseAdvances[], CGPoint origins[]), (run, range, baseAdvances, origins))
-#endif
-
-
 // Note: CTFontDescriptorRefs can live forever in caches inside CoreText, so this object can too.
 @interface WebCascadeList : NSArray {
     @private
@@ -107,13 +100,13 @@ SOFT_LINK(CoreText, CTRunGetBaseAdvancesAndOrigins, void, (CTRunRef run, CFRange
 namespace WebCore {
 
 ComplexTextController::ComplexTextRun::ComplexTextRun(CTRunRef ctRun, const Font& font, const UChar* characters, unsigned stringLocation, size_t stringLength, CFRange runRange)
-    : m_initialAdvance(CTRunGetInitialAdvance(ctRun))
-    , m_font(font)
+    : m_font(font)
     , m_characters(characters)
+    , m_stringLocation(stringLocation)
     , m_stringLength(stringLength)
     , m_indexBegin(runRange.location)
     , m_indexEnd(runRange.location + runRange.length)
-    , m_stringLocation(stringLocation)
+    , m_initialAdvance(CTRunGetInitialAdvance(ctRun))
     , m_isLTR(!(CTRunGetStatus(ctRun) & kCTRunStatusRightToLeft))
     , m_isMonotonic(true)
 {
@@ -132,31 +125,24 @@ ComplexTextController::ComplexTextRun::ComplexTextRun(CTRunRef ctRun, const Font
         m_glyphs = m_glyphsVector.data();
     }
 
-#if USE_LAYOUT_SPECIFIC_ADVANCES
-    m_baseAdvancesVector.grow(m_glyphCount);
-    m_glyphOrigins.grow(m_glyphCount);
-    CTRunGetBaseAdvancesAndOrigins(ctRun, CFRangeMake(0, 0), m_baseAdvancesVector.data(), m_glyphOrigins.data());
-    m_baseAdvances = m_baseAdvancesVector.data();
-#else
-    m_baseAdvances = CTRunGetAdvancesPtr(ctRun);
-    if (!m_baseAdvances) {
-        m_baseAdvancesVector.grow(m_glyphCount);
-        CTRunGetAdvances(ctRun, CFRangeMake(0, 0), m_baseAdvancesVector.data());
-        m_baseAdvances = m_baseAdvancesVector.data();
+    m_advances = CTRunGetAdvancesPtr(ctRun);
+    if (!m_advances) {
+        m_advancesVector.grow(m_glyphCount);
+        CTRunGetAdvances(ctRun, CFRangeMake(0, 0), m_advancesVector.data());
+        m_advances = m_advancesVector.data();
     }
-#endif
 }
 
 // Missing glyphs run constructor. Core Text will not generate a run of missing glyphs, instead falling back on
 // glyphs from LastResort. We want to use the primary font's missing glyph in order to match the fast text code path.
 ComplexTextController::ComplexTextRun::ComplexTextRun(const Font& font, const UChar* characters, unsigned stringLocation, size_t stringLength, bool ltr)
-    : m_initialAdvance(CGSizeZero)
-    , m_font(font)
+    : m_font(font)
     , m_characters(characters)
+    , m_stringLocation(stringLocation)
     , m_stringLength(stringLength)
     , m_indexBegin(0)
     , m_indexEnd(stringLength)
-    , m_stringLocation(stringLocation)
+    , m_initialAdvance(CGSizeZero)
     , m_isLTR(ltr)
     , m_isMonotonic(true)
 {
@@ -164,8 +150,10 @@ ComplexTextController::ComplexTextRun::ComplexTextRun(const Font& font, const UC
     unsigned r = 0;
     while (r < m_stringLength) {
         m_coreTextIndicesVector.uncheckedAppend(r);
-        UChar32 character;
-        U16_NEXT(m_characters, r, m_stringLength, character);
+        if (U_IS_LEAD(m_characters[r]) && r + 1 < m_stringLength && U_IS_TRAIL(m_characters[r + 1]))
+            r += 2;
+        else
+            r++;
     }
     m_glyphCount = m_coreTextIndicesVector.size();
     if (!ltr) {
@@ -177,8 +165,8 @@ ComplexTextController::ComplexTextRun::ComplexTextRun(const Font& font, const UC
     // Synthesize a run of missing glyphs.
     m_glyphsVector.fill(0, m_glyphCount);
     m_glyphs = m_glyphsVector.data();
-    m_baseAdvancesVector.fill(CGSizeMake(m_font.widthForGlyph(0), 0), m_glyphCount);
-    m_baseAdvances = m_baseAdvancesVector.data();
+    m_advancesVector.fill(CGSizeMake(m_font.widthForGlyph(0), 0), m_glyphCount);
+    m_advances = m_advancesVector.data();
 }
 
 struct ProviderInfo {
index 025b912..ee5e22b 100644 (file)
@@ -68,7 +68,6 @@ bool CTFontTransformGlyphs(CTFontRef, CGGlyph glyphs[], CGSize advances[], CFInd
 
 CGSize CTRunGetInitialAdvance(CTRunRef run);
 CTLineRef CTLineCreateWithUniCharProvider(CTUniCharProviderCallback provide, CTUniCharDisposeCallback dispose, void* refCon);
-void CTRunGetBaseAdvancesAndOrigins(CTRunRef, CFRange, CGSize baseAdvances[], CGPoint origins[]);
 CTTypesetterRef CTTypesetterCreateWithUniCharProviderAndOptions(CTUniCharProviderCallback provide, CTUniCharDisposeCallback dispose, void* refCon, CFDictionaryRef options);
 bool CTFontGetVerticalGlyphsForCharacters(CTFontRef, const UniChar characters[], CGGlyph glyphs[], CFIndex count);