Make RenderTextHarfBuzz examine full font list.

When evaluating which font to use for a run,
RenderTextHarfBuzz should examine all fonts in the font list
that was supplied to it. Previously, it only tried the
primary font before resorting to its fallback logic.

BUG=none

Review URL: https://codereview.chromium.org/678683003

Cr-Commit-Position: refs/heads/master@{#301284}

1 files changed, 5 insertions, 4 deletions

diff --git a/ui/gfx/render_text_harfbuzz.cc b/ui/gfx/render_text_harfbuzz.cc
index 3ee5332..642c76e 100644
--- a/ui/gfx/render_text_harfbuzz.cc
+++ b/ui/gfx/render_text_harfbuzz.cc
@@ -1097,10 +1097,11 @@ void RenderTextHarfBuzz::ShapeRun(internal::TextRunHarfBuzz* run) {
   FontRenderParams best_render_params;
   size_t best_missing_glyphs = std::numeric_limits<size_t>::max();
 
-  if (CompareFamily(run, primary_font.GetFontName(),
-                    primary_font.GetFontRenderParams(),
-                    &best_family, &best_render_params, &best_missing_glyphs))
-    return;
+  for (const Font& font : font_list().GetFonts()) {
+    if (CompareFamily(run, font.GetFontName(), font.GetFontRenderParams(),
+                      &best_family, &best_render_params, &best_missing_glyphs))
+      return;
+  }
 
 #if defined(OS_WIN)
   Font uniscribe_font;