Clear Download section prefs on "Downloads history" removal. |
||
Issue descriptionAs of today, there is no "Downloads history" option in Clear Browsing Data dialogue. However, it should appear there and we have to remove dismissed items when it is triggered. The event is BrowsingDataRemover::RemoveDataMask::REMOVE_DOWNLOADS.
,
Nov 21 2016
So the event for that already exists? Are you sure that it doesn't get triggered somewhere else?
,
Nov 21 2016
> So the event for that already exists? I believe the event exists, because the code is shared with the desktop. On the desktop, there is such option in Clear Browsing Data dialogue. > Are you sure that it doesn't get triggered somewhere else? No, I am not, but I believe that then we would be told about this during the privacy review. However, only Clear Browsing Data dialogue was mentioned.
,
Nov 21 2016
I see. Ok, on desktop it makes sense. Because there is indeed a downloads history (even if the user deletes the downloaded file via the filesystem, Chrome still remembers the download). Since such a thing doesn't exist on Android, I'd say we stick to the YAGNI principle and close this bug. :-) SG?
,
Nov 21 2016
SGTM. Moreover, we clear this on Browser History removal. In case they add the option, we will reopen this bug. |
||
►
Sign in to add a comment |
||
Comment 1 by vitaliii@chromium.org
, Nov 17 2016