• Balazs Engedy's avatar
    Introduce interface NotificationPermissionUiSelector. · e9934acb
    Balazs Engedy authored
    Rename PermissionRequestAutoBlocker to NotificationPermissionUiSelector
    to better reflect its current responsibility, and to disambiguate from
    PermissionDecisionAutoBlocker.
    
    Factor out all logic from ShouldCurrentRequestUseQuietUI() into an
    NotificationPermissionUiSelector implementation, in preparation for the
    drop-in replacement ContextualNotifictionPermissionUiSelector from a
    follow-up CL.
    
    Also update calls sites on Android to call the PermissionRequestManager
    instead of the QuietNotificationPermissionUiState, so that these call
    sites will be able to tolerate showing only a fraction of requests
    using the quiet UI once Safe Browsing based triggering is enabled.
    
    Bug: 1030633
    Change-Id: I355072e0f40cae665536c66b0631d8e609be9b8f
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1951105Reviewed-by: default avatarAndy Paicu <andypaicu@chromium.org>
    Commit-Queue: Balazs Engedy <engedy@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#721690}
    e9934acb
permission_request_manager.cc 23.3 KB