Get rid of ExtensionServiceTestBase::ResetThreadBundle

Get rid of ExtensionServiceTestBase::ResetThreadBundle

The method is fundamentally flawed (it invalidates the TaskRunners for
anything that might have already be initialized), and also isn't
actually used other than in one particular test that doesn't seem to
need what it does anyway.

BUG=None

Review-Url: https://codereview.chromium.org/2863023002
Cr-Commit-Position: refs/heads/master@{#469563}
Committed: https://chromium.googlesource.com/chromium/src/+/82cbc419c9725b0d7dddb403bce2f367638dfb35

[Marijn Kruisselbrink, 2017-05-04 23:34:57]

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

[commit-bot: I haz the power, 2017-05-04 23:35:35]

Dry run: CQ is trying da patch.

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

[Marijn Kruisselbrink, 2017-05-04 23:48:37]

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

[commit-bot: I haz the power, 2017-05-05 00:08:47]

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

[commit-bot: I haz the power, 2017-05-05 00:08:47]

Dry run: This issue passed the CQ dry run.

[Devlin, 2017-05-05 01:12:24]

LGTM.

if everything still works without this, I'm happy to have it gone!  Next step -
avoid running all these tests on an IO_MAINLOOP.

[Marijn Kruisselbrink, 2017-05-05 01:13:38]

The CQ bit was checked by mek@chromium.org

[commit-bot: I haz the power, 2017-05-05 01:14:03]

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-05-05 01:19:10]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493946818940490, "parent_rev":
"bfbe9c7802a991ba8aac2664970947fd18e4748c", "commit_rev":
"82cbc419c9725b0d7dddb403bce2f367638dfb35"}

[commit-bot: I haz the power, 2017-05-05 01:19:54]

Description was changed from

==========
Get rid of ExtensionServiceTestBase::ResetThreadBundle

The method is fundamentally flawed (it invalidates the TaskRunners for
anything that might have already be initialized), and also isn't
actually used other than in one particular test that doesn't seem to
need what it does anyway.

BUG=None
==========

to

==========
Get rid of ExtensionServiceTestBase::ResetThreadBundle

The method is fundamentally flawed (it invalidates the TaskRunners for
anything that might have already be initialized), and also isn't
actually used other than in one particular test that doesn't seem to
need what it does anyway.

BUG=None

Review-Url: https://codereview.chromium.org/2863023002
Cr-Commit-Position: refs/heads/master@{#469563}
Committed:
https://chromium.googlesource.com/chromium/src/+/82cbc419c9725b0d7dddb403bce2...
==========

[commit-bot: I haz the power, 2017-05-05 01:19:55]

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