Disable color/date chooser in VR mode via WebPreferences

third_party/WebKit/Source/core/frame/Settings.json5

Index: third_party/WebKit/Source/core/frame/Settings.json5
diff --git a/third_party/WebKit/Source/core/frame/Settings.json5 b/third_party/WebKit/Source/core/frame/Settings.json5
index 84b492711ee8d6d63ea75f2bb147504cf877c21c..bde5a955c53e6543496ec78a8c02fd9dd8afea48 100644
--- a/third_party/WebKit/Source/core/frame/Settings.json5
+++ b/third_party/WebKit/Source/core/frame/Settings.json5
@@ -231,6 +231,11 @@
     },
 
     {

[Rick Byers, 2017/05/23 21:36:36]

nit: please add a comment that describes (as precisely as possible) what exactly
is being enabled/disabled.  I think "Page Popup" is a bit of an overloaded term.
 If you can qualify specifically what properties makes it so that VR needs to
disable this, that would make things more clear.  It's probably also useful to
specifically describe the current use case in the comments ("used by VR mode" or
whatever).

+      name: "pagePopupsSuppressed",

[Rick Byers, 2017/05/23 21:36:36]

Nit: Prefer a positive name?
From a quick search of other settings it seems we prefer "fooEnabled" settings
that default to true over "fooSuppressed" that default to false.  Eg.
"supportsMultipleWindows".  Is there a reason this case should be special in
using the inverted "suppressed" semantics?

+      initial: false,
+    },
+
+    {
       name: "scrollAnimatorEnabled",
       initial: true,
     },