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
Review-Url: https://codereview.chromium.org/2740403002
Cr-Commit-Position: refs/heads/master@{#456219}
Committed: https://chromium.googlesource.com/chromium/src/+/1401b43ed20688607c38be6e012f9310badf3f2d

[Bryan McQuade, 2017-03-10 22:05:18]

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

[commit-bot: I haz the power, 2017-03-10 22:05:46]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Bryan McQuade, 2017-03-10 22:20:12]

Description was changed from

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

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

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=
==========

[Bryan McQuade, 2017-03-10 22:20:45]

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

[commit-bot: I haz the power, 2017-03-10 22:20:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Bryan McQuade, 2017-03-10 22:21:57]

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=
==========

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
==========

[Bryan McQuade, 2017-03-10 22:28:37]

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

[commit-bot: I haz the power, 2017-03-10 22:28:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Bryan McQuade, 2017-03-10 22:32:58]

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

[Bryan McQuade, 2017-03-10 22:33:15]

tbansal, PTAL for use of UINQEService, thanks!

[tbansal1, 2017-03-10 22:47:50]

lgtm. Thanks for doing this.

[commit-bot: I haz the power, 2017-03-10 23:16:18]

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

[commit-bot: I haz the power, 2017-03-10 23:16:19]

Dry run: This issue passed the CQ dry run.

[Bryan McQuade, 2017-03-10 23:49:00]

The CQ bit was checked by bmcquade@chromium.org

[commit-bot: I haz the power, 2017-03-10 23:49:25]

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-10 23:54:22]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1489189740887200,
"parent_rev": "20e72f4beb2a8a35466094dfded73f954266ee5d", "commit_rev":
"1401b43ed20688607c38be6e012f9310badf3f2d"}

[commit-bot: I haz the power, 2017-03-10 23:55:04]

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
==========

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

Review-Url: https://codereview.chromium.org/2740403002
Cr-Commit-Position: refs/heads/master@{#456219}
Committed:
https://chromium.googlesource.com/chromium/src/+/1401b43ed20688607c38be6e012f...
==========

[commit-bot: I haz the power, 2017-03-10 23:55:05]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/1401b43ed20688607c38be6e012f...