Added StringPiece overloads for base::JoinString.

base/strings/string_util_unittest.cc

 // Copyright 2013 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 "base/strings/string_util.h"
 
 #include <math.h>
 #include <stdarg.h>
 #include <stddef.h>
 #include <stdint.h>
@@ skipping 689 matching lines @@
   parts.push_back(ASCIIToUTF16("b"));
   parts.push_back(ASCIIToUTF16("c"));
   EXPECT_EQ(ASCIIToUTF16("a, b, c"), JoinString(parts, separator));
 
   parts.push_back(ASCIIToUTF16(""));
   EXPECT_EQ(ASCIIToUTF16("a, b, c, "), JoinString(parts, separator));
   parts.push_back(ASCIIToUTF16(" "));
   EXPECT_EQ(ASCIIToUTF16("a|b|c|| "), JoinString(parts, ASCIIToUTF16("|")));
 }
 
+TEST(StringUtilTest, JoinStringPiece) {
+  std::string separator(", ");
+  std::vector<base::StringPiece> parts;
+  EXPECT_EQ(base::StringPiece(), JoinStringPiece(parts, separator));
+
+  parts.push_back("a");
+  EXPECT_EQ("a", JoinStringPiece(parts, separator));
+
+  parts.push_back("b");
+  parts.push_back("c");
+  EXPECT_EQ("a, b, c", JoinStringPiece(parts, separator));
+
+  parts.push_back(base::StringPiece());
+  EXPECT_EQ("a, b, c, ", JoinStringPiece(parts, separator));
+  parts.push_back(" ");
+  EXPECT_EQ("a|b|c|| ", JoinStringPiece(parts, "|"));
+}
+
+TEST(StringUtilTest, JoinStringPiece16) {
+  string16 separator = ASCIIToUTF16(", ");
+  std::vector<base::StringPiece16> parts;
+  EXPECT_EQ(base::StringPiece16(), JoinStringPiece(parts, separator));
+
+  const string16 kA = ASCIIToUTF16("a");
+  parts.push_back(kA);
+  EXPECT_EQ(ASCIIToUTF16("a"), JoinStringPiece(parts, separator));
+
+  const string16 kB = ASCIIToUTF16("b");
+  parts.push_back(kB);
+  const string16 kC = ASCIIToUTF16("c");
+  parts.push_back(kC);
+  EXPECT_EQ(ASCIIToUTF16("a, b, c"), JoinStringPiece(parts, separator));
+
+  parts.push_back(base::StringPiece16());
+  EXPECT_EQ(ASCIIToUTF16("a, b, c, "), JoinStringPiece(parts, separator));
+  const string16 kSpace = ASCIIToUTF16(" ");
+  parts.push_back(kSpace);
+  EXPECT_EQ(ASCIIToUTF16("a|b|c|| "),
+            JoinStringPiece(parts, ASCIIToUTF16("|")));
+}
+
 TEST(StringUtilTest, StartsWith) {
   EXPECT_TRUE(StartsWith("javascript:url", "javascript",
                          base::CompareCase::SENSITIVE));
   EXPECT_FALSE(StartsWith("JavaScript:url", "javascript",
                           base::CompareCase::SENSITIVE));
   EXPECT_TRUE(StartsWith("javascript:url", "javascript",
                          base::CompareCase::INSENSITIVE_ASCII));
   EXPECT_TRUE(StartsWith("JavaScript:url", "javascript",
                          base::CompareCase::INSENSITIVE_ASCII));
   EXPECT_FALSE(StartsWith("java", "javascript", base::CompareCase::SENSITIVE));
@@ skipping 401 matching lines @@
   const std::string live = kLive;
   std::string dead = live;
   strncpy(WriteInto(&dead, 5), kDead, 4);
   EXPECT_EQ(kDead, dead);
   EXPECT_EQ(4u, dead.size());
   EXPECT_EQ(kLive, live);
   EXPECT_EQ(4u, live.size());
 }
 
 }  // namespace base