Android: Fix ChildThreadImpl::ShutdownThread()

Android: Fix ChildThreadImpl::ShutdownThread()

Make this work independent for MsgLoop vs. ChildThread destruction
order and simplify the code a bit.

BUG=465397
Committed: https://crrev.com/d294ee3e94047801a1d3bdf662b4176b64c69207
Cr-Commit-Position: refs/heads/master@{#330655}

[no sievers, 2015-05-19 21:33:36]

sievers@chromium.org changed reviewers:
+ piman@chromium.org

[no sievers, 2015-05-19 21:33:43]

ptal

[no sievers, 2015-05-19 21:40:30]

sievers@chromium.org changed reviewers:
- piman@chromium.org

[no sievers, 2015-05-19 21:53:10]

sievers@chromium.org changed reviewers:
+ piman@chromium.org

[piman, 2015-05-19 22:36:29]

lgtm

https://codereview.chromium.org/1140633005/diff/40001/content/child/child_thr...
File content/child/child_thread_impl.cc (right):

https://codereview.chromium.org/1140633005/diff/40001/content/child/child_thr...
content/child/child_thread_impl.cc:210: cond_var_.Signal();
nit: You can signal while the lock is taken, it's actually the usual pattern.
But this works (I don't think it matters in this case).

[no sievers, 2015-05-19 22:41:55]

https://codereview.chromium.org/1140633005/diff/40001/content/child/child_thr...
File content/child/child_thread_impl.cc (right):

https://codereview.chromium.org/1140633005/diff/40001/content/child/child_thr...
content/child/child_thread_impl.cc:210: cond_var_.Signal();
On 2015/05/19 22:36:29, piman (Very slow to review) wrote:
> nit: You can signal while the lock is taken, it's actually the usual pattern.
> But this works (I don't think it matters in this case).

Done.

[no sievers, 2015-05-19 22:43:17]

The CQ bit was checked by sievers@chromium.org

[no sievers, 2015-05-19 22:43:17]

The patchset sent to the CQ was uploaded after l-g-t-m from piman@chromium.org
Link to the patchset: https://codereview.chromium.org/1140633005/#ps60001
(title: "address comment")

[commit-bot: I haz the power, 2015-05-19 22:43:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1140633005/60001

[commit-bot: I haz the power, 2015-05-20 00:27:48]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-20 00:28:54]

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