Store base::Value in ContentSettingPatternSource instead of an enum

components/content_settings/core/common/content_settings_utils.h

+// Copyright 2017 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 COMPONENTS_CONTENT_SETTINGS_CORE_COMMON_CONTENT_SETTINGS_UTILS_H_
+#define COMPONENTS_CONTENT_SETTINGS_CORE_COMMON_CONTENT_SETTINGS_UTILS_H_
+
+#include <memory>
+
+#include "components/content_settings/core/common/content_settings.h"
+
+namespace base {
+class Value;
+}
+
+namespace content_settings {
+
+// Converts |value| to |ContentSetting|.
+ContentSetting ValueToContentSetting(const base::Value* value);
+
+// Returns a base::Value representation of |setting| if |setting| is
+// a valid content setting. Otherwise, returns a nullptr.

[lazyboy, 2017/06/30 18:11:40]

drive by:
Isn't CONTENT_SETTING_DEFAULT a valid content setting? This function will return
nullptr in that case, whereas the comment calls it invalid.

[tbansal1, 2017/06/30 18:32:31]

DEFAULT is encoded as a null value. In the function that reverse maps in
components/content_settings/core/common/content_settings_utils.cc, there is also
a comment which says:
"Note that |CONTENT_SETTING_DEFAULT| is encoded as a NULL value, so it is not
allowed as an integer value."

+std::unique_ptr<base::Value> ContentSettingToValue(ContentSetting setting);
+
+}  // namespace content_settings
+
+#endif  // COMPONENTS_CONTENT_SETTINGS_CORE_COMMON_CONTENT_SETTINGS_UTILS_H_