NQE: Move recording of metrics to when the response headers are received

NQE: Move recording of metrics to when the response headers are received

Currently, NQE (Network Quality Estimator) records the network quality
metrics when the transaction starts. This CL moves the recording of 
the metrics to when the response headers are received.

BUG=691798
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=ryansturm@chromium.org

Review-Url: https://codereview.chromium.org/2702013002
Cr-Commit-Position: refs/heads/master@{#451446}
Committed: https://chromium.googlesource.com/chromium/src/+/69415b51072fb20605fc129f136950f7c2560ceb

[tbansal1, 2017-02-18 07:37:30]

Description was changed from

==========
w


w

BUG=
==========

to

==========
NQE: Move recording of metrics to when the response headers are received

Currently, NQE (Network Quality Estimator) records the network quality
metrics when the transaction starts. This CL moves the recording of the
metrics to when the response headers are received.

BUG=691798
==========

[tbansal1, 2017-02-18 07:37:45]

Description was changed from

==========
NQE: Move recording of metrics to when the response headers are received

Currently, NQE (Network Quality Estimator) records the network quality
metrics when the transaction starts. This CL moves the recording of the
metrics to when the response headers are received.

BUG=691798
==========

to

==========
NQE: Move recording of metrics to when the response headers are received

Currently, NQE (Network Quality Estimator) records the network quality
metrics when the transaction starts. This CL moves the recording of the
metrics to when the response headers are received.

BUG=691798
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[tbansal1, 2017-02-18 07:40:01]

The CQ bit was checked by tbansal@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-18 07:40:13]

Dry run: 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, 2017-02-18 08:36:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-18 08:36:47]

Dry run: This issue passed the CQ dry run.

[tbansal1, 2017-02-18 08:58:34]

Description was changed from

==========
NQE: Move recording of metrics to when the response headers are received

Currently, NQE (Network Quality Estimator) records the network quality
metrics when the transaction starts. This CL moves the recording of the
metrics to when the response headers are received.

BUG=691798
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
NQE: Move recording of metrics to when the response headers are received

Currently, NQE (Network Quality Estimator) records the network quality
metrics when the transaction starts. This CL moves the recording of 
the metrics to when the response headers are received.

BUG=691798
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[tbansal1, 2017-02-18 08:58:56]

Description was changed from

==========
NQE: Move recording of metrics to when the response headers are received

Currently, NQE (Network Quality Estimator) records the network quality
metrics when the transaction starts. This CL moves the recording of 
the metrics to when the response headers are received.

BUG=691798
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
NQE: Move recording of metrics to when the response headers are received

Currently, NQE (Network Quality Estimator) records the network quality
metrics when the transaction starts. This CL moves the recording of 
the metrics to when the response headers are received.

BUG=691798
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=ryansturm@chromium.org
==========

[tbansal1, 2017-02-18 09:00:27]

TBR'ing ryansturm to see if this fixes issues with the perf bot, and also
because this CL is a partial revert of
https://codereview.chromium.org/2698913006/

[tbansal1, 2017-02-18 09:00:40]

tbansal@chromium.org changed reviewers:
+ ryansturm@chromium.org

[tbansal1, 2017-02-18 09:00:46]

The CQ bit was checked by tbansal@chromium.org

[commit-bot: I haz the power, 2017-02-18 09:00:56]

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, 2017-02-18 09:03:59]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487408446709320, "parent_rev":
"dc652be25ec1400b7f120a96398e5a7de89b4cd4", "commit_rev":
"69415b51072fb20605fc129f136950f7c2560ceb"}

[commit-bot: I haz the power, 2017-02-18 09:04:33]

Description was changed from

==========
NQE: Move recording of metrics to when the response headers are received

Currently, NQE (Network Quality Estimator) records the network quality
metrics when the transaction starts. This CL moves the recording of 
the metrics to when the response headers are received.

BUG=691798
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=ryansturm@chromium.org
==========

to

==========
NQE: Move recording of metrics to when the response headers are received

Currently, NQE (Network Quality Estimator) records the network quality
metrics when the transaction starts. This CL moves the recording of 
the metrics to when the response headers are received.

BUG=691798
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
TBR=ryansturm@chromium.org

Review-Url: https://codereview.chromium.org/2702013002
Cr-Commit-Position: refs/heads/master@{#451446}
Committed:
https://chromium.googlesource.com/chromium/src/+/69415b51072fb20605fc129f1369...
==========

[commit-bot: I haz the power, 2017-02-18 09:04:34]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/69415b51072fb20605fc129f1369...