Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(627)

Unified Diff: content/renderer/render_frame_impl.cc

Issue 2316573002: PlzNavigate: Support ResourceTiming API (Closed)
Patch Set: Addressed comments (clamy@) Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index fbd8279f491e1b8f8686c548166e01c4fa128fa0..31a5d8acd478a665009ff302c0b96a1a9d44a90e 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -4829,6 +4829,8 @@ void RenderFrameImpl::OnCommitNavigation(
new StreamOverrideParameters());
stream_override->stream_url = stream_url;
stream_override->response = response;
+ stream_override->redirects = request_params.redirects;
+ stream_override->redirect_responses = request_params.redirect_response_infos;
NavigateInternal(common_params, StartNavigationParams(), request_params,
std::move(stream_override));
@@ -5470,11 +5472,15 @@ void RenderFrameImpl::NavigateInternal(
request.setHasUserGesture(start_params.has_user_gesture);
#endif
- // PlzNavigate: Make sure that Blink's loader will not try to use browser side
- // navigation for this request (since it already went to the browser).
- if (browser_side_navigation)
+ if (browser_side_navigation) {
+ // PlzNavigate: Make sure that Blink's loader will not try to use browser
+ // side navigation for this request (since it already went to the browser).
request.setCheckForBrowserSideNavigation(false);
+ request.setNavigationStartTime(
+ ConvertToBlinkTime(common_params.navigation_start));
+ }
+
// If we are reloading, then use the history state of the current frame.
// Otherwise, if we have history state, then we need to navigate to it, which
// corresponds to a back/forward navigation event. Update the parameters

Powered by Google App Engine
This is Rietveld 408576698