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

Unified Diff: third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.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/html/parser/HTMLConstructionSite.cpp
diff --git a/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp b/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
index 613f0e44e4c7a4a2301441c93df06e21cdf7efa7..1954a69616c8fd9ad9aa50e3e8cea660e2ac310b 100644
--- a/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
+++ b/third_party/WebKit/Source/core/html/parser/HTMLConstructionSite.cpp
@@ -456,109 +456,119 @@ void HTMLConstructionSite::setCompatibilityModeFromDoctype(
// Check for Quirks Mode.
if (name != "html" ||
publicId.startsWith("+//Silmaril//dtd html Pro v0r11 19970101//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith(
"-//AdvaSoft Ltd//DTD HTML 3.0 asWedit + extensions//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//AS//DTD HTML 3.0 asWedit + extensions//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//IETF//DTD HTML 2.0 Level 1//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//IETF//DTD HTML 2.0 Level 2//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//IETF//DTD HTML 2.0 Strict Level 1//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//IETF//DTD HTML 2.0 Strict Level 2//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//IETF//DTD HTML 2.0 Strict//",
- TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML 2.0//", TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML 2.1E//", TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML 3.0//", TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML 2.0//",
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML 2.1E//",
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML 3.0//",
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//IETF//DTD HTML 3.2 Final//",
- TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML 3.2//", TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML 3//", TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML Level 0//", TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML Level 1//", TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML Level 2//", TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML Level 3//", TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML 3.2//",
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML 3//", TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML Level 0//",
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML Level 1//",
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML Level 2//",
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML Level 3//",
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//IETF//DTD HTML Strict Level 0//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//IETF//DTD HTML Strict Level 1//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//IETF//DTD HTML Strict Level 2//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//IETF//DTD HTML Strict Level 3//",
- TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML Strict//", TextCaseInsensitive) ||
- publicId.startsWith("-//IETF//DTD HTML//", TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML Strict//",
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//IETF//DTD HTML//", TextCaseASCIIInsensitive) ||
publicId.startsWith("-//Metrius//DTD Metrius Presentational//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith(
"-//Microsoft//DTD Internet Explorer 2.0 HTML Strict//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//Microsoft//DTD Internet Explorer 2.0 HTML//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//Microsoft//DTD Internet Explorer 2.0 Tables//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith(
"-//Microsoft//DTD Internet Explorer 3.0 HTML Strict//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//Microsoft//DTD Internet Explorer 3.0 HTML//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//Microsoft//DTD Internet Explorer 3.0 Tables//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//Netscape Comm. Corp.//DTD HTML//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//Netscape Comm. Corp.//DTD Strict HTML//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//O'Reilly and Associates//DTD HTML 2.0//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//O'Reilly and Associates//DTD HTML Extended 1.0//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith(
"-//O'Reilly and Associates//DTD HTML Extended Relaxed 1.0//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//SoftQuad Software//DTD HoTMetaL PRO "
"6.0::19990601::extensions to HTML 4.0//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//SoftQuad//DTD HoTMetaL PRO "
"4.0::19971010::extensions to HTML 4.0//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//Spyglass//DTD HTML 2.0 Extended//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//SQ//DTD HTML 2.0 HoTMetaL + extensions//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//Sun Microsystems Corp.//DTD HotJava HTML//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith(
"-//Sun Microsystems Corp.//DTD HotJava Strict HTML//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//W3C//DTD HTML 3 1995-03-24//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//W3C//DTD HTML 3.2 Draft//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//W3C//DTD HTML 3.2 Final//",
- TextCaseInsensitive) ||
- publicId.startsWith("-//W3C//DTD HTML 3.2//", TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//W3C//DTD HTML 3.2//", TextCaseASCIIInsensitive) ||
publicId.startsWith("-//W3C//DTD HTML 3.2S Draft//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//W3C//DTD HTML 4.0 Frameset//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//W3C//DTD HTML 4.0 Transitional//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//W3C//DTD HTML Experimental 19960712//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//W3C//DTD HTML Experimental 970421//",
- TextCaseInsensitive) ||
- publicId.startsWith("-//W3C//DTD W3 HTML//", TextCaseInsensitive) ||
- publicId.startsWith("-//W3O//DTD W3 HTML 3.0//", TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//W3C//DTD W3 HTML//", TextCaseASCIIInsensitive) ||
+ publicId.startsWith("-//W3O//DTD W3 HTML 3.0//",
+ TextCaseASCIIInsensitive) ||
equalIgnoringCase(publicId, "-//W3O//DTD W3 HTML Strict 3.0//EN//") ||
publicId.startsWith("-//WebTechs//DTD Mozilla HTML 2.0//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//WebTechs//DTD Mozilla HTML//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
equalIgnoringCase(publicId, "-/W3C/DTD HTML 4.0 Transitional/EN") ||
equalIgnoringCase(publicId, "HTML") ||
equalIgnoringCase(
@@ -566,25 +576,25 @@ void HTMLConstructionSite::setCompatibilityModeFromDoctype(
"http://www.ibm.com/data/dtd/v11/ibmxhtml1-transitional.dtd") ||
(systemId.isEmpty() &&
publicId.startsWith("-//W3C//DTD HTML 4.01 Frameset//",
- TextCaseInsensitive)) ||
+ TextCaseASCIIInsensitive)) ||
(systemId.isEmpty() &&
publicId.startsWith("-//W3C//DTD HTML 4.01 Transitional//",
- TextCaseInsensitive))) {
+ TextCaseASCIIInsensitive))) {
setCompatibilityMode(Document::QuirksMode);
return;
}
// Check for Limited Quirks Mode.
if (publicId.startsWith("-//W3C//DTD XHTML 1.0 Frameset//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
publicId.startsWith("-//W3C//DTD XHTML 1.0 Transitional//",
- TextCaseInsensitive) ||
+ TextCaseASCIIInsensitive) ||
(!systemId.isEmpty() &&
publicId.startsWith("-//W3C//DTD HTML 4.01 Frameset//",
- TextCaseInsensitive)) ||
+ TextCaseASCIIInsensitive)) ||
(!systemId.isEmpty() &&
publicId.startsWith("-//W3C//DTD HTML 4.01 Transitional//",
- TextCaseInsensitive))) {
+ TextCaseASCIIInsensitive))) {
setCompatibilityMode(Document::LimitedQuirksMode);
return;
}

Powered by Google App Engine
This is Rietveld 408576698