Enforce SafetyMode for YouTube if prefs::kForceSafeSearch is on.

base/strings/string_util.cc

Index: base/strings/string_util.cc
diff --git a/base/strings/string_util.cc b/base/strings/string_util.cc
index e64b95f7635dc1ddeec4d8f0d97b4d92c0d9b2e3..4840699cf0455525a4ba9e1d607878180cd38aae 100644
--- a/base/strings/string_util.cc
+++ b/base/strings/string_util.cc
@@ -617,6 +617,30 @@ string16 JoinString(const std::vector<string16>& parts,
   return JoinStringT(parts, separator);
 }
 
+static std::string JoinStringKeyValuePair(

[erikwright (departed), 2014/07/10 14:26:24]

These methods are only used from a single file AFAICT.

They seem relatively specialized (i.e., many application might require escaping,
etc).

There also are no tests for them.

Personally I suggest just moving them to the anonymous namespace in the file
that uses them. Otherwise tests would be required.

In any case I would expect the owners of base to object to their addition with a
single client. OTOH, if you can find some existing code that could be replaced
with this, more power to you!

[Marc Treib, 2014/07/10 15:40:39]

I moved these functions into safe_search_util.cc.

+    const StringPairs::value_type& key_value,
+    char delimiter) {
+  return key_value.first + delimiter + key_value.second;
+}
+
+std::string JoinStringKeyValuePairs(const StringPairs& pairs,
+                                    char key_value_delimiter,
+                                    char key_value_pair_delimiter) {
+  if (pairs.empty())
+    return std::string();
+
+  StringPairs::const_iterator it = pairs.begin();
+  std::string result = JoinStringKeyValuePair(*it, key_value_delimiter);
+  ++it;
+
+  for (; it != pairs.end(); ++it) {
+    result += key_value_pair_delimiter;
+    result += JoinStringKeyValuePair(*it, key_value_delimiter);
+  }
+
+  return result;
+}
+
 template<class FormatStringType, class OutStringType>
 OutStringType DoReplaceStringPlaceholders(const FormatStringType& format_string,
     const std::vector<OutStringType>& subst, std::vector<size_t>* offsets) {