Correct net::EmbeddedTestServer usage in google_apis_unittests.

Adding EmbeddedTestServer handlers after the server has started isn't thread safe.

This CL updates a couple google_apis/drive/* tests to make EmbeddedTestServer thread safe. Either by starting the server after the handlers have been added or by only initializing the server before adding the handlers.

BUG=546060
Committed: https://crrev.com/98e39c76641d4f66dcc1203fb19f8c7f248ee248
Cr-Commit-Position: refs/heads/master@{#429836}

[martijnc, 2016-11-03 17:37:29]

Description was changed from

==========
Correct net::EmbeddedTestServer usage in google_apis_unittests.

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

to

==========
Adding EmbeddedTestServer handlers after the server has started isn't thread
safe.

This CL updates a couple google_apis/drive/* tests to make EmbeddedTestServer
thread safe. Either by starting the server after the handlers have been added or
by only initializing the server before adding the handlers.

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

[martijnc, 2016-11-03 17:37:29]

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

[martijnc, 2016-11-03 17:40:18]

Hi, can you review this patch?

Thanks!

[hidehiko, 2016-11-04 00:02:26]

Thank you for cleaning up. LGTM.

[martijnc, 2016-11-04 08:41:45]

The CQ bit was checked by martijn@martijnc.be

[commit-bot: I haz the power, 2016-11-04 08:42:10]

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-04 08:55:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-04 08:59:04]

Description was changed from

==========
Adding EmbeddedTestServer handlers after the server has started isn't thread
safe.

This CL updates a couple google_apis/drive/* tests to make EmbeddedTestServer
thread safe. Either by starting the server after the handlers have been added or
by only initializing the server before adding the handlers.

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

to

==========
Adding EmbeddedTestServer handlers after the server has started isn't thread
safe.

This CL updates a couple google_apis/drive/* tests to make EmbeddedTestServer
thread safe. Either by starting the server after the handlers have been added or
by only initializing the server before adding the handlers.

BUG=546060

Committed: https://crrev.com/98e39c76641d4f66dcc1203fb19f8c7f248ee248
Cr-Commit-Position: refs/heads/master@{#429836}
==========

[commit-bot: I haz the power, 2016-11-04 08:59:05]

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