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

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
« no previous file with comments | « Source/modules/fetch/RequestInit.cpp ('k') | Source/platform/exported/WebServiceWorkerRequest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/modules/fetch/RequestTest.cpp
diff --git a/Source/modules/fetch/RequestTest.cpp b/Source/modules/fetch/RequestTest.cpp
index 398ebed6a8425b633a277fa8edc8f22ef2ebce69..c5e421050b11db3a5f8a976f747e8ae2784eac9d 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());
« no previous file with comments | « Source/modules/fetch/RequestInit.cpp ('k') | Source/platform/exported/WebServiceWorkerRequest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698