Making structure for ContentSettings and its corresponding strings.

chrome/browser/extensions/api/content_settings/content_settings_helpers.cc

 // 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.
 
 #include "chrome/browser/extensions/api/content_settings/content_settings_helpers.h"
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "components/content_settings/core/browser/website_settings_info.h"
 #include "components/content_settings/core/browser/website_settings_registry.h"
 #include "content/public/common/url_constants.h"
 #include "extensions/common/url_pattern.h"
 
 namespace {
 
 const char kNoPathWildcardsError[] =
     "Path wildcards in file URL patterns are not allowed.";
 const char kNoPathsError[] = "Specific paths are not allowed.";
 const char kInvalidPatternError[] = "The pattern \"*\" is invalid.";
 
-const char* const kContentSettingNames[] = {
-  "default",
-  "allow",
-  "block",
-  "ask",
-  "session_only",
-  "detect_important_content"
-};
-static_assert(arraysize(kContentSettingNames) <=
-              CONTENT_SETTING_NUM_SETTINGS,
-              "kContentSettingNames has an unexpected number of elements");
-
 // TODO(bauerb): Move this someplace where it can be reused.
 std::string GetDefaultPort(const std::string& scheme) {
   if (scheme == url::kHttpScheme)
     return "80";
   if (scheme == url::kHttpsScheme)
     return "443";
   NOTREACHED();
   return std::string();
 }
 
@@ skipping 71 matching lines @@
 }
 
 std::string ContentSettingsTypeToString(ContentSettingsType type) {
   return content_settings::WebsiteSettingsRegistry::GetInstance()
       ->Get(type)
       ->name();
 }
 
 bool StringToContentSetting(const std::string& setting_str,
                             ContentSetting* setting) {
-  for (size_t type = 0; type < arraysize(kContentSettingNames); ++type) {
-    if (setting_str == kContentSettingNames[type]) {
-      *setting = static_cast<ContentSetting>(type);
-      return true;
-    }
-  }
-  return false;
-}
-
-const char* ContentSettingToString(ContentSetting setting) {
-  size_t index = static_cast<size_t>(setting);
-  DCHECK_LT(index, arraysize(kContentSettingNames));
-  return kContentSettingNames[index];
+  *setting = content_settings::ContentSettingFromString(setting_str);

[Bernhard Bauer, 2015/10/06 15:37:25]

This is still incorrect. ContentSettingFromString will DCHECK if an invalid
string is passed, while the contract of this method requires it to return false
-- which will never happen because the DCHECK will crash beforehand. See also my
other comment for what we should do.

[Deepak, 2015/10/07 04:39:43]

Done.

+  return *setting != CONTENT_SETTING_DEFAULT;
 }
 
 }  // namespace content_settings_helpers
 }  // namespace extensions