Virtualize DataUseRecorder's destructor

Virtualize DataUseRecorder's destructor

DataUseRecorder's destructor is not virtual and ChromeDataUseRecorder
inherits from it. While there are no delete's called on DataUseRecorder
pointers, this may not hold true in the future.

BUG=676147
Committed: https://crrev.com/35bfb2b24de94145f1aac42a41c73e11427281d4
Cr-Commit-Position: refs/heads/master@{#439941}

[RyanSturm, 2016-12-20 22:55:20]

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

[commit-bot: I haz the power, 2016-12-20 22:56:15]

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

[RyanSturm, 2016-12-20 22:56:36]

ryansturm@chromium.org changed reviewers:
+ sclittle@chromium.org

[RyanSturm, 2016-12-20 22:56:37]

sclittle: PTAL

[sclittle, 2016-12-20 22:57:09]

LGTM

[commit-bot: I haz the power, 2016-12-20 23:48:14]

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

[commit-bot: I haz the power, 2016-12-20 23:48:15]

Dry run: This issue passed the CQ dry run.

[RyanSturm, 2016-12-20 23:53:32]

The CQ bit was checked by ryansturm@chromium.org

[commit-bot: I haz the power, 2016-12-20 23:53:59]

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-12-21 00:03:24]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482278012083020, "parent_rev":
"a91a883c0e05b1217c1fd159cb75ea0b4763b1fb", "commit_rev":
"e0a186993bbcbd1ce663f0a7ef0ed6bc696ea32e"}

[commit-bot: I haz the power, 2016-12-21 00:03:48]

Description was changed from

==========
Virtualize DataUseRecorder's destructor

DataUseRecorder's destructor is not virtual and ChromeDataUseRecorder
inherits from it. While there are no delete's called on DataUseRecorder
pointers, this may not hold true in the future.

BUG=676147
==========

to

==========
Virtualize DataUseRecorder's destructor

DataUseRecorder's destructor is not virtual and ChromeDataUseRecorder
inherits from it. While there are no delete's called on DataUseRecorder
pointers, this may not hold true in the future.

BUG=676147

Review-Url: https://codereview.chromium.org/2592653005
==========

[commit-bot: I haz the power, 2016-12-21 00:03:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-21 00:05:53]

Description was changed from

==========
Virtualize DataUseRecorder's destructor

DataUseRecorder's destructor is not virtual and ChromeDataUseRecorder
inherits from it. While there are no delete's called on DataUseRecorder
pointers, this may not hold true in the future.

BUG=676147

Review-Url: https://codereview.chromium.org/2592653005
==========

to

==========
Virtualize DataUseRecorder's destructor

DataUseRecorder's destructor is not virtual and ChromeDataUseRecorder
inherits from it. While there are no delete's called on DataUseRecorder
pointers, this may not hold true in the future.

BUG=676147

Committed: https://crrev.com/35bfb2b24de94145f1aac42a41c73e11427281d4
Cr-Commit-Position: refs/heads/master@{#439941}
==========

[commit-bot: I haz the power, 2016-12-21 00:05:55]

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