[vr] Deny permission requests in VR mode

chrome/browser/permissions/permission_request_manager.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/permissions/permission_request_manager.h"
 
 #include <algorithm>
 
 #include "base/command_line.h"
 #include "base/feature_list.h"
 #include "base/metrics/user_metrics.h"
 #include "base/metrics/user_metrics_action.h"
 #include "build/build_config.h"
 #include "chrome/browser/permissions/permission_request.h"
 #include "chrome/browser/permissions/permission_uma_util.h"
 #include "chrome/browser/ui/permission_bubble/permission_prompt.h"
 #include "chrome/common/chrome_features.h"
 #include "chrome/common/chrome_switches.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/navigation_handle.h"
+#include "device/vr/features/features.h"
 #include "url/origin.h"
 
+#if BUILDFLAG(ENABLE_VR) && defined(OS_ANDROID)
+#include "chrome/browser/android/vr_shell/vr_tab_helper.h"
+#endif  // BUILDFLAG(ENABLE_VR) && defined(OS_ANDROID)
+
 namespace {
 
 class CancelledRequest : public PermissionRequest {
  public:
   explicit CancelledRequest(PermissionRequest* cancelled)
       : icon_(cancelled->GetIconId()),
 #if defined(OS_ANDROID)
         message_(cancelled->GetMessageText()),
 #endif
         message_fragment_(cancelled->GetMessageTextFragment()),
@@ skipping 96 matching lines @@
   for (PermissionRequest* request : queued_requests_)
     request->RequestFinished();
   for (const auto& entry : duplicate_requests_)
     entry.second->RequestFinished();
 }
 
 void PermissionRequestManager::AddRequest(PermissionRequest* request) {
   // TODO(tsergeant): change the UMA to no longer mention bubbles.
   base::RecordAction(base::UserMetricsAction("PermissionBubbleRequest"));
 
+  if (PermissionRequestsDisabled()) {
+    request->PermissionDenied();

[raymes, 2017/07/11 01:43:23]

nit: you may want to change this to Cancelled() since it wasn't an active denial
by the user.

[raymes, 2017/07/11 01:44:37]

Ignore this - after I thought through it more I think it's better to add a
separate function to PermissionRequest for this case.

+    request->RequestFinished();
+    return;
+  }
+
   // TODO(gbillock): is there a race between an early request on a
   // newly-navigated page and the to-be-cleaned-up requests on the previous
   // page? We should maybe listen to DidStartNavigationToPendingEntry (and
   // any other renderer-side nav initiations?). Double-check this for
   // correct behavior on interstitials -- we probably want to basically queue
   // any request for which GetVisibleURL != GetLastCommittedURL.
   const GURL& request_url_ = web_contents()->GetLastCommittedURL();
   bool is_main_frame = url::Origin(request_url_)
                            .IsSameOriginWith(url::Origin(request->GetOrigin()));
 
@@ skipping 377 matching lines @@
     case DENY_ALL:
       Deny();
       break;
     case DISMISS:
       Closing();
       break;
     case NONE:
       NOTREACHED();
   }
 }
+
+bool PermissionRequestManager::PermissionRequestsDisabled() const {
+#if BUILDFLAG(ENABLE_VR) && defined(OS_ANDROID)
+  return vr_shell::VrTabHelper::IsInVr(web_contents());
+#else
+  return false;
+#endif
+}