[HBD] Gate the advertising of Flash on Site Engagement.

chrome/browser/plugins/plugins_field_trial.h

Index: chrome/browser/plugins/plugins_field_trial.h
diff --git a/chrome/browser/plugins/plugins_field_trial.h b/chrome/browser/plugins/plugins_field_trial.h
index ba106604dbd757ba8db9f98c4720d7ac83909b81..4053af956b03ce2eace02c358f8691ff3be83b9b 100644
--- a/chrome/browser/plugins/plugins_field_trial.h
+++ b/chrome/browser/plugins/plugins_field_trial.h
@@ -9,14 +9,26 @@
 #include "components/content_settings/core/common/content_settings.h"
 #include "components/content_settings/core/common/content_settings_types.h"
 
+class GURL;
+class HostContentSettingsMap;
+
 // This class manages the Plugins field trials.
 class PluginsFieldTrial {
  public:
+  // The name of the variations parameter used for updating the amount of site
+  // engagement required to permit Flash.
+  static const char* kSiteEngagementThresholdForFlashKey;
+
   // Returns the effective content setting for plugins. Passes non-plugin
   // content settings through without modification.
   static ContentSetting EffectiveContentSetting(ContentSettingsType type,
                                                 ContentSetting setting);
 
+  // Returns true if |url| has sufficient engagement to permit Flash based on
+  // the engagement in |settings|.
+  static bool HasSufficientEngagementForFlash(HostContentSettingsMap* settings,

[Bernhard Bauer, 2016/09/15 09:05:15]

I think it would make more sense to expose the engagement threshold from this
class rather than this method that just delegates to SiteEngagementService.

[dominickn, 2016/09/15 12:16:00]

Done.

+                                              const GURL& url);
+
  private:
   DISALLOW_IMPLICIT_CONSTRUCTORS(PluginsFieldTrial);
 };