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

Unified Diff: third_party/WebKit/Source/platform/weborigin/SecurityOriginTest.cpp

Issue 2378323003: Add url::Origin::GetURL() to convert Origins to URLs without reparsing (Closed)
Patch Set: propagate file:/// change to weborigin Created 4 years, 2 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/platform/weborigin/SecurityOriginTest.cpp
diff --git a/third_party/WebKit/Source/platform/weborigin/SecurityOriginTest.cpp b/third_party/WebKit/Source/platform/weborigin/SecurityOriginTest.cpp
index 1df9cb0d7a7141d7d1a860ecdf05d58ae139ba0d..cdb24f355512e9593eed81d11cf9b4b491af0cfa 100644
--- a/third_party/WebKit/Source/platform/weborigin/SecurityOriginTest.cpp
+++ b/third_party/WebKit/Source/platform/weborigin/SecurityOriginTest.cpp
@@ -418,8 +418,8 @@ TEST_F(SecurityOriginTest, CreateFromTuple) {
{"http", "example.com", 81, "http://example.com:81"},
{"https", "example.com", 443, "https://example.com"},
{"https", "example.com", 444, "https://example.com:444"},
- {"file", "", 0, "file://"},
- {"file", "example.com", 0, "file://"},
+ {"file", "", 0, "file:///"},
+ {"file", "example.com", 0, "file:///"},
};
for (const auto& test : cases) {
@@ -438,12 +438,12 @@ TEST_F(SecurityOriginTest, UniquenessPropagatesToBlobUrls) {
{"", true, "null"},
{"null", true, "null"},
{"data:text/plain,hello_world", true, "null"},
- {"file:///path", false, "file://"},
+ {"file:///path", false, "file:///"},
{"filesystem:http://host/filesystem-path", false, "http://host"},
- {"filesystem:file:///filesystem-path", false, "file://"},
+ {"filesystem:file:///filesystem-path", false, "file:///"},
{"filesystem:null/filesystem-path", true, "null"},
{"blob:http://host/blob-id", false, "http://host"},
- {"blob:file:///blob-id", false, "file://"},
+ {"blob:file:///blob-id", false, "file:///"},
{"blob:null/blob-id", true, "null"},
};

Powered by Google App Engine
This is Rietveld 408576698