tree 5f2050ac302ab84109856dc03a5f1326743ca7da
parent d6bfeb0d8c0ea591184951aac9fd7bbbabc557d6
author Maggie Cai <mxcai@chromium.org> 1551765494 +0000
committer Commit Bot <commit-bot@chromium.org> 1551765494 +0000

Settings: Stop repopulate the all sites list on permission changes.

Currently on permission changes, the all sites list will be
re-populated. This is a issue when we reset the permission for an
etld+1. Because this resets every single permission settings for all the
origins under the etld+1, which triggers 16*n (n=number of origins) on
permission changes, therefore makes the all sites list repopulate that
many time. This affect the performance, also make the list jump back to
the top and lost the last selected entry. This CL removes the event
listener for permission change in all sites page. In the future, we
should implement the similar banner with reload when permission changes
from other places (not all sites page) occurs.

BUG=835712

Change-Id: I54596b1ba7d9901bdcd3ddd2192c1a0973aa5acf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1496118
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Maggie Cai <mxcai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#637608}