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

Side by Side Diff: chrome/browser/plugins/plugins_field_trial.cc

Issue 1033103002: Plugin Power Saver: Overhaul plugin-power-saver flags. Show in UI. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move plugins field trial stuff to chrome/ 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/plugins/plugins_field_trial.h"
6
7 #include "base/command_line.h"
8 #include "base/metrics/field_trial.h"
9 #include "chrome/common/chrome_switches.h"
10
11 ContentSetting PluginsFieldTrial::EffectiveContentSetting(
12 ContentSettingsType type,
13 ContentSetting setting) {
14 if (type != CONTENT_SETTINGS_TYPE_PLUGINS)
15 return setting;
16
17 // For Plugins, ASK is obsolete. Show as BLOCK to reflect actual behavior.
18 if (setting == ContentSetting::CONTENT_SETTING_ASK)
19 return ContentSetting::CONTENT_SETTING_BLOCK;
20
21 // For Plugins, allow flag to override displayed content setting.
22 if (setting == ContentSetting::CONTENT_SETTING_ALLOW &&
23 IsForcePluginPowerSaverEnabled()) {
24 return ContentSetting::CONTENT_SETTING_DETECT_IMPORTANT_CONTENT;
25 }
26
27 return setting;
28 }
29
30 bool PluginsFieldTrial::IsForcePluginPowerSaverEnabled() {
31 const base::CommandLine* cl = base::CommandLine::ForCurrentProcess();
32 if (cl->HasSwitch(switches::kDisablePluginPowerSaver))
33 return false;
34 if (cl->HasSwitch(switches::kEnablePluginPowerSaver))
35 return true;
36
37 return base::FieldTrialList::FindFullName("ForcePluginPowerSaver") ==
38 "Dogfood";
Lei Zhang 2015/03/31 20:27:38 4 space indent from the front? Maybe "!= Disabled
tommycli 2015/03/31 20:34:56 Done.
39 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698