GitLab

With this CL, it is guaranteed that the file descriptor passed to
FileDescriptorWatcher::Watch(Readable|Writable) isn't used after
the returned Controller has been destroyed.

To enforce this when the MessageLoopForIO used to watch the file
descriptor lives on a separate thread, the destructor of
Controller uses a WaitableEvent to wait until all work on this
separate thread is done.

Bug: 741188
Change-Id: Ie1a5ead5324ae90ce1d73339cfc191c589590222
Reviewed-on: https://chromium-review.googlesource.com/c/695914
Commit-Queue: François Doray <fdoray@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#627021}