Record NQE ECT at main frame

Record NQE ECT at main frame

This will help in establishing the correlation between page-level metrics (such as PLT), and ECT.

BUG=614572
Committed: https://crrev.com/2052c61710876afbc63a0ff159509f91a152812b
Cr-Commit-Position: refs/heads/master@{#401383}

[tbansal1, 2016-06-18 00:36:50]

Patchset #1 (id:1) has been deleted

[tbansal1, 2016-06-18 00:38:23]

Description was changed from

==========
Record NQE ECT at main frame

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

to

==========
Record NQE ECT at main frame

BUG=614572
==========

[tbansal1, 2016-06-18 00:38:55]

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

[tbansal1, 2016-06-18 00:38:56]

kundaji: ptal. Thanks.

[Not at Google. Contact bengr, 2016-06-20 16:03:42]

lgtm

[tbansal1, 2016-06-20 18:34:58]

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

[tbansal1, 2016-06-20 18:34:58]

bengr: ptal. Thanks.

[bengr, 2016-06-20 20:11:24]

lgtm. Please flesh out the CL description.

https://codereview.chromium.org/2076353002/diff/20001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2076353002/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:34824: +    Effective connection type
estimated by network quality estimator. Suffixed
by -> by the

[tbansal1, 2016-06-20 20:27:30]

Description was changed from

==========
Record NQE ECT at main frame

BUG=614572
==========

to

==========
Record NQE ECT at main frame

This will help in establishing the correlation between page-level metrics (such
as PLT), and ECT.

BUG=614572
==========

[tbansal1, 2016-06-20 20:27:49]

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

[tbansal1, 2016-06-20 20:27:49]

asvitkine: ptal. Thanks.

[tbansal1, 2016-06-20 22:50:42]

tbansal@chromium.org changed reviewers:
+ rkaplow@chromium.org
- asvitkine@chromium.org

[tbansal1, 2016-06-20 22:50:43]

rkaplow: PTAL at histograms.xml. Thanks.

[rkaplow, 2016-06-21 14:39:31]

lgtm

[tbansal1, 2016-06-22 18:12:27]

The CQ bit was checked by tbansal@chromium.org

[tbansal1, 2016-06-22 18:12:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
kundaji@chromium.org, bengr@chromium.org, rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/2076353002/#ps40001
(title: "Rebased")

[commit-bot: I haz the power, 2016-06-22 18:13:36]

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

[commit-bot: I haz the power, 2016-06-22 19:48:46]

Description was changed from

==========
Record NQE ECT at main frame

This will help in establishing the correlation between page-level metrics (such
as PLT), and ECT.

BUG=614572
==========

to

==========
Record NQE ECT at main frame

This will help in establishing the correlation between page-level metrics (such
as PLT), and ECT.

BUG=614572
==========

[commit-bot: I haz the power, 2016-06-22 19:48:47]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-06-22 19:49:59]

Description was changed from

==========
Record NQE ECT at main frame

This will help in establishing the correlation between page-level metrics (such
as PLT), and ECT.

BUG=614572
==========

to

==========
Record NQE ECT at main frame

This will help in establishing the correlation between page-level metrics (such
as PLT), and ECT.

BUG=614572

Committed: https://crrev.com/2052c61710876afbc63a0ff159509f91a152812b
Cr-Commit-Position: refs/heads/master@{#401383}
==========

[commit-bot: I haz the power, 2016-06-22 19:50:00]

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