Use StringView for equalIgnoringCase.

third_party/WebKit/Source/core/html/parser/HTMLTreeBuilderSimulator.cpp

Index: third_party/WebKit/Source/core/html/parser/HTMLTreeBuilderSimulator.cpp
diff --git a/third_party/WebKit/Source/core/html/parser/HTMLTreeBuilderSimulator.cpp b/third_party/WebKit/Source/core/html/parser/HTMLTreeBuilderSimulator.cpp
index 67e9caff836c76efa3d406a4058f54b028526890..b022516db1fb1729e69f6fdf38c1c2c75aaee167 100644
--- a/third_party/WebKit/Source/core/html/parser/HTMLTreeBuilderSimulator.cpp
+++ b/third_party/WebKit/Source/core/html/parser/HTMLTreeBuilderSimulator.cpp
@@ -90,7 +90,9 @@ static bool tokenExitsForeignContent(const CompactHTMLToken& token)
 static bool tokenExitsSVG(const CompactHTMLToken& token)
 {
     // FIXME: It's very fragile that we special case foreignObject here to be case-insensitive.
-    return equalIgnoringCaseNonNull(token.data().impl(), SVGNames::foreignObjectTag.localName().impl());
+
+    // Save one branch inside StringView by derefing the StringImpl.
+    return equalIgnoringCaseAndNullity(*token.data().impl(), *SVGNames::foreignObjectTag.localName().impl());

[haraken, 2016/08/04 09:05:28]

Honestly speaking, I'm not sure how much it makes sense to have the micro
optimization. I might prefer just removing equalIgnoringCaseAndNullity.

[esprehn, 2016/08/04 18:30:07]

I can remove the optimization here, but we still want the function since it has
different semantics, like:

https://cs.chromium.org/chromium/src/third_party/WebKit/Source/wtf/text/WTFSt...

which is used by various code.

[esprehn, 2016/08/04 21:57:19]

I removed this optimization but left the one in the hash table.

 }
 
 static bool tokenExitsMath(const CompactHTMLToken& token)