Removing old prefs storage for BudgetManager

Removing old prefs storage for BudgetManager

Originally, the BudgetManager stored the budget for an origin in the
prefs in kBacgkroundBudgetMap. As the data stored became more complicated,
it switched to using the BudgetDatabase, which is a LevelDB wrapper. The
prefs storage mechanism was deprecated in Chrome 56 and code added to
cleanup any stored data there.

Now that Chrome 57 has branched, we can remove the cleanup code and any
remaining prefs artifacts.

BUG=617971
Review-Url: https://codereview.chromium.org/2682073002
Cr-Commit-Position: refs/heads/master@{#449035}
Committed: https://chromium.googlesource.com/chromium/src/+/24fb1c0d8ccf3d2ed71d341d925ac337253bfef3

[harkness, 2017-02-08 11:56:29]

harkness@chromium.org changed reviewers:
+ peter@chromium.org

[Peter Beverloo, 2017-02-08 12:12:59]

lgtm

[harkness, 2017-02-08 12:15:00]

harkness@chromium.org changed reviewers:
+ battre@chromium.org

[harkness, 2017-02-08 12:15:00]

battre@chromium.org: Please review changes in
chrome/browser/prefs/browser_prefs.cc.

[harkness, 2017-02-08 15:47:32]

harkness@chromium.org changed reviewers:
+ bauerb@google.com
- battre@chromium.org

[harkness, 2017-02-08 15:47:32]

bauerb@ please take a look at chrome/browser/prefs/browser_prefs.cc.

Removing battre@ since he's OOO today and tomorrow.

[Bernhard Bauer, 2017-02-08 16:49:47]

bauerb@chromium.org changed reviewers:
+ bauerb@chromium.org

[Bernhard Bauer, 2017-02-08 16:49:48]

LGTM!

[Bernhard Bauer, 2017-02-08 16:49:57]

Description was changed from

==========
Removing old prefs storage for BudgetManager

Originally, the BudgetManager stored the budget for an origin in the
prefs in kBacgkroundBudgetMap. As the data stored became more complicated,
it switched to using the BudgetDatabase, which is a LevelDB wrapper. The
prefs storage mechanism was deprecated in Chrome 56 and code added to
cleanup any stored data there.

Now that Chrome 57 has branched, we can remove the cleanup code and any
remaining prefs artifacts.

BUG=617971
==========

to

==========
Removing old prefs storage for BudgetManager

Originally, the BudgetManager stored the budget for an origin in the
prefs in kBacgkroundBudgetMap. As the data stored became more complicated,
it switched to using the BudgetDatabase, which is a LevelDB wrapper. The
prefs storage mechanism was deprecated in Chrome 56 and code added to
cleanup any stored data there.

Now that Chrome 57 has branched, we can remove the cleanup code and any
remaining prefs artifacts.

BUG=617971
==========

[Bernhard Bauer, 2017-02-08 16:49:57]

bauerb@chromium.org changed reviewers:
- bauerb@google.com

[harkness, 2017-02-08 16:55:52]

The CQ bit was checked by harkness@chromium.org

[commit-bot: I haz the power, 2017-02-08 16:56:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-08 18:04:00]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486572952054090, "parent_rev":
"f6fb2a71b8b6c25094720ab815204250d2debb00", "commit_rev":
"24fb1c0d8ccf3d2ed71d341d925ac337253bfef3"}

[commit-bot: I haz the power, 2017-02-08 18:04:36]

Description was changed from

==========
Removing old prefs storage for BudgetManager

Originally, the BudgetManager stored the budget for an origin in the
prefs in kBacgkroundBudgetMap. As the data stored became more complicated,
it switched to using the BudgetDatabase, which is a LevelDB wrapper. The
prefs storage mechanism was deprecated in Chrome 56 and code added to
cleanup any stored data there.

Now that Chrome 57 has branched, we can remove the cleanup code and any
remaining prefs artifacts.

BUG=617971
==========

to

==========
Removing old prefs storage for BudgetManager

Originally, the BudgetManager stored the budget for an origin in the
prefs in kBacgkroundBudgetMap. As the data stored became more complicated,
it switched to using the BudgetDatabase, which is a LevelDB wrapper. The
prefs storage mechanism was deprecated in Chrome 56 and code added to
cleanup any stored data there.

Now that Chrome 57 has branched, we can remove the cleanup code and any
remaining prefs artifacts.

BUG=617971

Review-Url: https://codereview.chromium.org/2682073002
Cr-Commit-Position: refs/heads/master@{#449035}
Committed:
https://chromium.googlesource.com/chromium/src/+/24fb1c0d8ccf3d2ed71d341d925a...
==========

[commit-bot: I haz the power, 2017-02-08 18:04:37]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/24fb1c0d8ccf3d2ed71d341d925a...