Allow component installers to specify a map of installer attributes.

components/update_client/utils.cc

 // Copyright 2014 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 "components/update_client/utils.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <algorithm>
@@ skipping 221 matching lines @@
 bool IsValidBrand(const std::string& brand) {
   const size_t kMaxBrandSize = 4;
   if (!brand.empty() && brand.size() != kMaxBrandSize)
     return false;
 
   return std::find_if_not(brand.begin(), brand.end(), [](char ch) {
            return base::IsAsciiAlpha(ch);
          }) == brand.end();
 }
 
-bool IsValidAp(const std::string& ap) {
-  const size_t kMaxApSize = 256;
-  if (ap.size() > kMaxApSize)
+// Helper function.
+// Returns true if |part| matches the expression
+// ^[<special_chars>a-zA-Z0-9]{min_length,max_length}$
+bool IsValidInstallerAttributePart(const std::string& part,
+                                   const std::string& special_chars,
+                                   size_t min_length,
+                                   size_t max_length) {
+  if (part.size() < min_length || part.size() > max_length)
     return false;
 
-  return std::find_if_not(ap.begin(), ap.end(), [](char ch) {
+  return std::find_if_not(part.begin(), part.end(), [&special_chars](char ch) {
            if (base::IsAsciiAlpha(ch) || base::IsAsciiDigit(ch))
              return true;
 
-           const char kSpecialChars[] = "+-_=";
-           for (auto c : kSpecialChars) {
+           for (auto c : special_chars) {
              if (c == ch)
                return true;
            }
 
            return false;
-         }) == ap.end();
+         }) == part.end();
+}
+
+// Returns true if the |name| parameter matches ^[-_a-zA-Z0-9]{1,256}$ .
+bool IsValidInstallerAttributeName(const std::string& name) {
+  return IsValidInstallerAttributePart(name, "-_", 1, 256);
+}
+
+// Returns true if the |value| parameter matches ^[-.,;+_=a-zA-Z0-9]{0,256}$ .
+bool IsValidInstallerAttributeValue(const std::string& value) {
+  return IsValidInstallerAttributePart(value, "-.,;+_=", 0, 256);
+}
+
+bool IsValidInstallerAttribute(const InstallerAttribute& attr) {
+  return IsValidInstallerAttributeName(attr.first) &&
+         IsValidInstallerAttributeValue(attr.second);
 }
 
 void RemoveUnsecureUrls(std::vector<GURL>* urls) {
   DCHECK(urls);
   urls->erase(std::remove_if(
                   urls->begin(), urls->end(),
                   [](const GURL& url) { return !url.SchemeIsCryptographic(); }),
               urls->end());
 }
 
 }  // namespace update_client