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

Unified Diff: third_party/WebKit/Source/web/tests/WebFrameTest.cpp

Issue 2734633002: PlzNavigate: Fix the http/tests/loading/307-after-303-after-post.html and the http/tests/loading/re… (Closed)
Patch Set: Fix comment 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/web/tests/WebFrameTest.cpp
diff --git a/third_party/WebKit/Source/web/tests/WebFrameTest.cpp b/third_party/WebKit/Source/web/tests/WebFrameTest.cpp
index d276af66905c075db75afc84d80e65087912c83b..fc14b67ddd26937fd95b1d721dce0e154f8682c0 100644
--- a/third_party/WebKit/Source/web/tests/WebFrameTest.cpp
+++ b/third_party/WebKit/Source/web/tests/WebFrameTest.cpp
@@ -7325,7 +7325,8 @@ class TestHistoryWebFrameClient : public FrameTestHelpers::TestWebFrameClient {
m_replacesCurrentHistoryItem = false;
}
- void didStartProvisionalLoad(WebDataSource* dataSource) {
+ void didStartProvisionalLoad(WebDataSource* dataSource,
+ WebURLRequest& request) {
m_replacesCurrentHistoryItem = dataSource->replacesCurrentHistoryItem();
}
@@ -10089,7 +10090,7 @@ class CallbackOrderingWebFrameClient
EXPECT_EQ(0, m_callbackCount++);
FrameTestHelpers::TestWebFrameClient::didStartLoading(toDifferentDocument);
}
- void didStartProvisionalLoad(WebDataSource*) override {
+ void didStartProvisionalLoad(WebDataSource*, WebURLRequest&) override {
EXPECT_EQ(1, m_callbackCount++);
}
void didCommitProvisionalLoad(WebLocalFrame*,

Powered by Google App Engine
This is Rietveld 408576698