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

Side by Side Diff: third_party/WebKit/Source/core/frame/Settings.json5

Issue 2885033007: Disable color/date chooser in VR mode via WebPreferences (Closed)
Patch Set: date time chooser Created 3 years, 7 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
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
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
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 }
OLDNEW
« no previous file with comments | « content/renderer/render_view_impl.cc ('k') | third_party/WebKit/Source/core/html/forms/ColorChooserClient.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698