[Cronet] Delay StartNetLog and StopNetLog until native request context is initialized

[Cronet] Delay StartNetLog and StopNetLog until native request context is initialized

Use CronetURLRequestAdapter::PostTaskToNetworkThread instead of posting the task
directly to network thread to make sure the start/stop netlog task is executed
after request context is fully initialized.

Also changed the test cases to make sure that we don't regress.

BUG=470196
Committed: https://crrev.com/bb7cf7be5f6204564859331576e0e8b0364e038a
Cr-Commit-Position: refs/heads/master@{#322577}

[xunjieli, 2015-03-26 17:21:08]

xunjieli@chromium.org changed reviewers:
+ pauljensen@chromium.org

[pauljensen, 2015-03-27 13:11:13]

lgtm.  Do the two tests pass without the cronet_url_request_context_adapter.cc
fix?

https://codereview.chromium.org/996163005/diff/1/components/cronet/android/cr...
File components/cronet/android/cronet_url_request_context_adapter.cc (right):

https://codereview.chromium.org/996163005/diff/1/components/cronet/android/cr...
components/cronet/android/cronet_url_request_context_adapter.cc:286: // Do
nothing if already logging to a file.
Can we add:
DCHECK(is_context_initialized_);
DCHECK(context_);

[xunjieli, 2015-03-27 13:34:14]

Without the fix, CronetUrlRequestContextTest#testNetLog will fail. However,
CronetUrlTest#testNetLog tests the old API (which does not have this problem),
so it won't fail, but I changed it so it starts netlog right after context
creation (to match the test case in CronetUrlRequestContextTest#testNetLog).

https://codereview.chromium.org/996163005/diff/1/components/cronet/android/cr...
File components/cronet/android/cronet_url_request_context_adapter.cc (right):

https://codereview.chromium.org/996163005/diff/1/components/cronet/android/cr...
components/cronet/android/cronet_url_request_context_adapter.cc:286: // Do
nothing if already logging to a file.
On 2015/03/27 13:11:13, pauljensen wrote:
> Can we add:
> DCHECK(is_context_initialized_);
> DCHECK(context_);

Done.

[xunjieli, 2015-03-27 14:35:45]

The CQ bit was checked by xunjieli@chromium.org

[xunjieli, 2015-03-27 14:35:46]

The patchset sent to the CQ was uploaded after l-g-t-m from
pauljensen@chromium.org
Link to the patchset: https://codereview.chromium.org/996163005/#ps20001 (title:
"Added DCHECKs")

[commit-bot: I haz the power, 2015-03-27 14:35:58]

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

[commit-bot: I haz the power, 2015-03-27 14:58:23]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-27 14:59:12]

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