Reviewed by Anders Carlsson.
authormitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Dec 2007 22:59:42 +0000 (22:59 +0000)
committermitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Dec 2007 22:59:42 +0000 (22:59 +0000)
        - fix regression in fast/text/international/bidi-override on Tiger

        * platform/graphics/GlyphPageTreeNode.cpp:
        (WebCore::GlyphPageTreeNode::initializePage): Add bidi overrides here
        too. I forgot to add them when I added them to treatAsZeroWidthSpace in
        an earlier patch.

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

WebCore/ChangeLog
WebCore/platform/graphics/GlyphPageTreeNode.cpp

index 4dddab69d544b197354bd3a3ddf29c56a78463a3..9974b0169c9c162e09c802b0e6bfd8a7984b64a1 100644 (file)
@@ -1,3 +1,14 @@
+2007-12-13  Dan Bernstein  <mitz@apple.com>
+
+        Reviewed by Anders Carlsson.
+
+        - fix regression in fast/text/international/bidi-override on Tiger
+
+        * platform/graphics/GlyphPageTreeNode.cpp:
+        (WebCore::GlyphPageTreeNode::initializePage): Add bidi overrides here
+        too. I forgot to add them when I added them to treatAsZeroWidthSpace in
+        an earlier patch.
+
 2007-12-13  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by Darin Adler.
index 9d3df6251fbbe146a1331e6ab14f69c2336560cd..8dee8e616dbe557bb9bb8db67e533b0d3a6fd386 100644 (file)
@@ -126,6 +126,8 @@ void GlyphPageTreeNode::initializePage(const FontData* fontData, unsigned pageNu
                     buffer[rightToLeftMark - start] = zeroWidthSpace;
                     buffer[leftToRightEmbed - start] = zeroWidthSpace;
                     buffer[rightToLeftEmbed - start] = zeroWidthSpace;
+                    buffer[leftToRightOverride - start] = zeroWidthSpace;
+                    buffer[rightToLeftOverride - start] = zeroWidthSpace;
                     buffer[popDirectionalFormatting - start] = zeroWidthSpace;
                 } else if (start == (objectReplacementCharacter & ~(GlyphPage::size - 1))) {
                     // Object replacement character must not render at all.