VR: Disable select element in VR mode via WebPreferences

third_party/WebKit/Source/web/ChromeClientImpl.cpp

Index: third_party/WebKit/Source/web/ChromeClientImpl.cpp
diff --git a/third_party/WebKit/Source/web/ChromeClientImpl.cpp b/third_party/WebKit/Source/web/ChromeClientImpl.cpp
index c530e82bde81dccf45383e65a34b7626352b1f6f..316410d15e8d993b5422b6898f2923b4251959b7 100644
--- a/third_party/WebKit/Source/web/ChromeClientImpl.cpp
+++ b/third_party/WebKit/Source/web/ChromeClientImpl.cpp
@@ -932,6 +932,10 @@ bool ChromeClientImpl::HasOpenedPopup() const {
 PopupMenu* ChromeClientImpl::OpenPopupMenu(LocalFrame& frame,
                                            HTMLSelectElement& select) {
   NotifyPopupOpeningObservers();
+
+  if (frame.GetDocument()->GetSettings()->GetPagePopupsSuppressed())

[Rick Byers, 2017/05/25 21:06:17]

nit: in vollick's previous CL he bailed out before a
'NotifyPopupOpeningObservers' call.  I don't know the implications of doing this
before or after, but I think we should at least be consistent.

[asimjour, 2017/05/25 22:47:32]

Done.

+    return nullptr;
+
   if (WebViewBase::UseExternalPopupMenus())
     return new ExternalPopupMenu(frame, select, *web_view_);