RenderTextWin: Break runs at UBLOCK_MISCELLANEOUS_SYMBOLS.

ui/gfx/render_text_win.cc

Index: ui/gfx/render_text_win.cc
diff --git a/ui/gfx/render_text_win.cc b/ui/gfx/render_text_win.cc
index c808a74ca71e34dc198a4f318f5e55c8acf03f3a..c262e3cf657fa35c1f2acbafbd02ecb66feab947 100644
--- a/ui/gfx/render_text_win.cc
+++ b/ui/gfx/render_text_win.cc
@@ -918,7 +918,7 @@ void RenderTextWin::ItemizeLogicalText() {
         --run_break;
     }
 
-    // Break runs adjacent to UBLOCK_GEOMETRIC_SHAPES character substrings.
+    // Break runs adjacent to character substrings in certain code blocks.
     // This avoids using their fallback fonts for more characters than needed,
     // in cases like "\x25B6 Media Title", etc. http://crbug.com/278913
     if (run_break > run->range.start()) {
@@ -931,7 +931,9 @@ void RenderTextWin::ItemizeLogicalText() {
         const UBlockCode current_block_code = ublock_getCode(iter.get());
         if (current_block_code != first_block_code &&
             (current_block_code == UBLOCK_GEOMETRIC_SHAPES ||
-             first_block_code == UBLOCK_GEOMETRIC_SHAPES)) {
+             current_block_code == UBLOCK_MISCELLANEOUS_SYMBOLS ||
+             first_block_code == UBLOCK_GEOMETRIC_SHAPES ||
+             first_block_code == UBLOCK_MISCELLANEOUS_SYMBOLS)) {

[Alexei Svitkine (slow), 2013/11/04 20:59:15]

Since first_block_code doesn't change in the while loop, pull these two conds
out as a boolean outside the loop and just check that bool here.

[msw, 2013/11/04 22:22:15]

Done; plus I added a helper and refactored a little bit.

           run_break = run_start + iter.array_pos();
           break;
         }