Chromium Code Reviews
DescriptionRemove |accept_states_| from PermissionRequestManager
PermissionRequestManager previously stored accept_states_ for tracking
the state of the Allow/Block drop-down for each permission in a grouped
permission request. These drop-downs are being removed, so this removes
accept_states. There can still be grouped permission prompts though (for
mic/camera) so this changes the behavior of Accept() to accept all
permissions in the request, and Deny() to deny all permissions in the
request. The UMA recording functions have been updated accordingly.
BUG=728483
Review-Url: https://codereview.chromium.org/2922933002
Cr-Commit-Position: refs/heads/master@{#477439}
Committed: https://chromium.googlesource.com/chromium/src/+/85102c98e4932dba40d36b19c95cff077c86e0ba
Patch Set 1 #Patch Set 2 : Remove |accept_states_| from PermissionRequestManager #
Depends on Patchset: Dependent Patchsets: Messages
Total messages: 14 (10 generated)
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||