NetInfo: Return HTTP layer metrics instead of transport layer

NetInfo: Return HTTP layer metrics instead of transport layer

BUG=733881
Review-Url: https://codereview.chromium.org/2941713002
Cr-Commit-Position: refs/heads/master@{#480319}
Committed: https://chromium.googlesource.com/chromium/src/+/75451111c9feeebc1a8b4d7a9a407013ca981fb5

[tbansal1, 2017-06-14 05:43:55]

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

[tbansal1, 2017-06-14 05:44:13]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2017-06-14 05:44:18]

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-06-14 07:30:29]

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

[commit-bot: I haz the power, 2017-06-14 07:30:30]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[tbansal1, 2017-06-15 19:42:01]

Description was changed from

==========
Use HTTP RTT instead of transport RTT

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

to

==========
NetInfo: Return HTTP RTT instead of transport RTT

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

[tbansal1, 2017-06-15 19:44:08]

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

[commit-bot: I haz the power, 2017-06-15 19:44:34]

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-06-15 21:03:27]

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

[commit-bot: I haz the power, 2017-06-15 21:03:28]

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

[tbansal1, 2017-06-15 23:59:11]

Patchset #1 (id:20001) has been deleted

[tbansal1, 2017-06-15 23:59:30]

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

[commit-bot: I haz the power, 2017-06-15 23:59:51]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tbansal1, 2017-06-16 00:00:51]

Patchset #1 (id:40001) has been deleted

[commit-bot: I haz the power, 2017-06-16 00:06:18]

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

[commit-bot: I haz the power, 2017-06-16 00:06:19]

Dry run: Try jobs failed on following builders:
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[tbansal1, 2017-06-16 00:16:41]

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

[commit-bot: I haz the power, 2017-06-16 00:16:57]

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-06-16 00:24:55]

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

[commit-bot: I haz the power, 2017-06-16 00:24:59]

Dry run: Try jobs failed on following builders:
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[tbansal1, 2017-06-16 00:50:27]

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

[commit-bot: I haz the power, 2017-06-16 00:50:46]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tbansal1, 2017-06-16 01:15:36]

Description was changed from

==========
NetInfo: Return HTTP RTT instead of transport RTT

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

to

==========
NetInfo: Return HTTP layer metrics instead of transport layer

BUG=733881
==========

[tbansal1, 2017-06-16 01:15:52]

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

[tbansal1, 2017-06-16 01:16:17]

Patchset #1 (id:60001) has been deleted

[commit-bot: I haz the power, 2017-06-16 01:16:21]

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-06-16 03:08:46]

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

[commit-bot: I haz the power, 2017-06-16 03:08:48]

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

[tbansal1, 2017-06-16 13:28:01]

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

[commit-bot: I haz the power, 2017-06-16 13:28: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-06-16 14:13:05]

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

[commit-bot: I haz the power, 2017-06-16 14:13:05]

Dry run: This issue passed the CQ dry run.

[tbansal1, 2017-06-16 15:33:06]

Patchset #1 (id:80001) has been deleted

[tbansal1, 2017-06-16 15:33:34]

jkarlin: ptal at *. Thanks

[tbansal1, 2017-06-16 15:33:57]

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

[tbansal1, 2017-06-16 15:33:58]

jkarlin: ptal at *. Thanks.

[tbansal1, 2017-06-16 15:34:36]

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

[commit-bot: I haz the power, 2017-06-16 15:34:56]

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-06-16 16:44:39]

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

[commit-bot: I haz the power, 2017-06-16 16:44:40]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[tbansal1, 2017-06-16 16:45:12]

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

[commit-bot: I haz the power, 2017-06-16 16:45:31]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jkarlin, 2017-06-16 16:48:25]

lgtm!

[tbansal1, 2017-06-16 16:49:37]

tbansal@chromium.org changed reviewers:
+ creis@chromium.org, haraken@chromium.org

[tbansal1, 2017-06-16 16:49:38]

ptal:
haraken: third_party/WebKit/Source/platform/network/*
creis:  content/browser/net_info_browsertest.cc

Thanks.

[commit-bot: I haz the power, 2017-06-16 17:48:51]

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

[commit-bot: I haz the power, 2017-06-16 17:48:52]

Dry run: This issue passed the CQ dry run.

[haraken, 2017-06-17 01:37:56]

WebKit LGTM

[Charlie Reis, 2017-06-17 20:11:31]

content/ LGTM

[tbansal1, 2017-06-18 15:16:47]

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

[commit-bot: I haz the power, 2017-06-18 15:16:52]

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-06-18 17:35:55]

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

[commit-bot: I haz the power, 2017-06-18 17:35:57]

Dry run: This issue passed the CQ dry run.

[tbansal1, 2017-06-18 18:22:01]

The CQ bit was checked by tbansal@chromium.org

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

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-06-18 23:27:43]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1497810121814160,
"parent_rev": "a2af2b23b832546af96b349a65dffe3a330f9a93", "commit_rev":
"75451111c9feeebc1a8b4d7a9a407013ca981fb5"}

[commit-bot: I haz the power, 2017-06-18 23:27:54]

Description was changed from

==========
NetInfo: Return HTTP layer metrics instead of transport layer

BUG=733881
==========

to

==========
NetInfo: Return HTTP layer metrics instead of transport layer

BUG=733881

Review-Url: https://codereview.chromium.org/2941713002
Cr-Commit-Position: refs/heads/master@{#480319}
Committed:
https://chromium.googlesource.com/chromium/src/+/75451111c9feeebc1a8b4d7a9a40...
==========

[commit-bot: I haz the power, 2017-06-18 23:27:56]

Committed patchset #1 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/75451111c9feeebc1a8b4d7a9a40...