Issue 617585 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	treib@chromium.org
Closed:	Jul 2016
Cc:	█ dgn@chromium.org █ ramyan@chromium.org
Components:	UI>Browser>NewTabPage
EstimatedDays:	----
NextAction:	----
OS:	Android
Pri:	2
Type:	Bug
M-54

Remove deprecated prefs

Project Member Reported by dgn@chromium.org, Jun 6 2016

Issue description

M52 Shipped with some prefs that are removed in M53. M53 has code to clear them, so once that is done, M54 should remove references to them entirely.

prefs::kDeprecatedSnippets
prefs::kDeprecatedDiscardedSnippets

Project Member

Comment 1 by bugdroid1@chromium.org, Jul 11 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3a1752d314f7596c834f93fb20d78e893ce8b81a

commit 3a1752d314f7596c834f93fb20d78e893ce8b81a
Author: treib <treib@chromium.org>
Date: Mon Jul 11 08:33:42 2016

[NTP Snippets] Remove deprecated prefs

BUG= 617585 

Review-Url: https://codereview.chromium.org/2130363002
Cr-Commit-Position: refs/heads/master@{#404616}

[modify] https://crrev.com/3a1752d314f7596c834f93fb20d78e893ce8b81a/components/ntp_snippets/ntp_snippets_service.cc
[modify] https://crrev.com/3a1752d314f7596c834f93fb20d78e893ce8b81a/components/ntp_snippets/ntp_snippets_service.h
[modify] https://crrev.com/3a1752d314f7596c834f93fb20d78e893ce8b81a/components/ntp_snippets/pref_names.cc
[modify] https://crrev.com/3a1752d314f7596c834f93fb20d78e893ce8b81a/components/ntp_snippets/pref_names.h

Comment 2 by treib@chromium.org, Jul 11 2016

Status: Fixed (was: Assigned)

►

Issue 617585 link

Issue metadata

Remove deprecated prefs

Issue description

Comment 1 by bugdroid1@chromium.org, Jul 11 2016

Comment 1 Deleted

Comment 2 by treib@chromium.org, Jul 11 2016

Comment 2 Deleted

Sign in to add a comment