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

Unified Diff: third_party/WebKit/Source/core/frame/csp/CSPSourceTest.cpp

Issue 2556713002: Embedding-CSP: Ports subsumption (Closed)
Patch Set: Adding a comment Created 4 years 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 | « third_party/WebKit/Source/core/frame/csp/CSPSource.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/frame/csp/CSPSourceTest.cpp
diff --git a/third_party/WebKit/Source/core/frame/csp/CSPSourceTest.cpp b/third_party/WebKit/Source/core/frame/csp/CSPSourceTest.cpp
index e595028901c56cf6adf2fc10023237dec39682c9..39fb4f6075192e30dc235251a1b9dda66756bb43 100644
--- a/third_party/WebKit/Source/core/frame/csp/CSPSourceTest.cpp
+++ b/third_party/WebKit/Source/core/frame/csp/CSPSourceTest.cpp
@@ -388,6 +388,12 @@ TEST_F(CSPSourceTest, IsSimilar) {
{"wss", "example.com", "/", 0},
true}, // use default port
{{"http", "example.com", "/", 80}, {"http", "example.com", "/", 0}, true},
+ {{"http", "example.com", "/", 80},
+ {"https", "example.com", "/", 443},
+ true},
+ {{"http", "example.com", "/", 80},
+ {"https", "example.com", "/", 444},
+ false},
// Paths
{{"http", "example.com", "/", 0},
{"http", "example.com", "/1.html", 0},
@@ -600,6 +606,18 @@ TEST_F(CSPSourceTest, Intersect) {
CSPSource::NoWildcard},
{"http", "example.com", "/", 80, CSPSource::NoWildcard,
CSPSource::NoWildcard}},
+ {{"http", "example.com", "/", 80, CSPSource::NoWildcard,
+ CSPSource::NoWildcard},
+ {"https", "example.com", "/", 443, CSPSource::NoWildcard,
+ CSPSource::NoWildcard},
+ {"https", "example.com", "/", 443, CSPSource::NoWildcard,
+ CSPSource::NoWildcard}},
+ {{"https", "example.com", "/", 443, CSPSource::NoWildcard,
+ CSPSource::NoWildcard},
+ {"http", "example.com", "/", 80, CSPSource::NoWildcard,
+ CSPSource::NoWildcard},
+ {"https", "example.com", "/", 443, CSPSource::NoWildcard,
+ CSPSource::NoWildcard}},
// Paths
{{"http", "example.com", "/", 0, CSPSource::NoWildcard,
CSPSource::NoWildcard},
« no previous file with comments | « third_party/WebKit/Source/core/frame/csp/CSPSource.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698