Use FileDescriptorWatcher in signal_handler.cc.

Use FileDescriptorWatcher in signal_handler.cc.

This allows RegisterSignalHandler to be used from any thread that
instantiates a FileDescriptorWatcher (not just threads that run
a MessageLoopForIO). This will facilitate the migration of
BrowserThreads to base/task_scheduler.

BUG=645114
Committed: https://crrev.com/aae14a15971bc8f64e591593bbe626b44839993a
Cr-Commit-Position: refs/heads/master@{#424472}

[fdoray, 2016-10-05 15:56:25]

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

[commit-bot: I haz the power, 2016-10-05 15:56:49]

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, 2016-10-05 16:03:04]

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

[commit-bot: I haz the power, 2016-10-05 16:03:05]

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

[fdoray, 2016-10-05 19:41:13]

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

[commit-bot: I haz the power, 2016-10-05 19:41:44]

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, 2016-10-05 19:48:47]

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

[commit-bot: I haz the power, 2016-10-05 19:48:47]

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_...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[fdoray, 2016-10-07 14:10:55]

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

[commit-bot: I haz the power, 2016-10-07 14:11:05]

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, 2016-10-07 14:35:04]

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

[commit-bot: I haz the power, 2016-10-07 14:35:05]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-10-07 17:22:41]

fdoray@chromium.org changed reviewers:
+ joedow@chromium.org

[fdoray, 2016-10-07 17:22:41]

PTAL

[joedow, 2016-10-11 16:53:44]

joedow@chromium.org changed reviewers:
+ jamiewalch@chromium.org

[joedow, 2016-10-11 16:53:45]

I'm not that familiar with this file, Jamie would be a better person to look at
this change.

[Jamie, 2016-10-11 17:02:51]

lgtm

[fdoray, 2016-10-11 17:05:19]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-10-11 17:05:45]

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-10-11 17:33:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-11 17:35:00]

Description was changed from

==========
Use FileDescriptorWatcher in signal_handler.cc.

This allows RegisterSignalHandler to be used from any thread that
instantiates a FileDescriptorWatcher (not just threads that run
a MessageLoopForIO). This will facilitate the migration of
BrowserThreads to base/task_scheduler.

BUG=645114
==========

to

==========
Use FileDescriptorWatcher in signal_handler.cc.

This allows RegisterSignalHandler to be used from any thread that
instantiates a FileDescriptorWatcher (not just threads that run
a MessageLoopForIO). This will facilitate the migration of
BrowserThreads to base/task_scheduler.

BUG=645114

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

[commit-bot: I haz the power, 2016-10-11 17:35:01]

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