diff options
author | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2020-10-12 14:27:29 +0200 |
---|---|---|
committer | Allan Sandfeld Jensen <allan.jensen@qt.io> | 2020-10-13 09:35:20 +0000 |
commit | c30a6232df03e1efbd9f3b226777b07e087a1122 (patch) | |
tree | e992f45784689f373bcc38d1b79a239ebe17ee23 /chromium/components/permissions/permission_request_manager.cc | |
parent | 7b5b123ac58f58ffde0f4f6e488bcd09aa4decd3 (diff) | |
download | qtwebengine-chromium-85-based.tar.gz |
BASELINE: Update Chromium to 85.0.4183.14085-based
Change-Id: Iaa42f4680837c57725b1344f108c0196741f6057
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'chromium/components/permissions/permission_request_manager.cc')
-rw-r--r-- | chromium/components/permissions/permission_request_manager.cc | 45 |
1 files changed, 35 insertions, 10 deletions
diff --git a/chromium/components/permissions/permission_request_manager.cc b/chromium/components/permissions/permission_request_manager.cc index b16e1e1a943..60234f3733d 100644 --- a/chromium/components/permissions/permission_request_manager.cc +++ b/chromium/components/permissions/permission_request_manager.cc @@ -15,7 +15,6 @@ #include "base/metrics/user_metrics_action.h" #include "base/stl_util.h" #include "base/strings/string16.h" -#include "base/task/post_task.h" #include "base/threading/sequenced_task_runner_handle.h" #include "build/build_config.h" #include "components/permissions/features.h" @@ -48,6 +47,20 @@ const char kAbusiveNotificationRequestsWarningMessage[] = "fix the issues as soon as possible and submit your site for another " "review. Learn more at https://support.google.com/webtools/answer/9799048."; +constexpr char kAbusiveNotificationContentEnforcementMessage[] = + "Chrome is blocking notification permission requests on this site because " + "the site tends to show notifications with content that mislead or trick " + "users. You should fix the issues as soon as possible and submit your site " + "for another review. Learn more at " + "https://support.google.com/webtools/answer/9799048"; + +constexpr char kAbusiveNotificationContentWarningMessage[] = + "Chrome might start blocking notification permission requests on this site " + "in the future because the site tends to show notifications with content " + "that mislead or trick users. You should fix the issues as soon as " + "possible and submit your site for another review. Learn more at " + "https://support.google.com/webtools/answer/9799048"; + namespace { bool IsMessageTextEqual(PermissionRequest* a, PermissionRequest* b) { @@ -358,8 +371,8 @@ PermissionRequestManager::PermissionRequestManager( void PermissionRequestManager::ScheduleShowBubble() { base::RecordAction(base::UserMetricsAction("PermissionBubbleRequest")); - base::PostTask(FROM_HERE, {content::BrowserThread::UI}, - base::BindOnce(&PermissionRequestManager::ShowBubble, + content::GetUIThreadTaskRunner({})->PostTask( + FROM_HERE, base::BindOnce(&PermissionRequestManager::ShowBubble, weak_factory_.GetWeakPtr())); } @@ -421,18 +434,30 @@ void PermissionRequestManager::ShowBubble() { PermissionUmaUtil::PermissionPromptShown(requests_); if (ShouldCurrentRequestUseQuietUI()) { - if (ReasonForUsingQuietUi() == - QuietUiReason::kTriggeredDueToAbusiveRequests) { - LogWarningToConsole(kAbusiveNotificationRequestsEnforcementMessage); + switch (ReasonForUsingQuietUi()) { + case QuietUiReason::kEnabledInPrefs: + case QuietUiReason::kTriggeredByCrowdDeny: + break; + case QuietUiReason::kTriggeredDueToAbusiveRequests: + LogWarningToConsole(kAbusiveNotificationRequestsEnforcementMessage); + break; + case QuietUiReason::kTriggeredDueToAbusiveContent: + LogWarningToConsole(kAbusiveNotificationContentEnforcementMessage); + break; } base::RecordAction(base::UserMetricsAction( "Notifications.Quiet.PermissionRequestShown")); } - if (current_request_ui_to_use_->warning_reason && - *(current_request_ui_to_use_->warning_reason) == - WarningReason::kAbusiveRequests) { - LogWarningToConsole(kAbusiveNotificationRequestsWarningMessage); + if (current_request_ui_to_use_->warning_reason) { + switch (*(current_request_ui_to_use_->warning_reason)) { + case WarningReason::kAbusiveRequests: + LogWarningToConsole(kAbusiveNotificationRequestsWarningMessage); + break; + case WarningReason::kAbusiveContent: + LogWarningToConsole(kAbusiveNotificationContentWarningMessage); + break; + } } } current_request_view_shown_to_user_ = true; |