[Webkit-unassigned] [Bug 88806] [Chromium-Android] Remove Android-specific code in FontPlatformDataHarfBuzz.cpp

[bugzilla-daemon at webkit.org]

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


Peter Beverloo <peter at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |reed at google.com




--- Comment #1 from Peter Beverloo <peter at chromium.org>  2012-06-11 15:14:42 PST ---
Since getAdvancedTypefaceMetrics() is only called to get the em size in font units (the caller disregards all other returned information), I don't think this would be the right approach. See the following bugs for more context (also referred to by a TODO in the code):

Bug 75702 ([Chromium] Use SkFontHost::GetUnitsPerEm instead of advanced type metrics for Android in FontPlatformData)
Bug 75961 ([Chromium] Switch to using the new SkTypeface::GetUnitsPerEm API)

http://code.google.com/p/skia/issues/detail?id=441 (Add SkTypeface::GetUnitsPerEm())

+reed

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