Remove unnecessary logging from SimpleFontData
authormmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Oct 2014 18:32:01 +0000 (18:32 +0000)
committermmaxfield@apple.com <mmaxfield@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Oct 2014 18:32:01 +0000 (18:32 +0000)
https://bugs.webkit.org/show_bug.cgi?id=137699

Reviewed by Andreas Kling.

Because of web fonts, fonts that don't have a glyph page 0 are common and expected. Printing out a
message is not helpful, especially if you're not dealing with a font-related piece of WebKit.

No new tests because there is no behavior change.

* platform/graphics/SimpleFontData.cpp:
(WebCore::SimpleFontData::platformGlyphInit):

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

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/SimpleFontData.cpp

index 5f0b319a8f5032982c5ef404f93172d8b13278dc..e9b2e2f1802c02b7203c2b7ec5abe7669ff2196b 100644 (file)
@@ -1,3 +1,18 @@
+2014-10-14  Myles C. Maxfield  <mmaxfield@apple.com>
+
+        Remove unnecessary logging from SimpleFontData
+        https://bugs.webkit.org/show_bug.cgi?id=137699
+
+        Reviewed by Andreas Kling.
+
+        Because of web fonts, fonts that don't have a glyph page 0 are common and expected. Printing out a
+        message is not helpful, especially if you're not dealing with a font-related piece of WebKit.
+
+        No new tests because there is no behavior change.
+
+        * platform/graphics/SimpleFontData.cpp:
+        (WebCore::SimpleFontData::platformGlyphInit):
+
 2014-10-14  Martin Hock  <mhock@apple.com>
 
         Update ViewportConfiguration whenever its size attributes change
index 94174148a38387921d8d974cadc6cdd8f3115a4f..903357e34db02fe9db964f8e51bbea8027310f28 100644 (file)
@@ -115,7 +115,6 @@ void SimpleFontData::platformGlyphInit()
 {
     GlyphPage* glyphPageZero = GlyphPageTreeNode::getRootChild(this, 0)->page();
     if (!glyphPageZero) {
-        LOG_ERROR("Failed to get glyph page zero.");
         m_spaceGlyph = 0;
         m_spaceWidth = 0;
         m_zeroGlyph = 0;
@@ -127,6 +126,8 @@ void SimpleFontData::platformGlyphInit()
         return;
     }
 
+    // Ask for the glyph for 0 to avoid paging in ZERO WIDTH SPACE. Control characters, including 0,
+    // are mapped to the ZERO WIDTH SPACE glyph.
     m_zeroWidthSpaceGlyph = glyphPageZero->glyphDataForCharacter(0).glyph;
 
     // Nasty hack to determine if we should round or ceil space widths.
@@ -142,13 +143,9 @@ void SimpleFontData::platformGlyphInit()
 
     // Force the glyph for ZERO WIDTH SPACE to have zero width, unless it is shared with SPACE.
     // Helvetica is an example of a non-zero width ZERO WIDTH SPACE glyph.
-    // See <http://bugs.webkit.org/show_bug.cgi?id=13178>
-    // Ask for the glyph for 0 to avoid paging in ZERO WIDTH SPACE. Control characters, including 0,
-    // are mapped to the ZERO WIDTH SPACE glyph.
-    if (m_zeroWidthSpaceGlyph == m_spaceGlyph) {
+    // See <http://bugs.webkit.org/show_bug.cgi?id=13178> and SimpleFontData::isZeroWidthSpaceGlyph()
+    if (m_zeroWidthSpaceGlyph == m_spaceGlyph)
         m_zeroWidthSpaceGlyph = 0;
-        LOG_ERROR("Font maps SPACE and ZERO WIDTH SPACE to the same glyph. Glyph width will not be overridden.");
-    }
 
     m_missingGlyphData.fontData = this;
     m_missingGlyphData.glyph = 0;