Added StringPiece overloads for base::JoinString.

chrome/browser/webshare/share_service_impl.cc

Index: chrome/browser/webshare/share_service_impl.cc
diff --git a/chrome/browser/webshare/share_service_impl.cc b/chrome/browser/webshare/share_service_impl.cc
index 281a082040090daaa4e4e2d46c6ce8d4dc2d0fba..712fcc6be8361974739b6e421f49b223f95b0021 100644
--- a/chrome/browser/webshare/share_service_impl.cc
+++ b/chrome/browser/webshare/share_service_impl.cc
@@ -29,23 +29,6 @@ bool IsIdentifier(char c) {
   return base::IsAsciiAlpha(c) || base::IsAsciiDigit(c) || c == '-' || c == '_';
 }
 
-// Joins a std::vector<base::StringPiece> into a single std::string.
-// TODO(constantina): Implement a base::JoinString() that takes StringPieces.
-// i.e. move this to base/strings/string_util.h, and thoroughly test.
-std::string JoinString(const std::vector<base::StringPiece>& pieces) {
-  size_t total_size = 0;
-  for (const auto& piece : pieces) {
-    total_size += piece.size();
-  }
-  std::string joined_pieces;
-  joined_pieces.reserve(total_size);
-
-  for (const auto& piece : pieces) {
-    piece.AppendToString(&joined_pieces);
-  }
-  return joined_pieces;
-}
-
 }  // namespace
 
 ShareServiceImpl::ShareServiceImpl() = default;
@@ -115,7 +98,7 @@ bool ShareServiceImpl::ReplacePlaceholders(base::StringPiece url_template,
   split_template.push_back(url_template.substr(
       start_index_to_copy, url_template.size() - start_index_to_copy));
 
-  *url_template_filled = JoinString(split_template);
+  *url_template_filled = base::JoinString(split_template, "");
   return true;
 }