Commit d3a2a83f authored by benwells's avatar benwells Committed by Commit bot

Fix the Update Permissions infobar for geolocation.

Due to some earlier changes to do with mapping chrome to android
permissions, this infobar incorrectly thinks there are multiple
permissions being requested for geolocation.

BUG=709206

Review-Url: https://codereview.chromium.org/2807093002
Cr-Commit-Position: refs/heads/master@{#463418}
parent 583a61ad
......@@ -47,11 +47,13 @@ infobars::InfoBar* PermissionUpdateInfoBarDelegate::Create(
if (missing_permission_count > 1)
continue;
bool has_all_permissions = true;
for (auto it = permissions.begin() + previous_size; it != permissions.end();
++it) {
if (window_android->HasPermission(*it))
continue;
has_all_permissions &= window_android->HasPermission(*it);
}
if (!has_all_permissions) {
missing_permission_count++;
if (missing_permission_count > 1) {
message_id = IDS_INFOBAR_MISSING_MULTIPLE_PERMISSIONS_TEXT;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment