Move unusual character block logic to a helper function

ui/gfx/render_text_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/gfx/render_text.h"
 
 #include <algorithm>
 
 #include "base/format_macros.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 1885 matching lines @@
 
     const Range second_expected_range = kTestStrings[i].second_line_char_range;
     ASSERT_EQ(second_expected_range.IsValid() ? 1U : 0U,
               render_text->lines_[1].segments.size());
     if (second_expected_range.IsValid())
       EXPECT_EQ(second_expected_range,
                 render_text->lines_[1].segments[0].char_range);
   }
 }
 
-
 TEST_F(RenderTextTest, Win_BreakRunsByUnicodeBlocks) {
   scoped_ptr<RenderTextWin> render_text(
       static_cast<RenderTextWin*>(RenderText::CreateInstance()));
 
   // The '\x25B6' "play character" should break runs. http://crbug.com/278913
   render_text->SetText(WideToUTF16(L"x\x25B6y"));
   render_text->EnsureLayout();
   ASSERT_EQ(3U, render_text->runs_.size());
   EXPECT_EQ(Range(0, 1), render_text->runs_[0]->range);
   EXPECT_EQ(Range(1, 2), render_text->runs_[1]->range);
@@ skipping 50 matching lines @@
     std::copy(cases[i].glyph_to_char, cases[i].glyph_to_char + 4,
               run.glyph_to_char.get());
     run.is_rtl = cases[i].is_rtl;
     for (size_t j = 0; j < 4; ++j) {
       SCOPED_TRACE(base::StringPrintf("Case %" PRIuS ", char %" PRIuS, i, j));
       EXPECT_EQ(cases[i].char_to_glyph_expected[j], run.CharToGlyph(j));
       EXPECT_EQ(cases[i].char_range_to_glyph_range_expected[j],
                 run.CharRangeToGlyphRange(Range(j, j + 1)));
     }
   }
-
 }
 
 TEST_F(RenderTextTest, HarfBuzz_RunDirection) {
   RenderTextHarfBuzz render_text;
   const base::string16 mixed =
       WideToUTF16(L"\x05D0\x05D1" L"1234" L"\x05D2\x05D3");
   render_text.SetText(mixed);
   render_text.EnsureLayout();
   ASSERT_EQ(3U, render_text.runs_.size());
   EXPECT_TRUE(render_text.runs_[0]->is_rtl);
   EXPECT_FALSE(render_text.runs_[1]->is_rtl);
   EXPECT_TRUE(render_text.runs_[2]->is_rtl);
 }
 
+TEST_F(RenderTextTest, HarfBuzz_BreakRunsByUnicodeBlocks) {
+  RenderTextHarfBuzz render_text;
+
+  // The '\x25B6' "play character" should break runs. http://crbug.com/278913
+  render_text.SetText(WideToUTF16(L"x\x25B6y"));
+  render_text.EnsureLayout();
+  ASSERT_EQ(3U, render_text.runs_.size());
+  EXPECT_EQ(Range(0, 1), render_text.runs_[0]->range);
+  EXPECT_EQ(Range(1, 2), render_text.runs_[1]->range);
+  EXPECT_EQ(Range(2, 3), render_text.runs_[2]->range);
+
+  render_text.SetText(WideToUTF16(L"x \x25B6 y"));
+  render_text.EnsureLayout();
+  ASSERT_EQ(3U, render_text.runs_.size());
+  EXPECT_EQ(Range(0, 2), render_text.runs_[0]->range);
+  EXPECT_EQ(Range(2, 3), render_text.runs_[1]->range);
+  EXPECT_EQ(Range(3, 5), render_text.runs_[2]->range);
+}
+
 }  // namespace gfx