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

Unified Diff: chrome/browser/about_flags.cc

Issue 10825027: Turn on the fixed position stacking context flag by default (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 5 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
« no previous file with comments | « no previous file | content/browser/web_contents/web_contents_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/about_flags.cc
diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc
index ce427d395847b0cf979c498076b6ac94a195ec3f..9ce7df913027097f555cd739b4d748e023c54d3d 100644
--- a/chrome/browser/about_flags.cc
+++ b/chrome/browser/about_flags.cc
@@ -106,6 +106,14 @@ const Experiment::Choice kOmniboxInlineHistoryQuickProviderChoices[] = {
switches::kOmniboxInlineHistoryQuickProviderProhibited }
};
+const Experiment::Choice kFixedPositionCreatesStackingContextChoices[] = {
+ { IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" },
+ { IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED,
+ switches::kEnableFixedPositionCreatesStackingContext, ""},
+ { IDS_GENERIC_EXPERIMENT_CHOICE_DISABLED,
+ switches::kDisableFixedPositionCreatesStackingContext, ""}
+};
+
const Experiment::Choice kForceCompositingModeChoices[] = {
{ IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" },
{ IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED,
@@ -304,7 +312,7 @@ const Experiment kExperiments[] = {
IDS_FLAGS_FIXED_POSITION_CREATES_STACKING_CONTEXT_NAME,
IDS_FLAGS_FIXED_POSITION_CREATES_STACKING_CONTEXT_DESCRIPTION,
kOsAll,
- SINGLE_VALUE_TYPE(switches::kFixedPositionCreatesStackingContext)
+ MULTI_VALUE_TYPE(kFixedPositionCreatesStackingContextChoices)
},
// TODO(dspringer): When NaCl is on by default, remove this flag entry.
{
« no previous file with comments | « no previous file | content/browser/web_contents/web_contents_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698