DescriptionDon't delete the PermissionPromptAndroid when switching tabs
The PermissionRequestManager creates and deletes its PermissionPrompt
when switching to and from a given tab. On desktop this is the mechanism
by which we hide/show permission prompts when switching tabs. On Android
however, we use infobars which already have their visibility managed, so
it's odd for the permissions code to be deleting and re-creating the
infobars instead of letting them work normally.
This patch makes us leave the PermissionPromptAndroid alone instead of
deleting and re-creating it on switching tabs. While this makes our
model on Android differ from desktop, this lets the permission infobars
be managed normally, and the PermissionPromptAndroid won't need to keep
a reference to the infobar.
This is mostly a revert of https://codereview.chromium.org/2522373002.
BUG=606138
Review-Url: https://codereview.chromium.org/2781803002
Cr-Commit-Position: refs/heads/master@{#461042}
Committed: https://chromium.googlesource.com/chromium/src/+/f9de5648278de3e68a7c7ef87215a8d3128eb763
Patch Set 1 #Patch Set 2 : . #
Total comments: 2
Dependent Patchsets: Messages
Total messages: 29 (17 generated)
|