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

Unified Diff: third_party/WebKit/Source/core/loader/FrameLoader.cpp

Issue 2734633002: PlzNavigate: Fix the http/tests/loading/307-after-303-after-post.html and the http/tests/loading/re… (Closed)
Patch Set: Pass the WebURLRequest in didStartProvisionalLoad() Created 3 years, 9 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: third_party/WebKit/Source/core/loader/FrameLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/FrameLoader.cpp b/third_party/WebKit/Source/core/loader/FrameLoader.cpp
index f1814ac8b1dfe477984aebf277dc53df2f32b212..0cf629422444dbaa0b9c95c474080abc049bf326 100644
--- a/third_party/WebKit/Source/core/loader/FrameLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/FrameLoader.cpp
@@ -1778,7 +1778,7 @@ void FrameLoader::startLoad(FrameLoadRequest& frameLoadRequest,
// consumers
// of
// the didStartProvisionalLoad() notification.
- client()->dispatchDidStartProvisionalLoad(loader);
+ client()->dispatchDidStartProvisionalLoad(loader, &resourceRequest);
DCHECK(loader);
loader->setSentDidFinishLoad();
loader->detachFromFrame();
@@ -1814,7 +1814,8 @@ void FrameLoader::startLoad(FrameLoadRequest& frameLoadRequest,
// TODO(ananta)
// We should get rid of the dependency on the DocumentLoader in consumers of
// the didStartProvisionalLoad() notification.
- client()->dispatchDidStartProvisionalLoad(m_provisionalDocumentLoader);
+ client()->dispatchDidStartProvisionalLoad(m_provisionalDocumentLoader,
+ &resourceRequest);
DCHECK(m_provisionalDocumentLoader);
m_provisionalDocumentLoader->startLoadingMainResource();

Powered by Google App Engine
This is Rietveld 408576698