Use FileDescriptorWatcher in BrlapiConnectionImpl.

Use FileDescriptorWatcher in BrlapiConnectionImpl.

This allows BrlapiConnectionImpl 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/ddf00b80457978be7e72730b1f78d4c411572f6e
Cr-Commit-Position: refs/heads/master@{#423602}

[fdoray, 2016-10-03 18:36:46]

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

[commit-bot: I haz the power, 2016-10-03 18:37: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, 2016-10-03 18:49:23]

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

[commit-bot: I haz the power, 2016-10-03 18:49:23]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[fdoray, 2016-10-03 18:55:35]

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

[commit-bot: I haz the power, 2016-10-03 18:56: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, 2016-10-03 19:20:11]

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

[commit-bot: I haz the power, 2016-10-03 19:20:13]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-10-03 19:52:35]

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

[fdoray, 2016-10-03 19:52:36]

PTAL

[David Tseng, 2016-10-06 18:06:17]

lgtm

[fdoray, 2016-10-06 18:15:41]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-10-06 18:16:17]

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-06 18:42:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-06 18:44:41]

Description was changed from

==========
Use FileDescriptorWatcher in BrlapiConnectionImpl.

This allows BrlapiConnectionImpl 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 BrlapiConnectionImpl.

This allows BrlapiConnectionImpl 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/ddf00b80457978be7e72730b1f78d4c411572f6e
Cr-Commit-Position: refs/heads/master@{#423602}
==========

[commit-bot: I haz the power, 2016-10-06 18:44:42]

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