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

Unified Diff: third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp

Issue 2508033004: Reduce unnecessary usage of TextCaseSensitivity::TextCaseInsensitive. (Closed)
Patch Set: 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: third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp
diff --git a/third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp b/third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp
index b24a79d5e87aa69051e59fa16543b3cf678fb9e1..a5e0f7125e574477ff0e9a7ae99f74b87aa91144 100644
--- a/third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp
+++ b/third_party/WebKit/Source/core/xml/parser/XMLDocumentParser.cpp
@@ -578,8 +578,8 @@ static bool isLibxmlDefaultCatalogFile(const String& urlString) {
return true;
// On Windows, libxml computes a URL relative to where its DLL resides.
- if (urlString.startsWith("file:///", TextCaseInsensitive) &&
- urlString.endsWith("/etc/catalog", TextCaseInsensitive))
+ if (urlString.startsWith("file:///", TextCaseASCIIInsensitive) &&
+ urlString.endsWith("/etc/catalog", TextCaseASCIIInsensitive))
return true;
return false;
}
@@ -594,12 +594,13 @@ static bool shouldAllowExternalLoad(const KURL& url) {
// The most common DTD. There isn't much point in hammering www.w3c.org by
// requesting this URL for every XHTML document.
- if (urlString.startsWith("http://www.w3.org/TR/xhtml", TextCaseInsensitive))
+ if (urlString.startsWith("http://www.w3.org/TR/xhtml",
+ TextCaseASCIIInsensitive))
return false;
// Similarly, there isn't much point in requesting the SVG DTD.
if (urlString.startsWith("http://www.w3.org/Graphics/SVG",
- TextCaseInsensitive))
+ TextCaseASCIIInsensitive))
return false;
// The libxml doesn't give us a lot of context for deciding whether to allow

Powered by Google App Engine
This is Rietveld 408576698