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

Unified Diff: third_party/WebKit/Source/core/fetch/CrossOriginAccessControlTest.cpp

Issue 2642043004: Fix a bug in origin header generation for CORS preflight in extensions (Closed)
Patch Set: Created 3 years, 11 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/fetch/CrossOriginAccessControlTest.cpp
diff --git a/third_party/WebKit/Source/core/fetch/CrossOriginAccessControlTest.cpp b/third_party/WebKit/Source/core/fetch/CrossOriginAccessControlTest.cpp
index bac83cc9b260776cf9fd420731fb1c153771bd63..17061c9f6b73c84b7ec8c12b585d738da26b4bd4 100644
--- a/third_party/WebKit/Source/core/fetch/CrossOriginAccessControlTest.cpp
+++ b/third_party/WebKit/Source/core/fetch/CrossOriginAccessControlTest.cpp
@@ -14,16 +14,7 @@ namespace blink {
namespace {
-class CreateAccessControlPreflightRequestTest : public ::testing::Test {
- protected:
- virtual void SetUp() {
- m_securityOrigin = SecurityOrigin::createFromString("http://example.com");
- }
-
- RefPtr<SecurityOrigin> m_securityOrigin;
-};
-
-TEST_F(CreateAccessControlPreflightRequestTest, LexicographicalOrder) {
+TEST(CreateAccessControlPreflightRequestTest, LexicographicalOrder) {
ResourceRequest request;
request.addHTTPHeaderField("Orange", "Orange");
request.addHTTPHeaderField("Apple", "Red");
@@ -31,55 +22,49 @@ TEST_F(CreateAccessControlPreflightRequestTest, LexicographicalOrder) {
request.addHTTPHeaderField("Content-Type", "application/octet-stream");
request.addHTTPHeaderField("Strawberry", "Red");
- ResourceRequest preflight =
- createAccessControlPreflightRequest(request, m_securityOrigin.get());
+ ResourceRequest preflight = createAccessControlPreflightRequest(request);
EXPECT_EQ("apple, content-type, kiwifruit, orange, strawberry",
preflight.httpHeaderField("Access-Control-Request-Headers"));
}
-TEST_F(CreateAccessControlPreflightRequestTest, ExcludeSimpleHeaders) {
+TEST(CreateAccessControlPreflightRequestTest, ExcludeSimpleHeaders) {
ResourceRequest request;
request.addHTTPHeaderField("Accept", "everything");
request.addHTTPHeaderField("Accept-Language", "everything");
request.addHTTPHeaderField("Content-Language", "everything");
request.addHTTPHeaderField("Save-Data", "on");
- ResourceRequest preflight =
- createAccessControlPreflightRequest(request, m_securityOrigin.get());
+ ResourceRequest preflight = createAccessControlPreflightRequest(request);
EXPECT_EQ("", preflight.httpHeaderField("Access-Control-Request-Headers"));
}
-TEST_F(CreateAccessControlPreflightRequestTest,
- ExcludeSimpleContentTypeHeader) {
+TEST(CreateAccessControlPreflightRequestTest, ExcludeSimpleContentTypeHeader) {
ResourceRequest request;
request.addHTTPHeaderField("Content-Type", "text/plain");
- ResourceRequest preflight =
- createAccessControlPreflightRequest(request, m_securityOrigin.get());
+ ResourceRequest preflight = createAccessControlPreflightRequest(request);
EXPECT_EQ("", preflight.httpHeaderField("Access-Control-Request-Headers"));
}
-TEST_F(CreateAccessControlPreflightRequestTest, IncludeNonSimpleHeader) {
+TEST(CreateAccessControlPreflightRequestTest, IncludeNonSimpleHeader) {
ResourceRequest request;
request.addHTTPHeaderField("X-Custom-Header", "foobar");
- ResourceRequest preflight =
- createAccessControlPreflightRequest(request, m_securityOrigin.get());
+ ResourceRequest preflight = createAccessControlPreflightRequest(request);
EXPECT_EQ("x-custom-header",
preflight.httpHeaderField("Access-Control-Request-Headers"));
}
-TEST_F(CreateAccessControlPreflightRequestTest,
- IncludeNonSimpleContentTypeHeader) {
+TEST(CreateAccessControlPreflightRequestTest,
+ IncludeNonSimpleContentTypeHeader) {
ResourceRequest request;
request.addHTTPHeaderField("Content-Type", "application/octet-stream");
- ResourceRequest preflight =
- createAccessControlPreflightRequest(request, m_securityOrigin.get());
+ ResourceRequest preflight = createAccessControlPreflightRequest(request);
EXPECT_EQ("content-type",
preflight.httpHeaderField("Access-Control-Request-Headers"));

Powered by Google App Engine
This is Rietveld 408576698