Add support for joining strings on strings rather than just chars.

BUG=


Review URL: https://chromiumcodereview.appspot.com/10828217

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153568 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 13 insertions, 4 deletions

diff --git a/base/string_util.cc b/base/string_util.cc
index 2eecb79..9e3422d 100644
--- a/base/string_util.cc
+++ b/base/string_util.cc
@@ -750,8 +750,7 @@ size_t Tokenize(const base::StringPiece& str,
 }
 
 template<typename STR>
-static STR JoinStringT(const std::vector<STR>& parts,
-                       typename STR::value_type sep) {
+static STR JoinStringT(const std::vector<STR>& parts, const STR& sep) {
   if (parts.empty())
     return STR();
 
@@ -768,11 +767,21 @@ static STR JoinStringT(const std::vector<STR>& parts,
 }
 
 std::string JoinString(const std::vector<std::string>& parts, char sep) {
-  return JoinStringT(parts, sep);
+  return JoinStringT(parts, std::string(1, sep));
 }
 
 string16 JoinString(const std::vector<string16>& parts, char16 sep) {
-  return JoinStringT(parts, sep);
+  return JoinStringT(parts, string16(1, sep));
+}
+
+std::string JoinString(const std::vector<std::string>& parts,
+                       const std::string& separator) {
+  return JoinStringT(parts, separator);
+}
+
+string16 JoinString(const std::vector<string16>& parts,
+                    const string16& separator) {
+  return JoinStringT(parts, separator);
 }
 
 template<class FormatStringType, class OutStringType>