Issue metadata
Sign in to add a comment
|
Remove last usage functions from HostContentSettingsMap and Prefs |
||||||||||||||||||||||
Issue descriptionLast usage has been unused for over two years. Raymes previously had a patch up at https://codereview.chromium.org/1411653004 to remove these. Since there are still no concrete plans to actually use this, we should remove it and clean up the last_usage prefs.
,
Mar 7 2017
,
Jul 6 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e369fbdecbe1d12a2665218c5c9e3ae1d2e563d9 commit e369fbdecbe1d12a2665218c5c9e3ae1d2e563d9 Author: Timothy Loh <timloh@chromium.org> Date: Thu Jul 06 00:21:20 2017 Remove last usage pref clean-up This patch removes code for cleaning up last usage prefs. This code was around in M58/59/60, so any active user of Chrome will have had these prefs cleaned up. Bug: 691893 Change-Id: I6abf643c78636d6eb5cfb71d531a98dbee8edf63 Reviewed-on: https://chromium-review.googlesource.com/558842 Reviewed-by: Raymes Khoury <raymes@chromium.org> Commit-Queue: Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#484405} [modify] https://crrev.com/e369fbdecbe1d12a2665218c5c9e3ae1d2e563d9/chrome/browser/content_settings/content_settings_pref_provider_unittest.cc [modify] https://crrev.com/e369fbdecbe1d12a2665218c5c9e3ae1d2e563d9/components/content_settings/core/browser/content_settings_pref_provider.cc |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by bugdroid1@chromium.org
, Feb 21 2017