Remove activity_log's Util::DropObsoleteTables().

Remove activity_log's Util::DropObsoleteTables().

After three years, the obsolete tables are probably gone for good.

Committed: https://crrev.com/47a8860f5d7cd3fc9862b7bec4b6a8883b47d305
Cr-Commit-Position: refs/heads/master@{#417349}

[Lei Zhang, 2016-09-07 07:25:48]

The CQ bit was checked by thestig@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 07:25:57]

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

[Lei Zhang, 2016-09-07 07:27:47]

Description was changed from

==========
Remove activity_log's Util::DropObsoleteTables().

After three years, the obsolete tables are probably gone for good.
==========

to

==========
Remove activity_log's Util::DropObsoleteTables().

After three years, the obsolete tables are probably gone for good.
==========

[Lei Zhang, 2016-09-07 07:27:47]

thestig@chromium.org changed reviewers:
+ felt@chromium.org, mvrable@chromium.org

[commit-bot: I haz the power, 2016-09-07 08:13:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-07 08:13:50]

Dry run: This issue passed the CQ dry run.

[Lei Zhang, 2016-09-07 08:17:45]

Safe to delete the deleter?

https://codereview.chromium.org/2318833003/diff/1/chrome/browser/extensions/a...
File chrome/browser/extensions/activity_log/activity_log_policy.cc (left):

https://codereview.chromium.org/2318833003/diff/1/chrome/browser/extensions/a...
chrome/browser/extensions/activity_log/activity_log_policy.cc:29: // found.
This probably should have had a TODO attached to remove after a year or two.

[mvrable, 2016-09-07 20:25:03]

lgtm

I haven't been actively working on Chromium in a while, but these old database
tables have been obsolete for ~3 years, and probably the obsolete ones only ever
saw use by those of us working on the activity logging feature anyway.  I agree
that deleting the cleanup code for them now is a good idea.

[Lei Zhang, 2016-09-08 17:22:19]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2016-09-08 17:22:40]

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, 2016-09-08 18:33:47]

Description was changed from

==========
Remove activity_log's Util::DropObsoleteTables().

After three years, the obsolete tables are probably gone for good.
==========

to

==========
Remove activity_log's Util::DropObsoleteTables().

After three years, the obsolete tables are probably gone for good.
==========

[commit-bot: I haz the power, 2016-09-08 18:33:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-08 18:36:45]

Description was changed from

==========
Remove activity_log's Util::DropObsoleteTables().

After three years, the obsolete tables are probably gone for good.
==========

to

==========
Remove activity_log's Util::DropObsoleteTables().

After three years, the obsolete tables are probably gone for good.

Committed: https://crrev.com/47a8860f5d7cd3fc9862b7bec4b6a8883b47d305
Cr-Commit-Position: refs/heads/master@{#417349}
==========

[commit-bot: I haz the power, 2016-09-08 18:36:46]

Patchset 1 (id:??) landed as
https://crrev.com/47a8860f5d7cd3fc9862b7bec4b6a8883b47d305
Cr-Commit-Position: refs/heads/master@{#417349}