New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 687398 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 504237



Sign in to add a comment

Report NT2 instance as early as requested.

Project Member Reported by sunjian@chromium.org, Feb 1 2017

Issue description

Report NT2 instance as early as requested.

Currently NT2 instance is not available until loadEventEnd happens. However,
this new feature contradicts with the way current clients use NT1 which is available
as soon as it gets requested. In order to align with NT1's usage behavior, we
need to also make NT2 available as soon as clients request for it.

Background: https://docs.google.com/a/chromium.org/document/d/1QX2zVcYpuFTsq0MEZH9J8ZUT4CCBezXKE--eX2ctOhw/edit?usp=sharing

 
Description: Show this description
Description: Show this description
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/36d0286404018e441692f66d1289f33ff33a05d1

commit 36d0286404018e441692f66d1289f33ff33a05d1
Author: sunjian <sunjian@chromium.org>
Date: Fri Mar 17 20:11:49 2017

Report nav timing 2 instance as soon as it's requested.

Currently NT2 instance is not available until loadEventEnd happens. However,
this new feature contradicts with the way current clients use NT1 which is available
as soon as it gets requested. In order to align with NT1's usage behavior, we
need to also make NT2 available as soon as clients request for it.

Doc: https://docs.google.com/a/chromium.org/document/d/1QX2zVcYpuFTsq0MEZH9J8ZUT4CCBezXKE--eX2ctOhw/edit?usp=sharing

BUG= 687398 

Review-Url: https://codereview.chromium.org/2647643004
Cr-Commit-Position: refs/heads/master@{#457871}

[add] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_instance_accessible_from_the_start.html
[delete] https://crrev.com/f00e8b48a63cdac2e5d21fe63581bb488c3a355f/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_open_blank_page.html
[delete] https://crrev.com/f00e8b48a63cdac2e5d21fe63581bb488c3a355f/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_open_data_uri.html
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/frame/LocalDOMWindow.cpp
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/loader/DocumentLoader.cpp
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/timing/Performance.cpp
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/timing/Performance.h
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/timing/PerformanceBase.cpp
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/timing/PerformanceBase.h
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/timing/PerformanceBaseTest.cpp
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/timing/PerformanceEntry.h
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/timing/PerformanceNavigationTiming.cpp
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/timing/PerformanceNavigationTiming.h
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/timing/PerformanceResourceTiming.cpp
[modify] https://crrev.com/36d0286404018e441692f66d1289f33ff33a05d1/third_party/WebKit/Source/core/timing/PerformanceResourceTiming.h

Status: Fixed (was: Assigned)

Sign in to add a comment