Add PermissionPromptAndroid skeleton

chrome/browser/permissions/permission_context_base.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_context_base.h"
 
 #include <stddef.h>
 #include <utility>
 
 #include "base/callback.h"
 #include "base/logging.h"
 #include "base/strings/stringprintf.h"
 #include "build/build_config.h"
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
 #include "chrome/browser/permissions/permission_decision_auto_blocker.h"
 #include "chrome/browser/permissions/permission_request.h"
 #include "chrome/browser/permissions/permission_request_id.h"
+#include "chrome/browser/permissions/permission_request_impl.h"
+#include "chrome/browser/permissions/permission_request_manager.h"
 #include "chrome/browser/permissions/permission_uma_util.h"
 #include "chrome/browser/permissions/permission_util.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/pref_names.h"
 #include "components/content_settings/core/browser/host_content_settings_map.h"
 #include "components/content_settings/core/browser/website_settings_registry.h"
 #include "components/prefs/pref_service.h"
 #include "components/variations/variations_associated_data.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/origin_util.h"
 #include "url/gurl.h"
 
 #if defined(OS_ANDROID)
 #include "chrome/browser/permissions/permission_queue_controller.h"
-#else
-#include "chrome/browser/permissions/permission_request_impl.h"
-#include "chrome/browser/permissions/permission_request_manager.h"
 #endif
 
 // static
 const char PermissionContextBase::kPermissionsKillSwitchFieldStudy[] =
     "PermissionsKillSwitch";
 // static
 const char PermissionContextBase::kPermissionsKillSwitchBlockedValue[] =
     "blocked";
 
 PermissionContextBase::PermissionContextBase(
@@ skipping 120 matching lines @@
 
 void PermissionContextBase::DecidePermission(
     content::WebContents* web_contents,
     const PermissionRequestID& id,
     const GURL& requesting_origin,
     const GURL& embedding_origin,
     bool user_gesture,
     const BrowserPermissionCallback& callback) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
-#if !defined(OS_ANDROID)
-  PermissionRequestManager* permission_request_manager =
-      PermissionRequestManager::FromWebContents(web_contents);
-  // TODO(felt): sometimes |permission_request_manager| is null. This check is
-  // meant to prevent crashes. See crbug.com/457091.
-  if (!permission_request_manager)
-    return;
-  std::unique_ptr<PermissionRequest> request_ptr(new PermissionRequestImpl(
-      requesting_origin, permission_type_, profile_, user_gesture,
-      base::Bind(&PermissionContextBase::PermissionDecided,
-                 weak_factory_.GetWeakPtr(), id, requesting_origin,
-                 embedding_origin, user_gesture, callback),
-      base::Bind(&PermissionContextBase::CleanUpRequest,
-                 weak_factory_.GetWeakPtr(), id)));
-  PermissionRequest* request = request_ptr.get();
+  if (PermissionRequestManager::IsEnabled()) {
+    PermissionRequestManager* permission_request_manager =
+        PermissionRequestManager::FromWebContents(web_contents);
+    // TODO(felt): sometimes |permission_request_manager| is null. This check is
+    // meant to prevent crashes. See crbug.com/457091.
+    if (!permission_request_manager)
+      return;
 
-  bool inserted =
-      pending_requests_.add(id.ToString(), std::move(request_ptr)).second;
-  DCHECK(inserted) << "Duplicate id " << id.ToString();
-  permission_request_manager->AddRequest(request);
+#if defined(OS_ANDROID)
+    // Create PermissionPrompt on Android.
+    permission_request_manager->DisplayPendingRequests();

[raymes, 2016/10/24 00:50:15]

As discussed, it seems like this might not be the right place for this. It looks
like it should be called once each time a tab is displayed, rather than per
request. We could remove it for now and address that in a subsequent CL if it
makes more sense.

[lshang, 2016/10/24 08:48:02]

Done. Removed it for now til we know clearly where to put it in.

+#endif
+
+    std::unique_ptr<PermissionRequest> request_ptr(new PermissionRequestImpl(
+        requesting_origin, permission_type_, profile_, user_gesture,
+        base::Bind(&PermissionContextBase::PermissionDecided,
+                   weak_factory_.GetWeakPtr(), id, requesting_origin,
+                   embedding_origin, user_gesture, callback),
+        base::Bind(&PermissionContextBase::CleanUpRequest,
+                   weak_factory_.GetWeakPtr(), id)));
+    PermissionRequest* request = request_ptr.get();
+
+    bool inserted =
+        pending_requests_.add(id.ToString(), std::move(request_ptr)).second;
+    DCHECK(inserted) << "Duplicate id " << id.ToString();
+    permission_request_manager->AddRequest(request);
+  } else {
+#if defined(OS_ANDROID)
+    GetQueueController()->CreateInfoBarRequest(
+        id, requesting_origin, embedding_origin, user_gesture,
+        base::Bind(&PermissionContextBase::PermissionDecided,
+                   weak_factory_.GetWeakPtr(), id, requesting_origin,
+                   embedding_origin, user_gesture, callback,
+                   // the queue controller takes care of persisting the
+                   // permission
+                   false));
 #else
-  GetQueueController()->CreateInfoBarRequest(
-      id, requesting_origin, embedding_origin, user_gesture,
-      base::Bind(&PermissionContextBase::PermissionDecided,
-                 weak_factory_.GetWeakPtr(), id, requesting_origin,
-                 embedding_origin, user_gesture, callback,
-                 // the queue controller takes care of persisting the
-                 // permission
-                 false));
+    NOTREACHED();
 #endif
+  }
 }
 
 void PermissionContextBase::PermissionDecided(
     const PermissionRequestID& id,
     const GURL& requesting_origin,
     const GURL& embedding_origin,
     bool user_gesture,
     const BrowserPermissionCallback& callback,
     bool persist,
     ContentSetting content_setting) {
@@ skipping 84 matching lines @@
                                       content_setting);
 }
 
 bool PermissionContextBase::IsPermissionKillSwitchOn() const {
   const std::string param = variations::GetVariationParamValue(
       kPermissionsKillSwitchFieldStudy,
       PermissionUtil::GetPermissionString(permission_type_));
 
   return param == kPermissionsKillSwitchBlockedValue;
 }