[webkit-reviews] review granted: [Bug 90296] Update complex fonts on Android to use fonts from a newer SDK : [Attachment 150216] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jun 29 10:51:22 PDT 2012


Nate Chapin <japhet at chromium.org> has granted Adam Barth <abarth at webkit.org>'s
request for review:
Bug 90296: Update complex fonts on Android to use fonts from a newer SDK
https://bugs.webkit.org/show_bug.cgi?id=90296

Attachment 150216: Patch
https://bugs.webkit.org/attachment.cgi?id=150216&action=review

------- Additional Comments from Nate Chapin <japhet at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=150216&action=review


>
Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp:251

> +    case HB_Script_Telugu:
> +	   fallbackScript = kTelugu_FallbackScript;

If we're going to reorder into alphabetical order, this should be before Thai.


More information about the webkit-reviews mailing list