Update NT2 name to be the requested URL instead of 'document'

Update NT2 name to be the requested URL instead of 'document'
There's a strong request to merge this in M57, see spec bug: https://github.com/w3c/navigation-timing/issues/59

BUG=675039
Review-Url: https://codereview.chromium.org/2675973004
Cr-Commit-Position: refs/heads/master@{#448517}
Committed: https://chromium.googlesource.com/chromium/src/+/0af62af4bf7a7b092fd5d81acc56e316a01244db

[panicker, 2017-02-04 00:50:03]

panicker@chromium.org changed reviewers:
+ haraken@chromium.org, sunjian@chromium.org, yoav@yoav.ws

[panicker, 2017-02-04 00:50:04]

PTAL

[panicker, 2017-02-04 00:50:55]

Description was changed from

==========
Update NT2 name to be the requested URL instead of 'document'

BUG=675039
==========

to

==========
Update NT2 name to be the requested URL instead of 'document'
There's a strong request to merge this in M57, see spec bug:
https://github.com/w3c/navigation-timing/issues/59

BUG=675039
==========

[haraken, 2017-02-04 09:35:56]

LGTM

Maybe you need to get an approval from an API owner since this is a web-exposed
change.

[sunjian, 2017-02-06 18:56:56]

On 2017/02/04 09:35:56, haraken wrote:
> LGTM
> 
> Maybe you need to get an approval from an API owner since this is a
web-exposed
> change.

LGTM

[panicker, 2017-02-06 18:59:00]

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

[commit-bot: I haz the power, 2017-02-06 18:59:32]

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-06 21:05:22]

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

[commit-bot: I haz the power, 2017-02-06 21:05:23]

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_...)

[panicker, 2017-02-07 00:28:13]

The CQ bit was checked by panicker@chromium.org

[panicker, 2017-02-07 00:28:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, sunjian@chromium.org
Link to the patchset: https://codereview.chromium.org/2675973004/#ps20001
(title: "fix failing wpt tests (imported after creation of this CL), and delete
a redundant, old test")

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

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-07 02:27:42]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486427293539480,
"parent_rev": "7eea8952954c9b0db9ac932efc2389ba793c6e55", "commit_rev":
"0af62af4bf7a7b092fd5d81acc56e316a01244db"}

[commit-bot: I haz the power, 2017-02-07 02:28:43]

Description was changed from

==========
Update NT2 name to be the requested URL instead of 'document'
There's a strong request to merge this in M57, see spec bug:
https://github.com/w3c/navigation-timing/issues/59

BUG=675039
==========

to

==========
Update NT2 name to be the requested URL instead of 'document'
There's a strong request to merge this in M57, see spec bug:
https://github.com/w3c/navigation-timing/issues/59

BUG=675039

Review-Url: https://codereview.chromium.org/2675973004
Cr-Commit-Position: refs/heads/master@{#448517}
Committed:
https://chromium.googlesource.com/chromium/src/+/0af62af4bf7a7b092fd5d81acc56...
==========

[commit-bot: I haz the power, 2017-02-07 02:28:44]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/0af62af4bf7a7b092fd5d81acc56...

[Yoav Weiss, 2017-02-07 08:03:11]

On 2017/02/07 02:28:44, commit-bot: I haz the power wrote:
> Committed patchset #2 (id:20001) as
>
https://chromium.googlesource.com/chromium/src/+/0af62af4bf7a7b092fd5d81acc56...

retroactive LGTM