Correct net::EmbeddedTestServer usage in some SSL related tests.

Correct net::EmbeddedTestServer usage in some SSL related tests.

Adding request handlers to the EmbeddedTestServer after it has started
isn't thread safe.

This CL updates some tests so that the EmbeddedTestServer's IO thread
is started after all handlers are added.

BUG=546060
Committed: https://crrev.com/839218498ade5aae2571b58325beef424905ba55
Cr-Commit-Position: refs/heads/master@{#431347}

[martijnc, 2016-11-09 21:23:49]

martijn@martijnc.be changed reviewers:
+ palmer@chromium.org

[martijnc, 2016-11-10 13:29:29]

Hi, can you review this patch?

Thanks!

[palmer, 2016-11-10 20:26:05]

lgtm

[martijnc, 2016-11-10 20:27:22]

The CQ bit was checked by martijn@martijnc.be

[commit-bot: I haz the power, 2016-11-10 20:27:44]

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-11-10 20:59:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-10 21:02:38]

Description was changed from

==========
Correct net::EmbeddedTestServer usage in some SSL related tests.

Adding request handlers to the EmbeddedTestServer after it has started
isn't thread safe.

This CL updates some tests so that the EmbeddedTestServer's IO thread
is started after all handlers are added.

BUG=546060
==========

to

==========
Correct net::EmbeddedTestServer usage in some SSL related tests.

Adding request handlers to the EmbeddedTestServer after it has started
isn't thread safe.

This CL updates some tests so that the EmbeddedTestServer's IO thread
is started after all handlers are added.

BUG=546060

Committed: https://crrev.com/839218498ade5aae2571b58325beef424905ba55
Cr-Commit-Position: refs/heads/master@{#431347}
==========

[commit-bot: I haz the power, 2016-11-10 21:02:39]

Patchset 1 (id:??) landed as
https://crrev.com/839218498ade5aae2571b58325beef424905ba55
Cr-Commit-Position: refs/heads/master@{#431347}