Remove FILE thread dependency in protocol_handler_registry_unittest.cc

Remove FILE thread dependency in protocol_handler_registry_unittest.cc

BUG=724293, 689520
Review-Url: https://codereview.chromium.org/2896473002
Cr-Commit-Position: refs/heads/master@{#473349}
Committed: https://chromium.googlesource.com/chromium/src/+/fd04fe52772ad91203606d3797d5e7a8b6226c04

[Patrick Monette, 2017-05-18 20:23:16]

Description was changed from

==========
Change
BUG=
==========

to

==========
Remove FILE thread dependency in protocol_handler_registry_unittest.cc

BUG=689520
==========

[Patrick Monette, 2017-05-18 20:23:22]

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

[commit-bot: I haz the power, 2017-05-18 20:24:52]

Dry run: 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-18 20:38:18]

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

[commit-bot: I haz the power, 2017-05-18 20:38:18]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Patrick Monette, 2017-05-18 22:22:14]

Description was changed from

==========
Remove FILE thread dependency in protocol_handler_registry_unittest.cc

BUG=689520
==========

to

==========
Remove FILE thread dependency in protocol_handler_registry_unittest.cc

BUG=724293, 689520
==========

[Patrick Monette, 2017-05-18 22:33:35]

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

[commit-bot: I haz the power, 2017-05-18 22:35:25]

Dry run: 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-18 23:35:39]

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

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

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Patrick Monette, 2017-05-18 23:44:59]

pmonette@chromium.org changed reviewers:
+ benwells@chromium.org

[Patrick Monette, 2017-05-18 23:45:00]

benwells@ PTAL at protocol_handler_registry*
pkasting@ PTAL at content_setting_bubble_model_unittest.cc

[Patrick Monette, 2017-05-18 23:45:15]

pmonette@chromium.org changed reviewers:
+ pkasting@chromium.org

[Patrick Monette, 2017-05-18 23:45:15]

Actually adding Peter

[Peter Kasting, 2017-05-19 02:05:06]

LGTM

[benwells, 2017-05-19 02:59:49]

lgtm, thanks for cleaning this up

[Patrick Monette, 2017-05-19 17:43:48]

Thanks!

[Patrick Monette, 2017-05-19 17:43:51]

The CQ bit was checked by pmonette@chromium.org

[commit-bot: I haz the power, 2017-05-19 17:44: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, 2017-05-19 20:03:48]

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

[commit-bot: I haz the power, 2017-05-19 20:03:49]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Patrick Monette, 2017-05-19 21:06:47]

The CQ bit was checked by pmonette@chromium.org

[commit-bot: I haz the power, 2017-05-19 21:08:07]

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-19 22:04:30]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1495228007447510,
"parent_rev": "5c8ebdfa82f6b1e33312000c06c874faf9a088a5", "commit_rev":
"fd04fe52772ad91203606d3797d5e7a8b6226c04"}

[commit-bot: I haz the power, 2017-05-19 22:04:42]

Description was changed from

==========
Remove FILE thread dependency in protocol_handler_registry_unittest.cc

BUG=724293, 689520
==========

to

==========
Remove FILE thread dependency in protocol_handler_registry_unittest.cc

BUG=724293, 689520

Review-Url: https://codereview.chromium.org/2896473002
Cr-Commit-Position: refs/heads/master@{#473349}
Committed:
https://chromium.googlesource.com/chromium/src/+/fd04fe52772ad91203606d3797d5...
==========

[commit-bot: I haz the power, 2017-05-19 22:04:44]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/fd04fe52772ad91203606d3797d5...