-
Timothy Loh authored
This patch fixes logging of ignored permission prompts so as to not log in cases where prompts weren't shown (background tabs, queued requests). Currently we log in ~PermissionRequestImpl; this patch moves it to PermissionRequestManager::FinalizeBubble, which is the same place as we use to log site engagement metrics for ignores. Repeated ignores embargo is similarly moved. BUG=774353 Change-Id: Ibcc1c09d63d58106d2db1640bfcbed003ded4169 Reviewed-on: https://chromium-review.googlesource.com/714798 Commit-Queue: Timothy Loh <timloh@chromium.org> Reviewed-by: Trent Apted <tapted@chromium.org> Reviewed-by: Raymes Khoury <raymes@chromium.org> Cr-Commit-Position: refs/heads/master@{#508986}
ab06431d