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

Issue 641912 link

Starred by 1 user
Status: Fixed
Owner:
arthurso...@chromium.org
Closed: Oct 2016
Cc:
clamy@chromium.org
carlosk@chromium.org
█ nasko@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocking:
issue 576261



Sign in to add a comment

PlzNavigate Resource Timing API support

Project Member Reported by arthurso...@chromium.org, Aug 29 2016 
The Resource Timing API should work with PlzNavigate enabled.

Resource Timing API works, except when the resource is the main page or an iframe.

Two things should be done:
1) Use the initial URL for iframe resources instead of the final URL after redirects.
2) Because the navigation happens now outside of the renderer, timing's data must be forwarded. 

Tests that are currently failing:
- http/tests/w3c/webperf/submission/Google/resource-timing/html/test_resource_redirects.html
- http/tests/w3c/webperf/submission/Intel/resource-timing/test_resource_timing_cross_origin_redirect_chain_allow_timing.html
- http/tests/w3c/webperf/submission/Intel/resource-timing/test_resource_timing_cross_origin_redirect_chain.html
- http/tests/w3c/webperf/submission/Intel/resource-timing/test_resource_timing_cross_origin_redirect_with_timing_allow_origin.html
- http/tests/w3c/webperf/submission/Intel/resource-timing/test_resource_timing_cross_origin_redirect.html
- http/tests/w3c/webperf/submission/Intel/resource-timing/test_resource_timing_same_origin_redirect.html
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 4 2016 

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

commit d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d
Author: arthursonzogni <arthursonzogni@chromium.org>
Date: Tue Oct 04 15:10:03 2016

PlzNavigate: Support ResourceTiming API

With browser-side-navigation, there are no trace of previous navigation
responses in the redirect chain. This CL forward these informations to
blink for making the Resource Timing API working.

BUG= 641912 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

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

[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/browser/frame_host/navigation_request.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/browser/loader/navigation_resource_handler.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/child/web_url_loader_impl.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/child/web_url_loader_impl.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/child/web_url_loader_impl_unittest.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/common/frame_messages.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/common/navigation_params.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/renderer/media/android/media_info_loader.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/renderer/media/android/media_info_loader.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/renderer/media/android/media_info_loader_unittest.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/renderer/pepper/pepper_url_loader_host.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/renderer/pepper/pepper_url_loader_host.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/content/renderer/render_frame_impl.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/media/blink/multibuffer_data_source_unittest.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/media/blink/resource_multibuffer_data_provider.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/media/blink/resource_multibuffer_data_provider.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/media/blink/resource_multibuffer_data_provider_unittest.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/LayoutTests/FlagExpectations/enable-browser-side-navigation
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/core/fetch/ResourceFetcher.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/core/fetch/ResourceFetcherTest.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/core/fetch/ResourceLoader.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/core/fetch/ResourceLoader.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/core/loader/PingLoader.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/core/timing/PerformanceResourceTiming.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/platform/exported/WebURLRequest.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/platform/exported/WebURLResponse.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/platform/network/ResourceRequest.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/platform/network/ResourceRequest.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/platform/network/ResourceResponse.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/platform/network/ResourceResponse.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/platform/network/ResourceTimingInfo.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/platform/network/ResourceTimingInfo.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/platform/testing/weburl_loader_mock.cc
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/web/AssociatedURLLoader.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/Source/web/AssociatedURLLoaderTest.cpp
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/public/platform/WebURLLoaderClient.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/public/platform/WebURLRequest.h
[modify] https://crrev.com/d563574ba6b1beec7cf5fcdb994ffb3cc49ec64d/third_party/WebKit/public/platform/WebURLResponse.h

Comment 2 by arthurso...@chromium.org, Oct 5 2016

Status: Fixed (was: Assigned)
►

Sign in to add a comment