Store base::Value in ContentSettingPatternSource instead of an enum

components/content_settings/core/common/content_settings.mojom

 // Copyright 2016 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.
 
 module content_settings.mojom;
 
+import "mojo/common/values.mojom";
+
 // This mirrors the C++ type in content_settings_pattern.h.
 struct PatternParts {
   // Lowercase string of the URL scheme to match. This string is empty if the
   // |is_scheme_wildcard| flag is set.
   string scheme;
 
   // True if the scheme wildcard is set.
   bool is_scheme_wildcard;
 
   // Normalized string that is either of the following:
@@ skipping 35 matching lines @@
   BLOCK,
   ASK,
   SESSION_ONLY,
   DETECT_IMPORTANT_CONTENT
 };
 
 // This mirrors the C++ type in content_settings.h.
 struct ContentSettingPatternSource {
   ContentSettingsPattern primary_pattern;
   ContentSettingsPattern secondary_pattern;
-  ContentSetting setting;
+  mojo.common.mojom.Value? setting_value;

[nasko, 2017/06/28 20:59:37]

Is the setting value really optional?

[tbansal1, 2017/06/29 15:57:53]

Done.

   string source;
   bool incognito;
 };
 
 // This mirrors the C++ type in content_settings.h.
 struct RendererContentSettingRules {
   array<ContentSettingPatternSource> image_rules;
   array<ContentSettingPatternSource> script_rules;
   array<ContentSettingPatternSource> autoplay_rules;
 };