Changeset 150106 in webkit

Timestamp:

May 15, 2013 2:03:56 AM (11 years ago)

Author:

Patrick Gansterer

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):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/wince/GlyphPageTreeNodeWinCE.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-05-15  Patrick Gansterer  <paroga@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  <bpoulain@apple.com>
 

trunk/Source/WebCore/platform/graphics/wince/GlyphPageTreeNodeWinCE.cpp

@@ -49 +49 @@
                         setGlyphDataForIndex(offset + i, buffer[i], fontData);
                     else
-                        setGlyphDataForIndex(offset + i, buffer[i], 0);
+                        setGlyphDataForIndex(offset + i, 0, 0);
                 }
                 return true;
@@ -59 +59 @@
                         setGlyphDataForIndex(offset + i, buffer[i], fontData);
                     else
-                        setGlyphDataForIndex(offset + i, buffer[i], 0);
+                        setGlyphDataForIndex(offset + i, 0, 0);
                 }
                 return true;