2009-02-02 Dimitri Glazkov <dglazkov@chromium.org>
authordglazkov@chromium.org <dglazkov@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 4 Feb 2009 19:06:37 +0000 (19:06 +0000)
committerdglazkov@chromium.org <dglazkov@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 4 Feb 2009 19:06:37 +0000 (19:06 +0000)
        Reviewed by Darin Adler.

        https://bugs.webkit.org/show_bug.cgi?id=23700
        Upstream local changes to due to unforking of SimpleFontData
        (corresponds to Chromium check-in http://src.chromium.org/viewvc/chrome?view=rev&revision=8646).

        * platform/graphics/chromium/GlyphPageTreeNodeChromiumWin.cpp:
        (WebCore::fillBMPGlyphs): Removed extra checks for ZWS chars.

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

WebCore/ChangeLog
WebCore/platform/graphics/chromium/GlyphPageTreeNodeChromiumWin.cpp

index b9dc10a9d0c11a28d9f8d19ab2830bf5b43427a4..4fd5b561a8d9803d9a28a9bfd1c24fb3e3fb0357 100644 (file)
@@ -1,3 +1,14 @@
+2009-02-02  Dimitri Glazkov  <dglazkov@chromium.org>
+
+        Reviewed by Darin Adler.
+
+        https://bugs.webkit.org/show_bug.cgi?id=23700
+        Upstream local changes to due to unforking of SimpleFontData
+        (corresponds to Chromium check-in http://src.chromium.org/viewvc/chrome?view=rev&revision=8646).
+
+        * platform/graphics/chromium/GlyphPageTreeNodeChromiumWin.cpp:
+        (WebCore::fillBMPGlyphs): Removed extra checks for ZWS chars.
+
 2009-02-04  David Hyatt  <hyatt@apple.com>
 
         <rdar://problem/6554115> REGRESSION (r40475): World leak (JS global object) after viewing a photo on Facebook
index 4c5cf7b7f27056decce2053e50da5008faa7f379..8db59805b6d586846c73c1adaa69acaa9cd952c6 100644 (file)
@@ -147,27 +147,17 @@ static bool fillBMPGlyphs(unsigned offset,
         // When this character should be a space, we ignore whatever the font
         // says and use a space. Otherwise, if fonts don't map one of these
         // space or zero width glyphs, we will get a box.
-        if (Font::treatAsSpace(c))
+        if (Font::treatAsSpace(c)) {
             // Hard code the glyph indices for characters that should be
             // treated like spaces.
             glyph = initSpaceGlyph(dc, &spaceGlyph);
-        else if (Font::treatAsZeroWidthSpace(c) || c == 0x200B) {
-            // FIXME: change Font::treatAsZeroWidthSpace to use
-            // u_hasBinaryProperty, per jungshik's comment here:
-            // https://bugs.webkit.org/show_bug.cgi?id=20237#c6.
-            // Then the additional OR above won't be necessary.
-            glyph = initSpaceGlyph(dc, &spaceGlyph);
-            glyphFontData = fontData->zeroWidthFontData();
         } else if (glyph == invalidGlyph) {
             // WebKit expects both the glyph index and FontData
             // pointer to be 0 if the glyph is not present
             glyph = 0;
             glyphFontData = 0;
-        } else {
-            if (SimpleFontData::isCJKCodePoint(c))
-                glyphFontData = fontData->cjkWidthFontData();
+        } else
             haveGlyphs = true;
-        }
         page->setGlyphDataForCharacter(offset + i, glyph, glyphFontData);
     }