Store base::Value in ContentSettingPatternSource instead of an enum

chrome/browser/engagement/site_engagement_service.cc

 // Copyright 2015 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/engagement/site_engagement_service.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <utility>
@@ skipping 57 matching lines @@
 
   // Fetch URLs of sites with engagement details stored.
   for (const auto& site : GetContentSettingsFromProfile(
            profile, CONTENT_SETTINGS_TYPE_SITE_ENGAGEMENT)) {
     urls.insert(GURL(site.primary_pattern.ToString()));
   }
 
   // Fetch URLs of sites for which notifications are allowed.
   for (const auto& site : GetContentSettingsFromProfile(
            profile, CONTENT_SETTINGS_TYPE_NOTIFICATIONS)) {
-    if (site.setting != CONTENT_SETTING_ALLOW)
+    if (site.GetContentSetting() != CONTENT_SETTING_ALLOW)
       continue;
     urls.insert(GURL(site.primary_pattern.ToString()));
   }
 
   return urls;
 }
 
 // Only accept a navigation event for engagement if it is one of:
 //  a. direct typed navigation
 //  b. clicking on an omnibox suggestion brought up by typing a keyword
@@ skipping 612 matching lines @@
     if (!engagement_score.last_shortcut_launch_time().is_null() &&
         engagement_score.last_shortcut_launch_time() > last_visit) {
       engagement_score.set_last_shortcut_launch_time(last_visit);
     }
 
     engagement_score.Commit();
   }
 
   SetLastEngagementTime(now);
 }