Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 { | 1 { |
| 2 // Defines properties which are available on the Settings object. | 2 // Defines properties which are available on the Settings object. |
| 3 // | 3 // |
| 4 // Please think carefully before adding a new Setting. Some questions to | 4 // Please think carefully before adding a new Setting. Some questions to |
| 5 // consider are: | 5 // consider are: |
| 6 // - Should this be a RuntimeEnabledFeature instead? Settings are for things | 6 // - Should this be a RuntimeEnabledFeature instead? Settings are for things |
| 7 // which we support either values of at runtime. Features are set at render er | 7 // which we support either values of at runtime. Features are set at render er |
| 8 // process startup and are never changed. Features also tend to be set to a | 8 // process startup and are never changed. Features also tend to be set to a |
| 9 // value based on the platform or the stability of the code in question, whe re | 9 // value based on the platform or the stability of the code in question, whe re |
| 10 // as settings both codepaths need to be stable. | 10 // as settings both codepaths need to be stable. |
| (...skipping 212 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 223 { | 223 { |
| 224 name: "presentationRequiresUserGesture", | 224 name: "presentationRequiresUserGesture", |
| 225 initial: true, | 225 initial: true, |
| 226 }, | 226 }, |
| 227 | 227 |
| 228 { | 228 { |
| 229 name: "embeddedMediaExperienceEnabled", | 229 name: "embeddedMediaExperienceEnabled", |
| 230 initial: false, | 230 initial: false, |
| 231 }, | 231 }, |
| 232 | 232 |
| 233 { | 233 { |
|
Rick Byers
2017/05/23 21:36:36
nit: please add a comment that describes (as preci
| |
| 234 name: "pagePopupsSuppressed", | |
|
Rick Byers
2017/05/23 21:36:36
Nit: Prefer a positive name?
From a quick search o
| |
| 235 initial: false, | |
| 236 }, | |
| 237 | |
| 238 { | |
| 234 name: "scrollAnimatorEnabled", | 239 name: "scrollAnimatorEnabled", |
| 235 initial: true, | 240 initial: true, |
| 236 }, | 241 }, |
| 237 | 242 |
| 238 // Used to disable threaded, compositor scrolling for testing purposes. | 243 // Used to disable threaded, compositor scrolling for testing purposes. |
| 239 // crbug.com/410974 tracks removal once alternative solutions for selective | 244 // crbug.com/410974 tracks removal once alternative solutions for selective |
| 240 // main thread scrolling are supported. | 245 // main thread scrolling are supported. |
| 241 { | 246 { |
| 242 name: "threadedScrollingEnabled", | 247 name: "threadedScrollingEnabled", |
| 243 initial: true, | 248 initial: true, |
| (...skipping 675 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 919 }, | 924 }, |
| 920 | 925 |
| 921 // Defines the autoplay policy to use. | 926 // Defines the autoplay policy to use. |
| 922 { | 927 { |
| 923 name: "autoplayPolicy", | 928 name: "autoplayPolicy", |
| 924 type: "AutoplayPolicy::Type", | 929 type: "AutoplayPolicy::Type", |
| 925 initial: "AutoplayPolicy::Type::kNoUserGestureRequired", | 930 initial: "AutoplayPolicy::Type::kNoUserGestureRequired", |
| 926 }, | 931 }, |
| 927 ], | 932 ], |
| 928 } | 933 } |
| OLD | NEW |