Persist EffectiveConnectionType in UKM on navigation start.

Persist EffectiveConnectionType in UKM on navigation start.

This change emits a UKM PageLoad metric for the EffectiveConnectionType
observed at navigation start.

This change also corrects an issue where we logged some information in
OnStart. When OnStart is invoked, we don't yet know if we're observing
a page load, so we move all logging to later in the flow, when we know
we're observing a page load.

BUG=700537
NOTRY=true
NOPRESUBMIT=true
TBR=tbansal

Review-Url: https://codereview.chromium.org/2740403002
Cr-Commit-Position: refs/heads/master@{#456219}
(cherry picked from commit 1401b43ed20688607c38be6e012f9310badf3f2d)

Review-Url: https://codereview.chromium.org/2749553002
Cr-Commit-Position: refs/branch-heads/3029@{#150}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed: https://chromium.googlesource.com/chromium/src/+/829838488f6690b59944b5f83d427f012b316cc9

[Bryan McQuade, 2017-03-13 15:33:28]

Description was changed from

==========
Persist EffectiveConnectionType in UKM on navigation start.

This change emits a UKM PageLoad metric for the EffectiveConnectionType
observed at navigation start.

This change also corrects an issue where we logged some information in
OnStart. When OnStart is invoked, we don't yet know if we're observing
a page load, so we move all logging to later in the flow, when we know
we're observing a page load.

BUG=700537

Review-Url: https://codereview.chromium.org/2740403002
Cr-Commit-Position: refs/heads/master@{#456219}
(cherry picked from commit 1401b43ed20688607c38be6e012f9310badf3f2d)
==========

to

==========
Persist EffectiveConnectionType in UKM on navigation start.

This change emits a UKM PageLoad metric for the EffectiveConnectionType
observed at navigation start.

This change also corrects an issue where we logged some information in
OnStart. When OnStart is invoked, we don't yet know if we're observing
a page load, so we move all logging to later in the flow, when we know
we're observing a page load.

BUG=700537
NOTRY=true
NOPRESUBMIT=true
TBR=tbansal

Review-Url: https://codereview.chromium.org/2740403002
Cr-Commit-Position: refs/heads/master@{#456219}
(cherry picked from commit 1401b43ed20688607c38be6e012f9310badf3f2d)
==========

[Bryan McQuade, 2017-03-13 15:34:37]

The CQ bit was checked by bmcquade@chromium.org

[commit-bot: I haz the power, 2017-03-13 15:34:54]

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-03-13 15:38:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489419277585740, "parent_rev":
"a8cb019452819dba009c8807cd729c33337c6b92", "commit_rev":
"829838488f6690b59944b5f83d427f012b316cc9"}

[commit-bot: I haz the power, 2017-03-13 15:39:07]

Description was changed from

==========
Persist EffectiveConnectionType in UKM on navigation start.

This change emits a UKM PageLoad metric for the EffectiveConnectionType
observed at navigation start.

This change also corrects an issue where we logged some information in
OnStart. When OnStart is invoked, we don't yet know if we're observing
a page load, so we move all logging to later in the flow, when we know
we're observing a page load.

BUG=700537
NOTRY=true
NOPRESUBMIT=true
TBR=tbansal

Review-Url: https://codereview.chromium.org/2740403002
Cr-Commit-Position: refs/heads/master@{#456219}
(cherry picked from commit 1401b43ed20688607c38be6e012f9310badf3f2d)
==========

to

==========
Persist EffectiveConnectionType in UKM on navigation start.

This change emits a UKM PageLoad metric for the EffectiveConnectionType
observed at navigation start.

This change also corrects an issue where we logged some information in
OnStart. When OnStart is invoked, we don't yet know if we're observing
a page load, so we move all logging to later in the flow, when we know
we're observing a page load.

BUG=700537
NOTRY=true
NOPRESUBMIT=true
TBR=tbansal

Review-Url: https://codereview.chromium.org/2740403002
Cr-Commit-Position: refs/heads/master@{#456219}
(cherry picked from commit 1401b43ed20688607c38be6e012f9310badf3f2d)

Review-Url: https://codereview.chromium.org/2749553002
Cr-Commit-Position: refs/branch-heads/3029@{#150}
Cr-Branched-From:
939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed:
https://chromium.googlesource.com/chromium/src/+/829838488f6690b59944b5f83d42...
==========

[commit-bot: I haz the power, 2017-03-13 15:39:09]

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