ExtensionPrefs: remove MigratePermissions

ExtensionPrefs: remove MigratePermissions

The pref change happened > 4 years ago (https://codereview.chromium.org/7003098), let's assume everything's migrated.

BUG=none

Committed: https://crrev.com/df09ded9d970e6847779d824faf887575f6c9fea
Cr-Commit-Position: refs/heads/master@{#370169}

[Marc Treib, 2016-01-15 17:05:56]

treib@chromium.org changed reviewers:
+ rdevlin.cronin@chromium.org

[Marc Treib, 2016-01-15 17:05:57]

Aaand one more. (That's the last one for today, I promise!)

[Devlin, 2016-01-15 18:00:59]

Good find.  Let this be an example of a reason to always put a TODO(foo): Remove
at such-and-such mstone/date/anything.  Then someone probably would have done
this after only *three* years. ;)

[Devlin, 2016-01-15 18:01:31]

hit publish too early.  Meant to also say "but it looks like you have failing
tests."

[Marc Treib, 2016-01-15 18:53:54]

On 2016/01/15 18:01:31, Devlin wrote:
> hit publish too early.  Meant to also say "but it looks like you have failing
> tests."

Hah, I do indeed. Looks like some extensions test data still uses the old prefs:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/test/data/e...
Last changed 4 years ago :D
I'll fix it next week.

[Marc Treib, 2016-01-18 10:37:47]

On 2016/01/15 18:53:54, Marc Treib wrote:
> On 2016/01/15 18:01:31, Devlin wrote:
> > hit publish too early.  Meant to also say "but it looks like you have
failing
> > tests."
> 
> Hah, I do indeed. Looks like some extensions test data still uses the old
prefs:
>
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/test/data/e...
> Last changed 4 years ago :D
> I'll fix it next week.

Test data updated, all green now! :)

[Devlin, 2016-01-19 18:27:14]

lgtm

[Marc Treib, 2016-01-19 18:37:10]

The CQ bit was checked by treib@chromium.org

[commit-bot: I haz the power, 2016-01-19 18:38:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1589403002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1589403002/20001

[commit-bot: I haz the power, 2016-01-19 19:48:24]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-19 19:49:12]

Description was changed from

==========
ExtensionPrefs: remove MigratePermissions

The pref change happened > 4 years ago
(https://codereview.chromium.org/7003098), let's assume everything's migrated.

BUG=none
==========

to

==========
ExtensionPrefs: remove MigratePermissions

The pref change happened > 4 years ago
(https://codereview.chromium.org/7003098), let's assume everything's migrated.

BUG=none

Committed: https://crrev.com/df09ded9d970e6847779d824faf887575f6c9fea
Cr-Commit-Position: refs/heads/master@{#370169}
==========

[commit-bot: I haz the power, 2016-01-19 19:49:13]

Patchset 2 (id:??) landed as
https://crrev.com/df09ded9d970e6847779d824faf887575f6c9fea
Cr-Commit-Position: refs/heads/master@{#370169}