GitLab

And updates the functions to better reflect what it is suppose to do. There was
a race conditions with how the code was before. In particular, because
access to content has to be on the io-thread care must be taken during
destruction. Creation of the interfaces happens on the io thread, which means
destruction of the object supplied to AddInterface() has to happen on the
io-thread. I separated out the parts that happen on the io thread into a
refcounted thread safe that is destroyed on the io thread.

BUG=837686
TEST=covered by tests

Change-Id: Ia3dce4894a3cf4b7a08cd5a9c4d1fae674d7e7c5
Reviewed-on: https://chromium-review.googlesource.com/1105611Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#568469}