NQE: Move ECT computation to NotifyHeadersReceived

NQE: Move ECT computation to NotifyHeadersReceived

Currently, NQE (Network Quality Estimator) computes the effective connection
type when the transaction starts. This CL moves the computation of effective
connection type 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/2703103002
Cr-Commit-Position: refs/heads/master@{#451535}
Committed: https://chromium.googlesource.com/chromium/src/+/747aab0de56d72b41287c65befbee89dfa433739

[tbansal1, 2017-02-19 22:10:21]

Description was changed from

==========
NQE: Move ECT computation to notify headers

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

to

==========
NQE: Move ECT computation to NotifyHeadersReceived

Currently, NQE (Network Quality Estimator) computes the effective connection
type when the transaction starts. This CL moves the computation of effective
connection type 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-19 22:11:28]

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

[commit-bot: I haz the power, 2017-02-19 22:11:41]

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-19 23:16:16]

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

[commit-bot: I haz the power, 2017-02-19 23:16:17]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tbansal1, 2017-02-20 00:45:44]

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

[tbansal1, 2017-02-20 00:45:45]

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-20 00:45:49]

The CQ bit was checked by tbansal@chromium.org

[commit-bot: I haz the power, 2017-02-20 00:46:05]

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-20 00:49:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487551549540710, "parent_rev":
"e3c137153d7e9995c53612e071d3afe094de39e2", "commit_rev":
"747aab0de56d72b41287c65befbee89dfa433739"}

[commit-bot: I haz the power, 2017-02-20 00:50:06]

Description was changed from

==========
NQE: Move ECT computation to NotifyHeadersReceived

Currently, NQE (Network Quality Estimator) computes the effective connection
type when the transaction starts. This CL moves the computation of effective
connection type 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 ECT computation to NotifyHeadersReceived

Currently, NQE (Network Quality Estimator) computes the effective connection
type when the transaction starts. This CL moves the computation of effective
connection type 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/2703103002
Cr-Commit-Position: refs/heads/master@{#451535}
Committed:
https://chromium.googlesource.com/chromium/src/+/747aab0de56d72b41287c65befbe...
==========

[commit-bot: I haz the power, 2017-02-20 00:50:07]

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