-// Copyright 2014 The Chromium Authors. All rights reserved.
+// Copyright 2014 The Chromium Authors
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
#include <memory>
#include <string>
-#include "base/bind.h"
#include "base/command_line.h"
+#include "base/functional/bind.h"
+#include "base/memory/raw_ptr.h"
#include "base/run_loop.h"
+#include "base/task/sequenced_task_runner.h"
+#include "base/test/bind.h"
#include "base/test/metrics/histogram_tester.h"
#include "base/test/scoped_feature_list.h"
-#include "base/threading/sequenced_task_runner_handle.h"
#include "build/build_config.h"
#include "components/permissions/features.h"
#include "components/permissions/permission_request.h"
#include "components/permissions/permission_request_manager.h"
#include "components/permissions/permission_ui_selector.h"
#include "components/permissions/permission_uma_util.h"
+#include "components/permissions/permission_util.h"
#include "components/permissions/request_type.h"
#include "components/permissions/test/mock_permission_prompt_factory.h"
#include "components/permissions/test/mock_permission_request.h"
#include "content/public/test/test_renderer_host.h"
#include "testing/gtest/include/gtest/gtest.h"
#include "third_party/abseil-cpp/absl/types/optional.h"
+#include "ui/events/base_event_utils.h"
+#include "ui/events/event.h"
namespace permissions {
using QuietUiReason = PermissionUiSelector::QuietUiReason;
}
-class PermissionRequestManagerTest
- : public content::RenderViewHostTestHarness,
- public ::testing::WithParamInterface<bool> {
+class PermissionRequestManagerTest : public content::RenderViewHostTestHarness {
public:
PermissionRequestManagerTest()
- : request1_(RequestType::kGeolocation,
+ : RenderViewHostTestHarness(
+ base::test::TaskEnvironment::TimeSource::MOCK_TIME),
+ request1_(RequestType::kGeolocation,
PermissionRequestGestureType::GESTURE),
request2_(RequestType::kMultipleDownloads,
PermissionRequestGestureType::NO_GESTURE),
PermissionRequestGestureType::NO_GESTURE),
request_camera_(RequestType::kCameraStream,
PermissionRequestGestureType::NO_GESTURE),
-#if !defined(OS_ANDROID)
+#if !BUILDFLAG(IS_ANDROID)
request_ptz_(RequestType::kCameraPanTiltZoom,
PermissionRequestGestureType::NO_GESTURE),
#endif
iframe_request_same_domain_(GURL("https://www.google.com/some/url"),
RequestType::kMidiSysex),
iframe_request_other_domain_(GURL("https://www.youtube.com"),
- RequestType::kGeolocation),
+ RequestType::kClipboard),
iframe_request_camera_other_domain_(GURL("https://www.youtube.com"),
RequestType::kStorageAccess),
iframe_request_mic_other_domain_(GURL("https://www.youtube.com"),
RequestType::kMicStream) {
- feature_list_.InitWithFeatureState(permissions::features::kPermissionChip,
- GetParam());
}
void SetUp() override {
content::RenderViewHostTestHarness::SetUp();
SetContents(CreateTestWebContents());
- NavigateAndCommit(GURL(permissions::MockPermissionRequest::kDefaultOrigin));
+ NavigateAndCommit(GURL(MockPermissionRequest::kDefaultOrigin));
PermissionRequestManager::CreateForWebContents(web_contents());
manager_ = PermissionRequestManager::FromWebContents(web_contents());
+ manager_->set_enabled_app_level_notification_permission_for_testing(true);
prompt_factory_ = std::make_unique<MockPermissionPromptFactory>(manager_);
}
void Accept() {
manager_->Accept();
- base::RunLoop().RunUntilIdle();
+ task_environment()->RunUntilIdle();
}
void Deny() {
manager_->Deny();
- base::RunLoop().RunUntilIdle();
+ task_environment()->RunUntilIdle();
}
void Closing() {
- manager_->Closing();
- base::RunLoop().RunUntilIdle();
+ manager_->Dismiss();
+ task_environment()->RunUntilIdle();
+ }
+
+ void OpenHelpCenterLink() {
+#if !BUILDFLAG(IS_ANDROID)
+ const ui::MouseEvent event(ui::ET_MOUSE_PRESSED, gfx::Point(), gfx::Point(),
+ ui::EventTimeForNow(), 0, 0);
+#else // BUILDFLAG(IS_ANDROID)
+ const ui::TouchEvent event(
+ ui::ET_TOUCH_MOVED, gfx::PointF(), gfx::PointF(), ui::EventTimeForNow(),
+ ui::PointerDetails(ui::EventPointerType::kTouch, 1));
+#endif
+ manager_->OpenHelpCenterLink(event);
+ task_environment()->RunUntilIdle();
}
void WaitForFrameLoad() {
// PermissionRequestManager ignores all parameters. Yay?
manager_->DOMContentLoaded(nullptr);
- base::RunLoop().RunUntilIdle();
+ task_environment()->RunUntilIdle();
}
void WaitForBubbleToBeShown() {
- manager_->DocumentOnLoadCompletedInMainFrame(main_rfh());
- base::RunLoop().RunUntilIdle();
+ manager_->DocumentOnLoadCompletedInPrimaryMainFrame();
+ task_environment()->RunUntilIdle();
}
void MockTabSwitchAway() {
std::unique_ptr<MockPermissionRequest> request =
std::make_unique<MockPermissionRequest>(
type, PermissionRequestGestureType::GESTURE);
- manager_->AddRequest(web_contents()->GetMainFrame(), request.get());
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), request.get());
WaitForBubbleToBeShown();
if (should_be_seen) {
EXPECT_TRUE(prompt_factory_->RequestTypeSeen(type));
MockPermissionRequest request2_;
MockPermissionRequest request_mic_;
MockPermissionRequest request_camera_;
-#if !defined(OS_ANDROID)
+#if !BUILDFLAG(IS_ANDROID)
MockPermissionRequest request_ptz_;
#endif
MockPermissionRequest iframe_request_same_domain_;
MockPermissionRequest iframe_request_other_domain_;
MockPermissionRequest iframe_request_camera_other_domain_;
MockPermissionRequest iframe_request_mic_other_domain_;
- PermissionRequestManager* manager_;
+ raw_ptr<PermissionRequestManager, DanglingUntriaged> manager_;
std::unique_ptr<MockPermissionPromptFactory> prompt_factory_;
TestPermissionsClient client_;
base::test::ScopedFeatureList feature_list_;
// General
////////////////////////////////////////////////////////////////////////////////
-TEST_P(PermissionRequestManagerTest, NoRequests) {
+TEST_F(PermissionRequestManagerTest, NoRequests) {
WaitForBubbleToBeShown();
EXPECT_FALSE(prompt_factory_->is_visible());
}
-TEST_P(PermissionRequestManagerTest, SingleRequest) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+TEST_F(PermissionRequestManagerTest, SingleRequest) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
EXPECT_TRUE(request1_.granted());
}
-TEST_P(PermissionRequestManagerTest, SequentialRequests) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+TEST_F(PermissionRequestManagerTest, SequentialRequests) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
EXPECT_TRUE(request1_.granted());
EXPECT_FALSE(prompt_factory_->is_visible());
- manager_->AddRequest(web_contents()->GetMainFrame(), &request2_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
Accept();
EXPECT_TRUE(request2_.granted());
}
-TEST_P(PermissionRequestManagerTest, ForgetRequestsOnPageNavigation) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request2_);
- manager_->AddRequest(web_contents()->GetMainFrame(),
+TEST_F(PermissionRequestManagerTest, ForgetRequestsOnPageNavigation) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
&iframe_request_other_domain_);
WaitForBubbleToBeShown();
EXPECT_TRUE(iframe_request_other_domain_.finished());
}
-TEST_P(PermissionRequestManagerTest, RequestsDontNeedUserGesture) {
+TEST_F(PermissionRequestManagerTest, RequestsDontNeedUserGesture) {
WaitForFrameLoad();
WaitForBubbleToBeShown();
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
- manager_->AddRequest(web_contents()->GetMainFrame(),
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
&iframe_request_other_domain_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request2_);
- base::RunLoop().RunUntilIdle();
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
+ task_environment()->RunUntilIdle();
EXPECT_TRUE(prompt_factory_->is_visible());
}
-TEST_P(PermissionRequestManagerTest, RequestsNotSupported) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+TEST_F(PermissionRequestManagerTest, RequestsNotSupported) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
WaitForBubbleToBeShown();
Accept();
EXPECT_TRUE(request1_.granted());
manager_->set_web_contents_supports_permission_requests(false);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request2_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
EXPECT_TRUE(request2_.cancelled());
}
// Requests grouping
////////////////////////////////////////////////////////////////////////////////
+// Android is the only platform that does not support the permission chip.
+#if BUILDFLAG(IS_ANDROID)
// Most requests should never be grouped.
-TEST_P(PermissionRequestManagerTest, TwoRequestsUngrouped) {
- // Grouping for chip feature is tested in ThreeRequestsStackOrderChip.
- if (GetParam())
- return;
-
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request2_);
+// Grouping for chip feature is tested in ThreeRequestsStackOrderChip.
+TEST_F(PermissionRequestManagerTest, TwoRequestsUngrouped) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
ASSERT_EQ(prompt_factory_->show_count(), 2);
}
-TEST_P(PermissionRequestManagerTest, ThreeRequestsStackOrderChip) {
- if (!GetParam())
- return;
-
+// Tests for non-Android platforms which support the permission chip.
+#else // BUILDFLAG(IS_ANDROID)
+TEST_F(PermissionRequestManagerTest, ThreeRequestsStackOrderChip) {
// Test new permissions order, requests shouldn't be grouped.
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request2_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_mic_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
}
// Test new permissions order by adding requests one at a time.
-TEST_P(PermissionRequestManagerTest, ThreeRequestsOneByOneStackOrderChip) {
- if (!GetParam())
- return;
-
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+TEST_F(PermissionRequestManagerTest, ThreeRequestsOneByOneStackOrderChip) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
WaitForBubbleToBeShown();
- manager_->AddRequest(web_contents()->GetMainFrame(), &request2_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
WaitForBubbleToBeShown();
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_mic_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
Accept();
EXPECT_TRUE(request1_.granted());
}
+#endif // BUILDFLAG(IS_ANDROID)
// Only mic/camera requests from the same origin should be grouped.
-TEST_P(PermissionRequestManagerTest, MicCameraGrouped) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_mic_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_camera_);
+TEST_F(PermissionRequestManagerTest, MicCameraGrouped) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_camera_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
// If mic/camera requests come from different origins, they should not be
// grouped.
-TEST_P(PermissionRequestManagerTest, MicCameraDifferentOrigins) {
- manager_->AddRequest(web_contents()->GetMainFrame(),
+TEST_F(PermissionRequestManagerTest, MicCameraDifferentOrigins) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
&iframe_request_mic_other_domain_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_camera_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_camera_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
ASSERT_EQ(prompt_factory_->request_count(), 1);
}
-#if !defined(OS_ANDROID)
+#if !BUILDFLAG(IS_ANDROID)
// Only camera/ptz requests from the same origin should be grouped.
-TEST_P(PermissionRequestManagerTest, CameraPtzGrouped) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_camera_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_ptz_);
+TEST_F(PermissionRequestManagerTest, CameraPtzGrouped) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_camera_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_ptz_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
EXPECT_TRUE(request_ptz_.granted());
}
-TEST_P(PermissionRequestManagerTest, CameraPtzDifferentOrigins) {
+TEST_F(PermissionRequestManagerTest, CameraPtzDifferentOrigins) {
// If camera/ptz requests come from different origins, they should not be
// grouped.
- manager_->AddRequest(web_contents()->GetMainFrame(),
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
&iframe_request_camera_other_domain_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_ptz_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_ptz_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
}
// Only mic/camera/ptz requests from the same origin should be grouped.
-TEST_P(PermissionRequestManagerTest, MicCameraPtzGrouped) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_mic_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_camera_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_ptz_);
+TEST_F(PermissionRequestManagerTest, MicCameraPtzGrouped) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_camera_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_ptz_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
// If mic/camera/ptz requests come from different origins, they should not be
// grouped.
-TEST_P(PermissionRequestManagerTest, MicCameraPtzDifferentOrigins) {
- manager_->AddRequest(web_contents()->GetMainFrame(),
+TEST_F(PermissionRequestManagerTest, MicCameraPtzDifferentOrigins) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
&iframe_request_mic_other_domain_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_camera_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_ptz_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_camera_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_ptz_);
WaitForBubbleToBeShown();
// Requests should be split into two groups and each one will contain less
EXPECT_TRUE(prompt_factory_->is_visible());
ASSERT_LT(prompt_factory_->request_count(), 3);
}
-#endif // !defined(OS_ANDROID)
+#endif // !BUILDFLAG(IS_ANDROID)
// Tests mix of grouped media requests and non-groupable request.
-TEST_P(PermissionRequestManagerTest, MixOfMediaAndNotMediaRequests) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_camera_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_mic_);
+TEST_F(PermissionRequestManagerTest, MixOfMediaAndNotMediaRequests) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_camera_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
WaitForBubbleToBeShown();
// Requests should be split into two groups and each one will contain less
Accept();
}
+TEST_F(PermissionRequestManagerTest, OpenHelpCenterLink) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ &iframe_request_camera_other_domain_);
+ WaitForBubbleToBeShown();
+ EXPECT_TRUE(prompt_factory_->is_visible());
+
+ OpenHelpCenterLink();
+ SUCCEED();
+}
+
+TEST_F(PermissionRequestManagerTest, OpenHelpCenterLink_RequestNotSupported) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ WaitForBubbleToBeShown();
+ EXPECT_TRUE(prompt_factory_->is_visible());
+
+ EXPECT_DEATH_IF_SUPPORTED(OpenHelpCenterLink(), "");
+}
+
////////////////////////////////////////////////////////////////////////////////
// Tab switching
////////////////////////////////////////////////////////////////////////////////
-TEST_P(PermissionRequestManagerTest, TwoRequestsTabSwitch) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_mic_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_camera_);
+#if BUILDFLAG(IS_ANDROID)
+TEST_F(PermissionRequestManagerTest, TwoRequestsTabSwitch) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_camera_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
ASSERT_EQ(prompt_factory_->request_count(), 2);
MockTabSwitchAway();
-#if defined(OS_ANDROID)
EXPECT_TRUE(prompt_factory_->is_visible());
-#else
- EXPECT_FALSE(prompt_factory_->is_visible());
-#endif
MockTabSwitchBack();
WaitForBubbleToBeShown();
EXPECT_TRUE(request_mic_.granted());
EXPECT_TRUE(request_camera_.granted());
}
+#endif // BUILDFLAG(IS_ANDROID)
-TEST_P(PermissionRequestManagerTest, PermissionRequestWhileTabSwitchedAway) {
+TEST_F(PermissionRequestManagerTest, PermissionRequestWhileTabSwitchedAway) {
MockTabSwitchAway();
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
WaitForBubbleToBeShown();
EXPECT_FALSE(prompt_factory_->is_visible());
// Duplicated requests
////////////////////////////////////////////////////////////////////////////////
-TEST_P(PermissionRequestManagerTest, SameRequestRejected) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+TEST_F(PermissionRequestManagerTest, SameRequestRejected) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
EXPECT_FALSE(request1_.finished());
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
ASSERT_EQ(prompt_factory_->request_count(), 1);
Accept();
- base::RunLoop().RunUntilIdle();
+ task_environment()->RunUntilIdle();
EXPECT_TRUE(request1_.granted());
EXPECT_FALSE(prompt_factory_->is_visible());
}
-TEST_P(PermissionRequestManagerTest, DuplicateRequest) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+TEST_F(PermissionRequestManagerTest, WeakDuplicateRequests) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
WaitForBubbleToBeShown();
- manager_->AddRequest(web_contents()->GetMainFrame(), &request2_);
+ auto dupe_request_1 = request1_.CreateDuplicateRequest();
+ auto dupe_request_2 = request1_.CreateDuplicateRequest();
+ auto dupe_request_3 = request1_.CreateDuplicateRequest();
+ auto dupe_request_4 = request1_.CreateDuplicateRequest();
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ dupe_request_1.get());
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ dupe_request_2.get());
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ dupe_request_3.get());
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ dupe_request_4.get());
+ dupe_request_1.reset();
+ dupe_request_3.reset();
+ EXPECT_EQ(4ul, manager_->duplicate_requests_.front().size());
+ EXPECT_EQ(1ul, manager_->duplicate_requests_.size());
+ auto request_list = manager_->FindDuplicateRequestList(&request1_);
+ EXPECT_NE(request_list, manager_->duplicate_requests_.end());
+ EXPECT_EQ(3ul, manager_->duplicate_requests_.front().size());
+ dupe_request_4.reset();
+ manager_->VisitDuplicateRequests(
+ base::BindRepeating(
+ [](const base::WeakPtr<PermissionRequest>& weak_request) {}),
+ &request1_);
+ EXPECT_EQ(1ul, manager_->duplicate_requests_.front().size());
+ EXPECT_EQ(1ul, manager_->duplicate_requests_.size());
+ Accept();
+ EXPECT_EQ(0ul, manager_->duplicate_requests_.size());
+}
+
+class QuicklyDeletedRequest : public PermissionRequest {
+ public:
+ QuicklyDeletedRequest(const GURL& requesting_origin,
+ RequestType request_type,
+ PermissionRequestGestureType gesture_type)
+ : PermissionRequest(requesting_origin,
+ request_type,
+ gesture_type == PermissionRequestGestureType::GESTURE,
+ base::BindLambdaForTesting(
+ [](ContentSetting result,
+ bool is_one_time,
+ bool is_final_decision) { NOTREACHED(); }),
+ base::NullCallback()) {}
+
+ static std::unique_ptr<QuicklyDeletedRequest> CreateRequest(
+ MockPermissionRequest* request) {
+ return std::make_unique<QuicklyDeletedRequest>(request->requesting_origin(),
+ request->request_type(),
+ request->GetGestureType());
+ }
+};
+
+TEST_F(PermissionRequestManagerTest, WeakDuplicateRequestsAccept) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ WaitForBubbleToBeShown();
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
+ auto dupe_request_1 = QuicklyDeletedRequest::CreateRequest(&request1_);
+ auto dupe_request_2 = request1_.CreateDuplicateRequest();
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ dupe_request_1.get());
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ dupe_request_2.get());
+ auto dupe_request_3 = QuicklyDeletedRequest::CreateRequest(&request2_);
+ auto dupe_request_4 = request2_.CreateDuplicateRequest();
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ dupe_request_3.get());
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ dupe_request_4.get());
+ dupe_request_1.reset();
+ dupe_request_3.reset();
+ EXPECT_EQ(2ul, manager_->duplicate_requests_.size());
+ EXPECT_EQ(2ul, manager_->duplicate_requests_.front().size());
+ EXPECT_EQ(2ul, manager_->duplicate_requests_.back().size());
+ WaitForBubbleToBeShown();
+ Accept();
+ EXPECT_EQ(1ul, manager_->duplicate_requests_.size());
+ WaitForBubbleToBeShown();
+ Accept();
+ EXPECT_EQ(0ul, manager_->duplicate_requests_.size());
+}
+
+TEST_F(PermissionRequestManagerTest, DuplicateRequest) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ WaitForBubbleToBeShown();
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
auto dupe_request = request1_.CreateDuplicateRequest();
- manager_->AddRequest(web_contents()->GetMainFrame(), dupe_request.get());
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ dupe_request.get());
EXPECT_FALSE(dupe_request->finished());
EXPECT_FALSE(request1_.finished());
auto dupe_request2 = request2_.CreateDuplicateRequest();
- manager_->AddRequest(web_contents()->GetMainFrame(), dupe_request2.get());
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ dupe_request2.get());
EXPECT_FALSE(dupe_request2->finished());
EXPECT_FALSE(request2_.finished());
WaitForBubbleToBeShown();
Accept();
- if (GetParam()) {
+ if (PermissionUtil::DoesPlatformSupportChip()) {
EXPECT_TRUE(dupe_request2->finished());
EXPECT_TRUE(request2_.finished());
} else {
WaitForBubbleToBeShown();
Accept();
- if (GetParam()) {
+ if (PermissionUtil::DoesPlatformSupportChip()) {
EXPECT_TRUE(dupe_request->finished());
EXPECT_TRUE(request1_.finished());
} else {
// Requests from iframes
////////////////////////////////////////////////////////////////////////////////
-TEST_P(PermissionRequestManagerTest, MainFrameNoRequestIFrameRequest) {
- manager_->AddRequest(web_contents()->GetMainFrame(),
+TEST_F(PermissionRequestManagerTest, MainFrameNoRequestIFrameRequest) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
&iframe_request_same_domain_);
WaitForBubbleToBeShown();
WaitForFrameLoad();
EXPECT_TRUE(iframe_request_same_domain_.finished());
}
-TEST_P(PermissionRequestManagerTest, MainFrameAndIFrameRequestSameDomain) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
- manager_->AddRequest(web_contents()->GetMainFrame(),
+TEST_F(PermissionRequestManagerTest, MainFrameAndIFrameRequestSameDomain) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
&iframe_request_same_domain_);
WaitForFrameLoad();
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
ASSERT_EQ(1, prompt_factory_->request_count());
Closing();
- if (GetParam()) {
+ if (PermissionUtil::DoesPlatformSupportChip()) {
EXPECT_TRUE(iframe_request_same_domain_.finished());
EXPECT_FALSE(request1_.finished());
} else {
Closing();
EXPECT_FALSE(prompt_factory_->is_visible());
- if (GetParam())
+ if (PermissionUtil::DoesPlatformSupportChip()) {
EXPECT_TRUE(request1_.finished());
- else
+ } else {
EXPECT_TRUE(iframe_request_same_domain_.finished());
+ }
}
-TEST_P(PermissionRequestManagerTest, MainFrameAndIFrameRequestOtherDomain) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
- manager_->AddRequest(web_contents()->GetMainFrame(),
+TEST_F(PermissionRequestManagerTest, MainFrameAndIFrameRequestOtherDomain) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
&iframe_request_other_domain_);
WaitForFrameLoad();
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
Closing();
- if (GetParam()) {
+ if (PermissionUtil::DoesPlatformSupportChip()) {
EXPECT_TRUE(iframe_request_other_domain_.finished());
EXPECT_FALSE(request1_.finished());
} else {
EXPECT_TRUE(prompt_factory_->is_visible());
Closing();
EXPECT_TRUE(iframe_request_other_domain_.finished());
- if (GetParam())
+ if (PermissionUtil::DoesPlatformSupportChip()) {
EXPECT_TRUE(request1_.finished());
- else
+ } else {
EXPECT_TRUE(iframe_request_other_domain_.finished());
+ }
}
-TEST_P(PermissionRequestManagerTest, IFrameRequestWhenMainRequestVisible) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+TEST_F(PermissionRequestManagerTest, IFrameRequestWhenMainRequestVisible) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
- manager_->AddRequest(web_contents()->GetMainFrame(),
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
&iframe_request_same_domain_);
WaitForFrameLoad();
ASSERT_EQ(prompt_factory_->request_count(), 1);
Closing();
- if (GetParam()) {
+ if (PermissionUtil::DoesPlatformSupportChip()) {
EXPECT_TRUE(iframe_request_same_domain_.finished());
EXPECT_FALSE(request1_.finished());
} else {
ASSERT_EQ(prompt_factory_->request_count(), 1);
Closing();
EXPECT_TRUE(iframe_request_same_domain_.finished());
- if (GetParam())
+ if (PermissionUtil::DoesPlatformSupportChip()) {
EXPECT_TRUE(request1_.finished());
- else
+ } else {
EXPECT_TRUE(iframe_request_same_domain_.finished());
+ }
}
-TEST_P(PermissionRequestManagerTest,
+TEST_F(PermissionRequestManagerTest,
IFrameRequestOtherDomainWhenMainRequestVisible) {
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
- manager_->AddRequest(web_contents()->GetMainFrame(),
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
&iframe_request_other_domain_);
WaitForFrameLoad();
Closing();
- if (GetParam()) {
+ if (PermissionUtil::DoesPlatformSupportChip()) {
EXPECT_TRUE(iframe_request_other_domain_.finished());
EXPECT_FALSE(request1_.finished());
} else {
EXPECT_TRUE(prompt_factory_->is_visible());
Closing();
- if (GetParam())
+ if (PermissionUtil::DoesPlatformSupportChip()) {
EXPECT_TRUE(request1_.finished());
- else
+ } else {
EXPECT_TRUE(iframe_request_other_domain_.finished());
+ }
}
////////////////////////////////////////////////////////////////////////////////
// This code path (calling Accept on a non-merged bubble, with no accepted
// permission) would never be used in actual Chrome, but its still tested for
// completeness.
-TEST_P(PermissionRequestManagerTest, UMAForSimpleDeniedBubbleAlternatePath) {
+TEST_F(PermissionRequestManagerTest, UMAForSimpleDeniedBubbleAlternatePath) {
base::HistogramTester histograms;
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
WaitForBubbleToBeShown();
// No need to test UMA for showing prompts again, they were tested in
// UMAForSimpleAcceptedBubble.
1);
}
-TEST_P(PermissionRequestManagerTest, UMAForTabSwitching) {
+TEST_F(PermissionRequestManagerTest, UMAForTabSwitching) {
base::HistogramTester histograms;
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
WaitForBubbleToBeShown();
histograms.ExpectUniqueSample(PermissionUmaUtil::kPermissionsPromptShown,
static_cast<base::HistogramBase::Sample>(
absl::optional<QuietUiReason> quiet_ui_reason,
absl::optional<PermissionUmaUtil::PredictionGrantLikelihood>
prediction_likelihood,
- bool async)
+ absl::optional<base::TimeDelta> async_delay)
: quiet_ui_reason_(quiet_ui_reason),
prediction_likelihood_(prediction_likelihood),
- async_(async) {}
+ async_delay_(async_delay) {}
void SelectUiToUse(PermissionRequest* request,
DecisionMadeCallback callback) override {
+ selected_ui_to_use_ = true;
Decision decision(quiet_ui_reason_, Decision::ShowNoWarning());
- if (async_) {
- base::SequencedTaskRunnerHandle::Get()->PostTask(
- FROM_HERE, base::BindOnce(std::move(callback), decision));
+ if (async_delay_) {
+ base::SequencedTaskRunner::GetCurrentDefault()->PostDelayedTask(
+ FROM_HERE, base::BindOnce(std::move(callback), decision),
+ async_delay_.value());
} else {
std::move(callback).Run(decision);
}
static void CreateForManager(
PermissionRequestManager* manager,
absl::optional<QuietUiReason> quiet_ui_reason,
- bool async,
+ absl::optional<base::TimeDelta> async_delay,
absl::optional<PermissionUmaUtil::PredictionGrantLikelihood>
prediction_likelihood = absl::nullopt) {
manager->add_permission_ui_selector_for_testing(
std::make_unique<MockNotificationPermissionUiSelector>(
- quiet_ui_reason, prediction_likelihood, async));
+ quiet_ui_reason, prediction_likelihood, async_delay));
}
+ bool selected_ui_to_use() const { return selected_ui_to_use_; }
+
private:
absl::optional<QuietUiReason> quiet_ui_reason_;
absl::optional<PermissionUmaUtil::PredictionGrantLikelihood>
prediction_likelihood_;
- bool async_;
+ absl::optional<base::TimeDelta> async_delay_;
+ bool selected_ui_to_use_ = false;
};
// Same as the MockNotificationPermissionUiSelector but handling only the
absl::optional<QuietUiReason> quiet_ui_reason,
absl::optional<PermissionUmaUtil::PredictionGrantLikelihood>
prediction_likelihood,
- bool async)
+ absl::optional<base::TimeDelta> async_delay)
: MockNotificationPermissionUiSelector(quiet_ui_reason,
prediction_likelihood,
- async) {}
+ async_delay) {}
bool IsPermissionRequestSupported(RequestType request_type) override {
return request_type == RequestType::kCameraStream;
static void CreateForManager(
PermissionRequestManager* manager,
absl::optional<QuietUiReason> quiet_ui_reason,
- bool async,
+ absl::optional<base::TimeDelta> async_delay,
absl::optional<PermissionUmaUtil::PredictionGrantLikelihood>
prediction_likelihood = absl::nullopt) {
manager->add_permission_ui_selector_for_testing(
std::make_unique<MockCameraStreamPermissionUiSelector>(
- quiet_ui_reason, prediction_likelihood, async));
+ quiet_ui_reason, prediction_likelihood, async_delay));
}
};
-TEST_P(PermissionRequestManagerTest,
+TEST_F(PermissionRequestManagerTest,
UiSelectorNotUsedForPermissionsOtherThanNotification) {
manager_->clear_permission_ui_selector_for_testing();
MockNotificationPermissionUiSelector::CreateForManager(
manager_, PermissionUiSelector::QuietUiReason::kEnabledInPrefs,
- false /* async */);
+ absl::nullopt /* async_delay */);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request_camera_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_camera_);
WaitForBubbleToBeShown();
ASSERT_TRUE(prompt_factory_->is_visible());
EXPECT_TRUE(request_camera_.granted());
}
-TEST_P(PermissionRequestManagerTest, UiSelectorUsedForNotifications) {
+TEST_F(PermissionRequestManagerTest, UiSelectorUsedForNotifications) {
const struct {
absl::optional<PermissionUiSelector::QuietUiReason> quiet_ui_reason;
- bool async;
+ absl::optional<base::TimeDelta> async_delay;
} kTests[] = {
- {QuietUiReason::kEnabledInPrefs, true},
- {PermissionUiSelector::Decision::UseNormalUi(), true},
- {QuietUiReason::kEnabledInPrefs, false},
- {PermissionUiSelector::Decision::UseNormalUi(), false},
+ {QuietUiReason::kEnabledInPrefs, absl::make_optional<base::TimeDelta>()},
+ {PermissionUiSelector::Decision::UseNormalUi(),
+ absl::make_optional<base::TimeDelta>()},
+ {QuietUiReason::kEnabledInPrefs, absl::nullopt},
+ {PermissionUiSelector::Decision::UseNormalUi(), absl::nullopt},
};
for (const auto& test : kTests) {
manager_->clear_permission_ui_selector_for_testing();
MockNotificationPermissionUiSelector::CreateForManager(
- manager_, test.quiet_ui_reason, test.async);
+ manager_, test.quiet_ui_reason, test.async_delay);
MockPermissionRequest request(RequestType::kNotifications,
PermissionRequestGestureType::GESTURE);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
}
}
-TEST_P(PermissionRequestManagerTest,
+TEST_F(PermissionRequestManagerTest,
UiSelectionHappensSeparatelyForEachRequest) {
manager_->clear_permission_ui_selector_for_testing();
MockNotificationPermissionUiSelector::CreateForManager(
- manager_, QuietUiReason::kEnabledInPrefs, true);
+ manager_, QuietUiReason::kEnabledInPrefs,
+ absl::make_optional<base::TimeDelta>());
MockPermissionRequest request1(RequestType::kNotifications,
PermissionRequestGestureType::GESTURE);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request1);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1);
WaitForBubbleToBeShown();
EXPECT_TRUE(manager_->ShouldCurrentRequestUseQuietUI());
Accept();
PermissionRequestGestureType::GESTURE);
manager_->clear_permission_ui_selector_for_testing();
MockNotificationPermissionUiSelector::CreateForManager(
- manager_, PermissionUiSelector::Decision::UseNormalUi(), true);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request2);
+ manager_, PermissionUiSelector::Decision::UseNormalUi(),
+ absl::make_optional<base::TimeDelta>());
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2);
WaitForBubbleToBeShown();
EXPECT_FALSE(manager_->ShouldCurrentRequestUseQuietUI());
Accept();
}
-TEST_P(PermissionRequestManagerTest, MultipleUiSelectors) {
+TEST_F(PermissionRequestManagerTest, SkipNextUiSelector) {
+ manager_->clear_permission_ui_selector_for_testing();
+ MockNotificationPermissionUiSelector::CreateForManager(
+ manager_, QuietUiReason::kEnabledInPrefs,
+ /* async_delay */ absl::nullopt);
+ MockNotificationPermissionUiSelector::CreateForManager(
+ manager_, PermissionUiSelector::Decision::UseNormalUi(),
+ /* async_delay */ absl::nullopt);
+ MockPermissionRequest request1(RequestType::kNotifications,
+ PermissionRequestGestureType::GESTURE);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1);
+ WaitForBubbleToBeShown();
+ auto* next_selector =
+ manager_->get_permission_ui_selectors_for_testing().back().get();
+ EXPECT_FALSE(static_cast<MockNotificationPermissionUiSelector*>(next_selector)
+ ->selected_ui_to_use());
+ EXPECT_TRUE(manager_->ShouldCurrentRequestUseQuietUI());
+ Accept();
+}
+TEST_F(PermissionRequestManagerTest, MultipleUiSelectors) {
const struct {
std::vector<absl::optional<QuietUiReason>> quiet_ui_reasons;
std::vector<bool> simulate_delayed_decision;
{{QuietUiReason::kTriggeredByCrowdDeny, QuietUiReason::kEnabledInPrefs},
{false, false},
QuietUiReason::kTriggeredByCrowdDeny},
+ {{QuietUiReason::kTriggeredDueToDisruptiveBehavior,
+ QuietUiReason::kEnabledInPrefs},
+ {false, false},
+ QuietUiReason::kTriggeredDueToDisruptiveBehavior},
+ {{QuietUiReason::kTriggeredDueToDisruptiveBehavior,
+ QuietUiReason::kServicePredictedVeryUnlikelyGrant},
+ {false, false},
+ QuietUiReason::kTriggeredDueToDisruptiveBehavior},
+ {{QuietUiReason::kTriggeredDueToDisruptiveBehavior,
+ QuietUiReason::kTriggeredByCrowdDeny},
+ {false, false},
+ QuietUiReason::kTriggeredDueToDisruptiveBehavior},
// First selector is async but should still take priority even if it
// returns later.
{{QuietUiReason::kTriggeredByCrowdDeny, QuietUiReason::kEnabledInPrefs},
{true, false},
QuietUiReason::kTriggeredByCrowdDeny},
+ {{QuietUiReason::kTriggeredDueToDisruptiveBehavior,
+ QuietUiReason::kEnabledInPrefs},
+ {true, false},
+ QuietUiReason::kTriggeredDueToDisruptiveBehavior},
// The first selector that has a quiet ui decision should be used.
{{absl::nullopt, absl::nullopt,
QuietUiReason::kTriggeredDueToAbusiveContent,
absl::nullopt, QuietUiReason::kEnabledInPrefs},
{true, true, true, true, true, true, true, true},
QuietUiReason::kTriggeredDueToAbusiveRequests},
+ // Use a bunch of selectors both async and sync.
+ {{absl::nullopt, absl::nullopt, absl::nullopt, absl::nullopt,
+ absl::nullopt, QuietUiReason::kTriggeredDueToDisruptiveBehavior,
+ absl::nullopt, QuietUiReason::kEnabledInPrefs},
+ {true, false, false, true, true, true, false, false},
+ QuietUiReason::kTriggeredDueToDisruptiveBehavior},
};
for (const auto& test : kTests) {
for (size_t i = 0; i < test.quiet_ui_reasons.size(); ++i) {
MockNotificationPermissionUiSelector::CreateForManager(
manager_, test.quiet_ui_reasons[i],
- test.simulate_delayed_decision[i]);
+ test.simulate_delayed_decision[i]
+ ? absl::make_optional<base::TimeDelta>()
+ : absl::nullopt);
}
MockPermissionRequest request(RequestType::kNotifications,
PermissionRequestGestureType::GESTURE);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
}
}
-TEST_P(PermissionRequestManagerTest, SelectorsPredictionLikelihood) {
+TEST_F(PermissionRequestManagerTest, SelectorsPredictionLikelihood) {
using PredictionLikelihood = PermissionUmaUtil::PredictionGrantLikelihood;
const auto VeryLikely = PredictionLikelihood::
PermissionPrediction_Likelihood_DiscretizedLikelihood_VERY_LIKELY;
test.enable_quiet_uis[i]
? absl::optional<QuietUiReason>(QuietUiReason::kEnabledInPrefs)
: absl::nullopt,
- false /* async */, test.prediction_likelihoods[i]);
+ absl::nullopt /* async_delay */, test.prediction_likelihoods[i]);
}
MockPermissionRequest request(RequestType::kNotifications,
PermissionRequestGestureType::GESTURE);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
}
}
-TEST_P(PermissionRequestManagerTest, SelectorRequestTypes) {
+TEST_F(PermissionRequestManagerTest, SelectorRequestTypes) {
const struct {
RequestType request_type;
bool should_request_use_quiet_ui;
};
manager_->clear_permission_ui_selector_for_testing();
MockNotificationPermissionUiSelector::CreateForManager(
- manager_, QuietUiReason::kEnabledInPrefs, true);
+ manager_, QuietUiReason::kEnabledInPrefs,
+ absl::make_optional<base::TimeDelta>());
for (const auto& test : kTests) {
MockPermissionRequest request(test.request_type,
PermissionRequestGestureType::GESTURE);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request);
WaitForBubbleToBeShown();
EXPECT_EQ(test.should_request_use_quiet_ui,
manager_->ShouldCurrentRequestUseQuietUI());
}
// Adding a mock PermissionUiSelector that handles Camera stream.
MockCameraStreamPermissionUiSelector::CreateForManager(
- manager_, QuietUiReason::kEnabledInPrefs, true);
+ manager_, QuietUiReason::kEnabledInPrefs,
+ absl::make_optional<base::TimeDelta>());
// Now the RequestType::kCameraStream should show a quiet UI as well
MockPermissionRequest request2(RequestType::kCameraStream,
PermissionRequestGestureType::GESTURE);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request2);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2);
WaitForBubbleToBeShown();
EXPECT_TRUE(manager_->ShouldCurrentRequestUseQuietUI());
Accept();
// Quiet UI chip. Low priority for Notifications & Geolocation.
////////////////////////////////////////////////////////////////////////////////
-TEST_P(PermissionRequestManagerTest, NotificationsSingleBubbleAndChipRequest) {
+TEST_F(PermissionRequestManagerTest, NotificationsSingleBubbleAndChipRequest) {
MockPermissionRequest request(RequestType::kNotifications,
PermissionRequestGestureType::GESTURE);
- manager_->AddRequest(web_contents()->GetMainFrame(), &request);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request);
WaitForBubbleToBeShown();
EXPECT_TRUE(prompt_factory_->is_visible());
EXPECT_EQ(prompt_factory_->show_count(), 1);
}
+// Android is the only platform that does not support the permission chip.
+#if BUILDFLAG(IS_ANDROID)
// Quiet UI feature is disabled. Chip is disabled. No low priority requests, the
// first request is always shown.
//
// 1. Notification request shown
// 2. Geolocation request shown
// 3. Camera request shown
-TEST_P(PermissionRequestManagerTest,
+TEST_F(PermissionRequestManagerTest,
NotificationsGeolocationCameraBubbleRequest) {
- // permissions::features::kPermissionChip is enabled based on `GetParam()`.
- // That test is only for the default bubble.
- if (GetParam())
- return;
-
std::unique_ptr<MockPermissionRequest> request_notifications =
CreateAndAddRequest(RequestType::kNotifications, /*should_be_seen=*/true,
1);
EXPECT_EQ(prompt_factory_->show_count(), 3);
}
+// Tests for non-Android platforms which support the permission chip.
+#else // BUILDFLAG(IS_ANDROID)
// Quiet UI feature is disabled, no low priority requests, the last request is
// always shown.
//
// Permissions requested in order:
-// 1. Notification (non abusive)
-// 2. Geolocation
-// 3. Camera
+// 1. Camera
+// 2. Clipboard
+// 3. MIDI
//
// Prompt display order:
-// 1. Notifications request shown but is preempted
-// 2. Geolocation request shown but is preempted
-// 3. Camera request shown
-// 4. Geolocation request shown again
-// 5. Notifications request shown again
-TEST_P(PermissionRequestManagerTest,
- NotificationsGeolocationCameraChipRequest) {
- // permissions::features::kPermissionChip is enabled based on `GetParam()`.
- // That test is only for the chip UI.
- if (!GetParam())
- return;
-
- std::unique_ptr<MockPermissionRequest> request_notifications =
- CreateAndAddRequest(RequestType::kNotifications, /*should_be_seen=*/true,
- 1);
- std::unique_ptr<MockPermissionRequest> request_geolocation =
- CreateAndAddRequest(RequestType::kGeolocation, /*should_be_seen=*/true,
- 2);
+// 1. Camera request shown but is preempted
+// 2. Clipboard request shown but is preempted
+// 3. MIDI request shown
+// 4. Clipboard request shown again
+// 5. Camera request shown again
+TEST_F(PermissionRequestManagerTest,
+ CameraNotificationsGeolocationChipRequest) {
std::unique_ptr<MockPermissionRequest> request_camera = CreateAndAddRequest(
- RequestType::kCameraStream, /*should_be_seen=*/true, 3);
+ RequestType::kCameraStream, /*should_be_seen=*/true, 1);
+ std::unique_ptr<MockPermissionRequest> request_clipboard =
+ CreateAndAddRequest(RequestType::kClipboard, /*should_be_seen=*/true, 2);
+ std::unique_ptr<MockPermissionRequest> request_midi =
+ CreateAndAddRequest(RequestType::kMidi, /*should_be_seen=*/true, 3);
- for (auto* kRequest : {request_camera.get(), request_geolocation.get(),
- request_notifications.get()}) {
+ for (auto* kRequest :
+ {request_midi.get(), request_clipboard.get(), request_camera.get()}) {
WaitAndAcceptPromptForRequest(kRequest);
}
EXPECT_EQ(prompt_factory_->show_count(), 5);
}
-// Quiet UI feature is disabled, no low priority requests, the last request is
-// always shown.
+// Verifies order of simultaneous requests, with quiet chip enabled.
+// Simultaneous new requests are coming while we are waiting for UI selector
+// decisions.
//
// Permissions requested in order:
-// 1. Camera
-// 2. Notification (non abusive)
-// 3. Geolocation
+// 1. Geolocation, UI selector takes 2 seconds to decide.
+// 2. Notification then mic. Notification will preempt geolocation
//
// Prompt display order:
-// 1. Camera request shown but is preempted
-// 2. Notifications request shown but is preempted
-// 3. Geolocation request shown
-// 4. Notifications request shown again
-// 5. Camera request shown again
-TEST_P(PermissionRequestManagerTest,
- CameraNotificationsGeolocationChipRequest) {
- // permissions::features::kPermissionChip is enabled based on `GetParam()`.
- // That test is only for the chip.
- if (!GetParam())
- return;
+// 1. Mic
+// 2. Clipboard
+// 3. Geolocation
+TEST_F(PermissionRequestManagerTest, NewHighPriorityRequestDuringUIDecision) {
+ manager_->clear_permission_ui_selector_for_testing();
+ MockNotificationPermissionUiSelector::CreateForManager(
+ manager_, QuietUiReason::kTriggeredDueToAbusiveRequests,
+ absl::make_optional<base::TimeDelta>(base::Seconds(2)));
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
- std::unique_ptr<MockPermissionRequest> request_camera = CreateAndAddRequest(
- RequestType::kCameraStream, /*should_be_seen=*/true, 1);
- std::unique_ptr<MockPermissionRequest> request_notifications =
- CreateAndAddRequest(RequestType::kNotifications, /*should_be_seen=*/true,
- 2);
- std::unique_ptr<MockPermissionRequest> request_geolocation =
- CreateAndAddRequest(RequestType::kGeolocation, /*should_be_seen=*/true,
- 3);
+ task_environment()->FastForwardBy(base::Seconds(1));
- for (auto* kRequest : {request_geolocation.get(), request_notifications.get(),
- request_camera.get()}) {
- WaitAndAcceptPromptForRequest(kRequest);
- }
+ MockPermissionRequest request(RequestType::kClipboard,
+ PermissionRequestGestureType::GESTURE);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
+ WaitForBubbleToBeShown();
+ manager_->clear_permission_ui_selector_for_testing();
- EXPECT_EQ(prompt_factory_->show_count(), 5);
-}
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request_mic_.granted());
+ EXPECT_FALSE(request.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
-class PermissionRequestManagerTestQuietChip
- : public PermissionRequestManagerTest {
- public:
- PermissionRequestManagerTestQuietChip() {
- feature_list_.InitWithFeatureState(
- permissions::features::kPermissionQuietChip, true);
- }
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
- private:
- base::test::ScopedFeatureList feature_list_;
-};
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request1_.granted());
+}
// Verifies that the quiet UI chip is not ignored if another request came in
// less than 8.5 seconds after.
// 1. Notifications request shown but is preempted because of quiet UI.
// 2. Geolocation request shown
// 3. Notifications request shown again
-TEST_P(PermissionRequestManagerTestQuietChip,
+TEST_F(PermissionRequestManagerTest,
AbusiveNotificationsGeolocationQuietUIChipRequest) {
MockNotificationPermissionUiSelector::CreateForManager(
manager_,
PermissionUiSelector::QuietUiReason::kTriggeredDueToAbusiveRequests,
- false /* async */);
+ absl::nullopt /* async_delay */);
std::unique_ptr<MockPermissionRequest> request_notifications =
CreateAndAddRequest(RequestType::kNotifications, /*should_be_seen=*/true,
// Less then 8.5 seconds.
manager_->set_current_request_first_display_time_for_testing(
- base::Time::Now() - base::TimeDelta::FromMilliseconds(5000));
+ base::Time::Now() - base::Milliseconds(5000));
std::unique_ptr<MockPermissionRequest> request_geolocation =
CreateAndAddRequest(RequestType::kGeolocation, /*should_be_seen=*/true,
// Prompt display order:
// 1. Notifications request shown but is preempted because of quiet UI.
// 2. Geolocation request shown
-TEST_P(PermissionRequestManagerTestQuietChip,
- AbusiveNotificationsShownLongEnough) {
+TEST_F(PermissionRequestManagerTest, AbusiveNotificationsShownLongEnough) {
MockNotificationPermissionUiSelector::CreateForManager(
manager_,
PermissionUiSelector::QuietUiReason::kTriggeredDueToAbusiveRequests,
- false /* async */);
+ absl::nullopt /* async_delay */);
std::unique_ptr<MockPermissionRequest> request_notifications =
CreateAndAddRequest(RequestType::kNotifications, /*should_be_seen=*/true,
// More then 8.5 seconds.
manager_->set_current_request_first_display_time_for_testing(
- base::Time::Now() - base::TimeDelta::FromMilliseconds(9000));
+ base::Time::Now() - base::Milliseconds(9000));
std::unique_ptr<MockPermissionRequest> request_geolocation =
CreateAndAddRequest(RequestType::kGeolocation, /*should_be_seen=*/true,
// 3. Camera request shown
// 4. Geolocation request shown again
// 5. Notifications quiet UI request shown again
-TEST_P(PermissionRequestManagerTestQuietChip,
+TEST_F(PermissionRequestManagerTest,
AbusiveNotificationsShownLongEnoughCamera) {
MockNotificationPermissionUiSelector::CreateForManager(
manager_,
PermissionUiSelector::QuietUiReason::kTriggeredDueToAbusiveRequests,
- false /* async */);
+ absl::nullopt /* async_delay */);
std::unique_ptr<MockPermissionRequest> request_notifications =
CreateAndAddRequest(RequestType::kNotifications, /*should_be_seen=*/true,
1);
// Less then 8.5 seconds.
manager_->set_current_request_first_display_time_for_testing(
- base::Time::Now() - base::TimeDelta::FromMilliseconds(5000));
+ base::Time::Now() - base::Milliseconds(5000));
std::unique_ptr<MockPermissionRequest> request_geolocation =
CreateAndAddRequest(RequestType::kGeolocation, /*should_be_seen=*/true,
// 1. Camera request shown
// 2. Geolocation request shown
// 3. Camera request shown
-TEST_P(PermissionRequestManagerTestQuietChip,
- CameraAbusiveNotificationsGeolocation) {
+TEST_F(PermissionRequestManagerTest, CameraAbusiveNotificationsGeolocation) {
MockNotificationPermissionUiSelector::CreateForManager(
manager_,
PermissionUiSelector::QuietUiReason::kTriggeredDueToAbusiveRequests,
- false /* async */);
+ absl::nullopt /* async_delay */);
std::unique_ptr<MockPermissionRequest> request_camera = CreateAndAddRequest(
RequestType::kCameraStream, /*should_be_seen=*/true, 1);
1);
// Less then 8.5 seconds.
manager_->set_current_request_first_display_time_for_testing(
- base::Time::Now() - base::TimeDelta::FromMilliseconds(5000));
+ base::Time::Now() - base::Milliseconds(5000));
// Geolocation is not shown because Camera has higher priority.
std::unique_ptr<MockPermissionRequest> request_geolocation =
// Prompt display order:
// 1. Camera request shown
// 2. MIDI request shown (or MIDI and then Camera, the order depends on
-// `GetParam()`)
+// `PermissionUtil::DoesPlatformSupportChip()`)
// 3. Geolocation request shown
// 4. Notifications request shown
// If Chip is enabled MIDI will replace Camera, hence 5 prompts will be
// shown. Otherwise 4.
-TEST_P(PermissionRequestManagerTestQuietChip,
+TEST_F(PermissionRequestManagerTest,
CameraAbusiveNotificationsGeolocationMIDI) {
MockNotificationPermissionUiSelector::CreateForManager(
manager_,
PermissionUiSelector::QuietUiReason::kTriggeredDueToAbusiveRequests,
- false /* async */);
+ absl::nullopt /* async_delay */);
std::unique_ptr<MockPermissionRequest> request_camera = CreateAndAddRequest(
RequestType::kCameraStream, /*should_be_seen=*/true, 1);
1);
// Less then 8.5 seconds.
manager_->set_current_request_first_display_time_for_testing(
- base::Time::Now() - base::TimeDelta::FromMilliseconds(5000));
+ base::Time::Now() - base::Milliseconds(5000));
// Geolocation is not shown because Camera has higher priority.
std::unique_ptr<MockPermissionRequest> request_geolocation =
std::unique_ptr<MockPermissionRequest> request_midi;
- // If Chip is enabled, MIDI should be shown, otherwise MIDI should not be
- // shown.
- if (GetParam()) {
- request_midi = CreateAndAddRequest(RequestType::kMidiSysex,
- /*should_be_seen=*/true, 2);
- } else {
- request_midi = CreateAndAddRequest(RequestType::kMidiSysex,
- /*should_be_seen=*/false, 1);
- }
+ // Since the chip is enabled, MIDI should be shown.
+ request_midi = CreateAndAddRequest(RequestType::kMidiSysex,
+ /*should_be_seen=*/true, 2);
// The second permission after quiet UI was requested in 8.5 second window,
// the quiet UI Notifiations request for an abusive origin is not
EXPECT_FALSE(request_notifications->granted());
EXPECT_FALSE(request_notifications->finished());
- WaitAndAcceptPromptForRequest(GetParam() ? request_midi.get()
- : request_camera.get());
- WaitAndAcceptPromptForRequest(GetParam() ? request_camera.get()
- : request_midi.get());
+ WaitAndAcceptPromptForRequest(request_midi.get());
+ WaitAndAcceptPromptForRequest(request_camera.get());
WaitAndAcceptPromptForRequest(request_geolocation.get());
WaitAndAcceptPromptForRequest(request_notifications.get());
- EXPECT_EQ(prompt_factory_->show_count(), GetParam() ? 5 : 4);
+ EXPECT_EQ(prompt_factory_->show_count(), 5);
}
// Verifies that non abusive chip behaves similar to others when Quiet UI Chip
// Prompt display order:
// 1. Camera request shown
// 2. MIDI request shown (or MIDI and then Camera, the order depends on
-// `GetParam()`)
+// `PermissionUtil::DoesPlatformSupportChip()`)
// 3. Geolocation request shown
// 4. Notifications request shown
// If Chip is enabled MIDI will replace Camera, hence 5 prompts will be
// shown. Otherwise 4.
-TEST_P(PermissionRequestManagerTestQuietChip,
+TEST_F(PermissionRequestManagerTest,
CameraNonAbusiveNotificationsGeolocationMIDI) {
std::unique_ptr<MockPermissionRequest> request_camera = CreateAndAddRequest(
RequestType::kCameraStream, /*should_be_seen=*/true, 1);
1);
// Less then 8.5 seconds.
manager_->set_current_request_first_display_time_for_testing(
- base::Time::Now() - base::TimeDelta::FromMilliseconds(5000));
+ base::Time::Now() - base::Milliseconds(5000));
// Geolocation is not shown because Camera has higher priority.
std::unique_ptr<MockPermissionRequest> request_geolocation =
// If Chip is enabled, MIDI should be shown, otherwise MIDI should not be
// shown.
- if (GetParam()) {
request_midi = CreateAndAddRequest(RequestType::kMidiSysex,
/*should_be_seen=*/true, 2);
- } else {
- request_midi = CreateAndAddRequest(RequestType::kMidiSysex,
- /*should_be_seen=*/false, 1);
- }
// The second permission after quiet UI was requested in 8.5 second window,
// the quiet UI Notifiations request for an abusive origin is not
EXPECT_FALSE(request_notifications->granted());
EXPECT_FALSE(request_notifications->finished());
- WaitAndAcceptPromptForRequest(GetParam() ? request_midi.get()
- : request_camera.get());
- WaitAndAcceptPromptForRequest(GetParam() ? request_camera.get()
- : request_midi.get());
+ WaitAndAcceptPromptForRequest(request_midi.get());
+ WaitAndAcceptPromptForRequest(request_camera.get());
WaitAndAcceptPromptForRequest(request_geolocation.get());
WaitAndAcceptPromptForRequest(request_notifications.get());
- EXPECT_EQ(prompt_factory_->show_count(), GetParam() ? 5 : 4);
+ EXPECT_EQ(prompt_factory_->show_count(), 5);
+}
+
+// Verifies order of requests with mixed low-high priority requests input, with
+// both chip and quiet chip enabled. New permissions are added and accepted one
+// by one.
+//
+// Permissions requested in order:
+// 1. Multiple Download (high)
+// 2. Geolocation (low)
+// 3. Mic (high)
+//
+// Prompt display order:
+// 1. Mic
+// 2. Multiple Download
+// 3. Geolocation
+TEST_F(PermissionRequestManagerTest, Mixed1Low2HighPriorityRequests) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
+ WaitForBubbleToBeShown();
+
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ WaitForBubbleToBeShown();
+
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request_mic_.granted());
+ EXPECT_FALSE(request2_.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request2_.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request1_.granted());
+}
+
+// Verifies order of requests with mixed low-high priority requests input, with
+// both chip and quiet chip enabled. New permissions are added and accepted one
+// by one.
+//
+// Permissions requested in order:
+// 1. Geolocation (low)
+// 2. Mic (high)
+// 3. Notification (low)
+//
+// Prompt display order:
+// 1. Mic
+// 2. Notification
+// 3. Geolocation
+TEST_F(PermissionRequestManagerTest, Mixed2Low1HighRequests) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ WaitForBubbleToBeShown();
+
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
+ WaitForBubbleToBeShown();
+
+ MockPermissionRequest request(RequestType::kNotifications,
+ PermissionRequestGestureType::GESTURE);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request);
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request_mic_.granted());
+ EXPECT_FALSE(request.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request1_.granted());
+}
+
+// Verifies order of requests with mixed low-high priority requests input, added
+// simultaneously, with both chip and quiet chip enabled.
+//
+// Permissions requested in order:
+// 1. Geolocation (low)
+// 2. Mic (high)
+// 3. Notification (low)
+//
+// Prompt display order:
+// 1. Mic
+// 2. Notification
+// 3. Geolocation
+TEST_F(PermissionRequestManagerTest, MultipleSimultaneous2Low1HighRequests) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
+ MockPermissionRequest request(RequestType::kNotifications,
+ PermissionRequestGestureType::GESTURE);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request);
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request_mic_.granted());
+ EXPECT_FALSE(request.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request1_.granted());
+}
+
+// Verifies order of requests with mixed low-high priority requests input,
+// added simultaneously, with both chip and quiet chip enabled.
+//
+// Permissions requested in order:
+// 1. MIDI (high)
+// 2. Geolocation (low)
+// 3. Mic (high)
+// 4. Notification (low)
+// 5. Multiple Download (high)
+//
+// Prompt display order:
+// 1. Multiple Download
+// 2. Mic
+// 3. Midi
+// 4. Notification
+// 5. Geolocation
+TEST_F(PermissionRequestManagerTest, MultipleSimultaneous2Low3HighRequests) {
+ MockPermissionRequest request_midi(RequestType::kMidiSysex,
+ PermissionRequestGestureType::GESTURE);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_midi);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
+ MockPermissionRequest request_notification(
+ RequestType::kNotifications, PermissionRequestGestureType::GESTURE);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(),
+ &request_notification);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_FALSE(request_mic_.granted());
+ EXPECT_TRUE(request2_.granted());
+ EXPECT_FALSE(request_notification.granted());
+ EXPECT_FALSE(request1_.granted());
+ EXPECT_FALSE(request_midi.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request_mic_.granted());
+ EXPECT_FALSE(request_notification.granted());
+ EXPECT_FALSE(request1_.granted());
+ EXPECT_FALSE(request_midi.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_FALSE(request_notification.granted());
+ EXPECT_FALSE(request1_.granted());
+ EXPECT_TRUE(request_midi.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request_notification.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request1_.granted());
+}
+
+// Verifies order of requests with mixed low-high priority requests input, added
+// simultaneously several times, with both chip and quiet chip enabled.
+//
+// Permissions requested in order:
+// 1. Geolocation(low) then Notification(low)
+// 2. Mic (high) then multiple downloads (high)
+// Prompt display order:
+// 1. Multiple Download
+// 2. Mic
+// 3. Notification
+// 4. Geolocation
+TEST_F(PermissionRequestManagerTest, MultipleSimultaneous2Low2HighRequests) {
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request1_);
+ MockPermissionRequest request(RequestType::kNotifications,
+ PermissionRequestGestureType::GESTURE);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request);
+ WaitForBubbleToBeShown();
+
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request_mic_);
+ manager_->AddRequest(web_contents()->GetPrimaryMainFrame(), &request2_);
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request2_.granted());
+ EXPECT_FALSE(request_mic_.granted());
+ EXPECT_FALSE(request.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request_mic_.granted());
+ EXPECT_FALSE(request.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request.granted());
+ EXPECT_FALSE(request1_.granted());
+ WaitForBubbleToBeShown();
+
+ EXPECT_TRUE(prompt_factory_->is_visible());
+ EXPECT_EQ(prompt_factory_->request_count(), 1);
+ Accept();
+ EXPECT_TRUE(request1_.granted());
}
-INSTANTIATE_TEST_SUITE_P(All,
- PermissionRequestManagerTest,
- ::testing::Values(false, true));
-INSTANTIATE_TEST_SUITE_P(All,
- PermissionRequestManagerTestQuietChip,
- ::testing::Values(false, true));
+#endif // BUILDFLAG(IS_ANDROID)
} // namespace permissions