Make ClearAllContentSettingsRules() clear resource IDs as well.
commit0a3f5498e0a200298a4393b4b06f9876610499f6
authormsramek <msramek@chromium.org>
Tue, 14 Apr 2015 10:17:36 +0000 (14 03:17 -0700)
committerCommit bot <commit-bot@chromium.org>
Tue, 14 Apr 2015 10:19:08 +0000 (14 10:19 +0000)
treeaa60f5daef21bbe297e663ebb25f5db1addab6dc
parentf72d9e86374adbf35a01550db0fbfd3fe42456f6
Make ClearAllContentSettingsRules() clear resource IDs as well.

This means that if a resource identifier rule (e.g. plugin allowed on a site) is delete in an old instance of Chrome, the deletion will not propagate to a new version.

This change also affects ProfileResetter, which calls this function to delete all content settings rules; it did not delete rules with resource identifier before.

Since the OriginIdentifierValueMap now only contains one content type per instance, we can delete all rules (regardless of resource identifier) by simply calling clear(), which greatly improves the deletion speed as well.

BUG=475951

Review URL: https://codereview.chromium.org/1079133003

Cr-Commit-Position: refs/heads/master@{#325022}
chrome/browser/content_settings/content_settings_pref_provider_unittest.cc
chrome/browser/content_settings/host_content_settings_map_unittest.cc
components/content_settings/core/browser/content_settings_pref.cc