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

Unified Diff: Source/modules/fetch/RequestTest.cpp

Issue 1143083002: Implement request's redirect mode and RequestRedirect for Fetch (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 7 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: Source/modules/fetch/RequestTest.cpp
diff --git a/Source/modules/fetch/RequestTest.cpp b/Source/modules/fetch/RequestTest.cpp
index 9927007afe894ed7d683a08aee05ab0a1e33551d..2cc574390b2123de265bc80bbdbfc407ed97fcd9 100644
--- a/Source/modules/fetch/RequestTest.cpp
+++ b/Source/modules/fetch/RequestTest.cpp
@@ -69,10 +69,12 @@ TEST_F(ServiceWorkerRequestTest, FromAndToWebRequest)
const String referrer = "http://www.referrer.com/";
const WebReferrerPolicy referrerPolicy = WebReferrerPolicyAlways;
const WebURLRequest::RequestContext context = WebURLRequest::RequestContextAudio;
+ const WebURLRequest::FetchRedirectMode redirect = WebURLRequest::FetchRedirectModeFollow;
webRequest.setURL(url);
webRequest.setMethod(method);
webRequest.setRequestContext(context);
+ webRequest.setRedirectMode(redirect);
for (int i = 0; headers[i].key; ++i)
webRequest.setHeader(WebString::fromUTF8(headers[i].key), WebString::fromUTF8(headers[i].value));
webRequest.setReferrer(referrer, referrerPolicy);
@@ -82,6 +84,7 @@ TEST_F(ServiceWorkerRequestTest, FromAndToWebRequest)
EXPECT_EQ(url, request->url());
EXPECT_EQ(method, request->method());
EXPECT_EQ("audio", request->context());
+ EXPECT_EQ("follow", request->redirect());
EXPECT_EQ(referrer, request->referrer());
Headers* requestHeaders = request->headers();
@@ -101,6 +104,7 @@ TEST_F(ServiceWorkerRequestTest, FromAndToWebRequest)
EXPECT_EQ(url, KURL(secondWebRequest.url()));
EXPECT_EQ(method, String(secondWebRequest.method()));
EXPECT_EQ(context, secondWebRequest.requestContext());
+ EXPECT_EQ(redirect, secondWebRequest.redirectMode());
EXPECT_EQ(referrer, KURL(secondWebRequest.referrerUrl()));
EXPECT_EQ(referrerPolicy, secondWebRequest.referrerPolicy());
EXPECT_EQ(webRequest.headers(), secondWebRequest.headers());

Powered by Google App Engine
This is Rietveld 408576698