[RenderText] added test for AppendText

ui/gfx/render_text_unittest.cc

Index: ui/gfx/render_text_unittest.cc
diff --git a/ui/gfx/render_text_unittest.cc b/ui/gfx/render_text_unittest.cc
index 42825579ec6cb0b68fe701de431a2c1444d5c1f7..9627f7d8b0c2d34a940128d441577f812c205218 100644
--- a/ui/gfx/render_text_unittest.cc
+++ b/ui/gfx/render_text_unittest.cc
@@ -352,6 +352,41 @@ TEST_F(RenderTextTest, ApplyStyles) {
 #endif  // OS_MACOSX
 }
 
+TEST_F(RenderTextTest, AppendText) {
+  // Ensure AppendText maintains current text styles.
+  scoped_ptr<RenderText> render_text(RenderText::CreateInstance());
+  const SkColor color = SK_ColorRED;
+  render_text->SetColor(color);
+  render_text->SetBaselineStyle(SUPERSCRIPT);
+  render_text->SetStyle(BOLD, true);
+  render_text->SetStyle(UNDERLINE, false);
+  const wchar_t* const cases[] = {L"Hello", L"", L" ", L"world"};
+  size_t total_length = 0;
+  for (size_t i = 0; i < arraysize(cases); ++i) {
+    EXPECT_TRUE(render_text->colors().EqualsValueForTesting(color));
+    EXPECT_TRUE(render_text->baselines().EqualsValueForTesting(SUPERSCRIPT));
+    EXPECT_TRUE(render_text->styles()[BOLD].EqualsValueForTesting(true));
+    EXPECT_TRUE(render_text->styles()[UNDERLINE].EqualsValueForTesting(false));
+    EXPECT_EQ(total_length, render_text->GetDisplayText().length());
+    base::string16 str = WideToUTF16(cases[i]);
+    render_text->AppendText(str);
+    total_length += str.length();
+    EXPECT_EQ(total_length, render_text->GetDisplayText().length());
+    std::vector<std::pair<size_t, SkColor>> expected_color;
+    expected_color.push_back(std::pair<size_t, SkColor>(0, color));

[msw, 2015/03/24 20:06:03]

None of this makes sense... Call ApplyStyle / ApplyColor / ApplyBaseLineStyle...

This test would be a lot simpler if you removed the loop and just did:
  // Set some text with ranged styles, colors, and baseline styles.
  set text ("abc")
  apply style (underline, true, 1-2) ... etc ...
  vector expected (0=false, 1=true, 2=false) ... etc ...
  expect style[underline]==expected ... etc ...
  // Append some text and ensure that ranged styles are not lost.
  append text ("def")
  expect text=="abcdef"
  expect style[underline]==expected .... etc ...

[dschuyler, 2015/03/24 20:54:08]

Done.

+    EXPECT_TRUE(render_text->colors().EqualsForTesting(expected_color));
+    std::vector<std::pair<size_t, bool>> expected_bold;
+    expected_bold.push_back(std::pair<size_t, bool>(0, true));
+    EXPECT_TRUE(render_text->styles()[BOLD].EqualsForTesting(expected_bold));
+    std::vector<std::pair<size_t, BaselineStyle>> expected_baseline;
+    expected_baseline.push_back(
+        std::pair<size_t, BaselineStyle>(0, SUPERSCRIPT));
+    EXPECT_TRUE(render_text->baselines().EqualsForTesting(expected_baseline));
+  }
+  const base::string16 result_string = ASCIIToUTF16("Hello world");
+  EXPECT_EQ(render_text->GetDisplayText(), result_string);
+}
+
 // TODO(asvitkine): Cursor movements tests disabled on Mac because RenderTextMac
 //                  does not implement this yet. http://crbug.com/131618
 #if !defined(OS_MACOSX)