[Webkit-unassigned] [Bug 98100] [Chromium] Should set unitsPerEm in SimpleFontDataSkia.cpp

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=98100





--- Comment #1 from Xianzhu Wang <wangxianzhu at chromium.org>  2012-10-01 17:17:41 PST ---
I'm making the final patch containing a layout test. Here is the patch for preview:

diff --git a/Source/WebCore/platform/graphics/skia/SimpleFontDataSkia.cpp b/Source/WebCore/platform/graphics/sk
index 3027e05..21f9962 100644
--- a/Source/WebCore/platform/graphics/skia/SimpleFontDataSkia.cpp
+++ b/Source/WebCore/platform/graphics/skia/SimpleFontDataSkia.cpp
@@ -159,6 +159,11 @@ void SimpleFontData::platformInit()
             }
         }
     }
+
+    if (SkAdvancedTypefaceMetrics* advancedMetrics = paint.getTypeface()->getAdvancedTypefaceMetrics(SkAdvance
+        m_fontMetrics.setUnitsPerEm(advancedMetrics->fEmSize);
+        advancedMetrics->unref();
+    }
 }

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.