Title: [150106] trunk/Source/WebCore
Revision
150106
Author
par...@webkit.org
Date
2013-05-15 02:03:56 -0700 (Wed, 15 May 2013)

Log Message

[WINCE] Fix calls to GlyphPage::setGlyphDataForIndex()
https://bugs.webkit.org/show_bug.cgi?id=116137

Reviewed by Andreas Kling.

Use zero for the glyp when the fontData pointer is null.
This aligns GlyphPageTreeNodeWinCE with the other implementations
and makes all ASSERT() pass in setGlyphDataForIndex().

* platform/graphics/wince/GlyphPageTreeNodeWinCE.cpp:
(WebCore::GlyphPage::fill):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (150105 => 150106)


--- trunk/Source/WebCore/ChangeLog	2013-05-15 08:28:43 UTC (rev 150105)
+++ trunk/Source/WebCore/ChangeLog	2013-05-15 09:03:56 UTC (rev 150106)
@@ -1,3 +1,17 @@
+2013-05-15  Patrick Gansterer  <par...@webkit.org>
+
+        [WINCE] Fix calls to GlyphPage::setGlyphDataForIndex()
+        https://bugs.webkit.org/show_bug.cgi?id=116137
+
+        Reviewed by Andreas Kling.
+
+        Use zero for the glyp when the fontData pointer is null.
+        This aligns GlyphPageTreeNodeWinCE with the other implementations
+        and makes all ASSERT() pass in setGlyphDataForIndex().
+
+        * platform/graphics/wince/GlyphPageTreeNodeWinCE.cpp:
+        (WebCore::GlyphPage::fill):
+
 2013-05-15  Benjamin Poulain  <bpoul...@apple.com>
 
         Do not bloat HTMLTokenizer with a giant inline InputStreamPreprocessor::peek

Modified: trunk/Source/WebCore/platform/graphics/wince/GlyphPageTreeNodeWinCE.cpp (150105 => 150106)


--- trunk/Source/WebCore/platform/graphics/wince/GlyphPageTreeNodeWinCE.cpp	2013-05-15 08:28:43 UTC (rev 150105)
+++ trunk/Source/WebCore/platform/graphics/wince/GlyphPageTreeNodeWinCE.cpp	2013-05-15 09:03:56 UTC (rev 150106)
@@ -48,7 +48,7 @@
                     if (!actualCodePages || (actualCodePages & fontCodePages))
                         setGlyphDataForIndex(offset + i, buffer[i], fontData);
                     else
-                        setGlyphDataForIndex(offset + i, buffer[i], 0);
+                        setGlyphDataForIndex(offset + i, 0, 0);
                 }
                 return true;
             } else if (IMLangFontLinkType* langFontLink = fontCache()->getFontLinkInterface()) {
@@ -58,7 +58,7 @@
                     if (!actualCodePages || (actualCodePages & fontCodePages))
                         setGlyphDataForIndex(offset + i, buffer[i], fontData);
                     else
-                        setGlyphDataForIndex(offset + i, buffer[i], 0);
+                        setGlyphDataForIndex(offset + i, 0, 0);
                 }
                 return true;
             }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to