Eliminate one copy of replace_char() function.

tests/StringTest.cpp

 /*
  * Copyright 2011 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include <stdarg.h>
 #include <stdio.h>
 #include "SkString.h"
@@ skipping 166 matching lines @@
     char buffer [40];
     memset(buffer, 'a', 40);
     REPORTER_ASSERT(reporter, buffer[18] == 'a');
     REPORTER_ASSERT(reporter, buffer[19] == 'a');
     REPORTER_ASSERT(reporter, buffer[20] == 'a');
     printfAnalog(buffer, 20, "%30d", 0);
     REPORTER_ASSERT(reporter, buffer[18] == ' ');
     REPORTER_ASSERT(reporter, buffer[19] == 0);
     REPORTER_ASSERT(reporter, buffer[20] == 'a');
 
+    SkString skia_2d("skia_2d");
+    SkString skia2d("skia-2d");
+    skia_2d.replace('_', '-');
+    REPORTER_ASSERT(reporter, skia_2d == skia2d);
+
+    SkString meat("meat");
+    SkString meet("meet");
+    meat.replace('a', 'e');
+    REPORTER_ASSERT(reporter, meat == meet);
+
+    SkString hangul("\xed\x95\x9c\xea\xb5\xad\xec\x96\xb4");

[tfarina, 2014/10/18 03:42:27]

Mike, could you take another look? Let me know if this is OK.

+    SkString hangul2("\xb4\x95\x9c\xea\xb5\xad\xec\x96\xb4");
+    hangul.replace('\xed', '\xb4');
+    REPORTER_ASSERT(reporter, hangul == hangul2);
 }
 
 DEF_TEST(String_SkStrSplit, r) {
     SkTArray<SkString> results;
 
     SkStrSplit("a-_b_c-dee--f-_-_-g-", "-_", &results);
     REPORTER_ASSERT(r, results.count() == 6);
     REPORTER_ASSERT(r, results[0].equals("a"));
     REPORTER_ASSERT(r, results[1].equals("b"));
     REPORTER_ASSERT(r, results[2].equals("c"));
     REPORTER_ASSERT(r, results[3].equals("dee"));
     REPORTER_ASSERT(r, results[4].equals("f"));
     REPORTER_ASSERT(r, results[5].equals("g"));
 }