From ab815a701ef025ce5eb382e8ecbb69d536c598c4 Mon Sep 17 00:00:00 2001
From: Raph Levien <raph@google.com>
Date: Tue, 4 Dec 2012 14:02:43 -0800
Subject: Fix for bug 7467609 Bad rendering of Telugu text

This change moves Telugu back before the fallback font. Previously it
had been moved to after it because it contained incorrect mappings of a
number of characters, including U+2212 and U+FFFD. These fonts have now
been fixed. When placed after the Droid Sans Fallback font, the glyph
count for complex script shaping exceeded the 2^16 limit, causing
incorrect rendering.

This patch depends on the fix for the Lohit-Telugu font.

Change-Id: Ib1bc665a246c4ad65a0dea1c261df5696b440344
---
 data/fonts/fallback_fonts.xml | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

(limited to 'data/fonts')

diff --git a/data/fonts/fallback_fonts.xml b/data/fonts/fallback_fonts.xml
index 50ff437..999ddc4 100644
--- a/data/fonts/fallback_fonts.xml
+++ b/data/fonts/fallback_fonts.xml
@@ -103,21 +103,23 @@
     </family>
     <family>
         <fileset>
-            <file>DroidSansFallback.ttf</file>
+            <file>Lohit-Telugu.ttf</file>
         </fileset>
     </family>
     <family>
         <fileset>
-            <file lang="ja">MTLmr3m.ttf</file>
+            <file>DroidSansFallback.ttf</file>
         </fileset>
     </family>
-    <!--
-        Fonts below this point have problematic glyphs and should not be moved
-        higher in the fallback list until those glyphs have been fixed.
-    -->
     <family>
         <fileset>
-            <file>Lohit-Telugu.ttf</file> <!-- masks U+FFBC-10007 -->
+            <file lang="ja">MTLmr3m.ttf</file>
         </fileset>
     </family>
+    <!-- Note: complex scripts (i.e. those requiring shaping in Harfbuzz) have
+         a cumulative limit of 64k glyphs. Thus, if they are placed after the
+         large fonts such as DroidSansFallback, they are likely to render
+         incorrectly. Please use caution when putting fonts toward the end of
+         the list.
+    -->
 </familyset>
-- 
cgit v1.1