Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(18)

Unified Diff: components/content_settings/core/browser/plugins_field_trial.h

Issue 1084123002: Make DETECT_IMPORTANT_CONTENT the default Plugins content setting based (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: gyp shuffle for ios Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/content_settings/core/browser/plugins_field_trial.h
diff --git a/chrome/browser/plugins/plugins_field_trial.h b/components/content_settings/core/browser/plugins_field_trial.h
similarity index 57%
rename from chrome/browser/plugins/plugins_field_trial.h
rename to components/content_settings/core/browser/plugins_field_trial.h
index 3b4d1c348c70e46f841aef5451518ef7743ec62e..ccc39dfb42ab5c7cc9afd9970aa3e93bd0c6fc1f 100644
--- a/chrome/browser/plugins/plugins_field_trial.h
+++ b/components/content_settings/core/browser/plugins_field_trial.h
@@ -2,27 +2,35 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CHROME_BROWSER_PLUGINS_PLUGINS_FIELD_TRIAL_H_
-#define CHROME_BROWSER_PLUGINS_PLUGINS_FIELD_TRIAL_H_
+#ifndef COMPONENTS_CONTENT_SETTINGS_CORE_BROWSER_CONTENT_SETTINGS_H_
+#define COMPONENTS_CONTENT_SETTINGS_CORE_BROWSER_CONTENT_SETTINGS_H_
#include "base/macros.h"
#include "components/content_settings/core/common/content_settings.h"
#include "components/content_settings/core/common/content_settings_types.h"
+namespace content_settings {
+
// This class manages the Plugins field trials.
class PluginsFieldTrial {
public:
+ static const char kFieldTrialName[];
+
// 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 the Plugin Power Saver feature is forced on, overriding
- // the user's Content Settings.
- static bool IsForcePluginPowerSaverEnabled();
+ // Returns true if the Plugin Power Saver feature is enabled.
+ static bool IsPluginPowerSaverEnabled();
+
+ // Get the default plugins content setting based on field trials/flags.
+ static ContentSetting GetDefaultPluginsContentSetting();
private:
DISALLOW_IMPLICIT_CONSTRUCTORS(PluginsFieldTrial);
};
-#endif // CHROME_BROWSER_PLUGINS_PLUGINS_FIELD_TRIAL_H_
+} // namespace content_settings
+
+#endif // COMPONENTS_CONTENT_SETTINGS_CORE_BROWSER_CONTENT_SETTINGS_H_

Powered by Google App Engine
This is Rietveld 408576698