Omnibox: Create DemoteByType Experiment

base/strings/string_split.h

 // Copyright (c) 2012 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.
 
 #ifndef BASE_STRINGS_STRING_SPLIT_H_
 #define BASE_STRINGS_STRING_SPLIT_H_
 
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 18 matching lines @@
 // Note: |c| must be in the ASCII range.
 BASE_EXPORT void SplitString(const std::string& str,
                              char c,
                              std::vector<std::string>* r);
 
 BASE_EXPORT bool SplitStringIntoKeyValues(const std::string& line,
                                           char key_value_delimiter,
                                           std::string* key,
                                           std::vector<std::string>* values);
 
+typedef std::vector<std::pair<std::string, std::string> > KeyValuePairs;

[darin (slow to review), 2013/08/08 04:40:55]

If I saw base::KeyValuePairs in code, it wouldn't be very obvious that it was
really just a vector of string pairs.  It seems like base::StringPairs would be
a much more descriptive name for this type.  It just so happens that the
SplitStringIntoKeyValuePairs function returns data of that type.  Seems like an
improvement, right?

[Mark P, 2013/08/08 13:16:02]

Yes, that's an improvement.  Done.

+
 BASE_EXPORT bool SplitStringIntoKeyValuePairs(
     const std::string& line,
     char key_value_delimiter,
     char key_value_pair_delimiter,
-    std::vector<std::pair<std::string, std::string> >* kv_pairs);
+    KeyValuePairs* kv_pairs);

[darin (slow to review), 2013/08/08 04:40:55]

nit: while you are here, perhaps kv_pairs can be named key_value_pairs for
consistency with other params?

[Mark P, 2013/08/08 13:16:02]

Okay.  Done.

 
 // The same as SplitString, but use a substring delimiter instead of a char.
 BASE_EXPORT void SplitStringUsingSubstr(const string16& str,
                                         const string16& s,
                                         std::vector<string16>* r);
 BASE_EXPORT void SplitStringUsingSubstr(const std::string& str,
                                         const std::string& s,
                                         std::vector<std::string>* r);
 
 // The same as SplitString, but don't trim white space.
@@ skipping 18 matching lines @@
 // characters defined by HTML 5). Each contiguous block of non-whitespace
 // characters is added to result.
 BASE_EXPORT void SplitStringAlongWhitespace(const string16& str,
                                             std::vector<string16>* result);
 BASE_EXPORT void SplitStringAlongWhitespace(const std::string& str,
                                             std::vector<std::string>* result);
 
 }  // namespace base
 
 #endif  // BASE_STRINGS_STRING_SPLIT_H_