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

Unified Diff: content/browser/fileapi/obfuscated_file_util_unittest.cc

Issue 2485253002: Remove unnecessary calls to GURL() (Closed)
Patch Set: Assert that StringPiece must always be canonicalized. Fix some constants. Created 4 years, 1 month 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: content/browser/fileapi/obfuscated_file_util_unittest.cc
diff --git a/content/browser/fileapi/obfuscated_file_util_unittest.cc b/content/browser/fileapi/obfuscated_file_util_unittest.cc
index 6afd9a78356f138d36e4c5600ba9d215e984534c..9eb3d5fac3a1a5213b51081543853f61a7ca4548 100644
--- a/content/browser/fileapi/obfuscated_file_util_unittest.cc
+++ b/content/browser/fileapi/obfuscated_file_util_unittest.cc
@@ -106,11 +106,11 @@ struct OriginEnumerationTestRecord {
};
const OriginEnumerationTestRecord kOriginEnumerationTestRecords[] = {
- {"http://example.com", false, true},
- {"http://example1.com", true, false},
- {"https://example1.com", true, true},
- {"file://", false, true},
- {"http://example.com:8000", false, true},
+ {"http://example.com/", false, true},
+ {"http://example1.com/", true, false},
+ {"https://example1.com/", true, true},
+ {"file:///", false, true},
+ {"http://example.com:8000/", false, true},
};
FileSystemURL FileSystemURLAppend(
@@ -1632,7 +1632,7 @@ TEST_F(ObfuscatedFileUtilTest, TestOriginEnumerator) {
++i) {
const OriginEnumerationTestRecord& record =
kOriginEnumerationTestRecords[i];
- if (GURL(record.origin_url) != origin_url)
+ if (origin_url != record.origin_url)
continue;
found = true;
EXPECT_EQ(record.has_temporary,
« no previous file with comments | « content/browser/appcache/appcache_request_handler_unittest.cc ('k') | content/browser/quota/quota_manager_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698