New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update HTMLFastPathParser::scanTagName() to return an ElementName #12270
Update HTMLFastPathParser::scanTagName() to return an ElementName #12270
Conversation
EWS run on previous version of this PR (hash 7fb9a90) |
Source/WebCore/dom/make_names.pl
Outdated
print F "ElementName findHTMLElementNameFromBuffer(Span<const LChar>);\n"; | ||
print F "ElementName findHTMLElementNameFromBuffer(Span<const UChar>);\n"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can these just be findHTMLElementName
? I donβt think we need βfrom bufferβ in the name.
@@ -845,16 +815,16 @@ class HTMLFastPathParser { | |||
|
|||
template<class... Tags> RefPtr<Element> parseSpecificElements() | |||
{ | |||
CharSpan tagName = scanTagName(); | |||
ElementName tagName = scanTagName(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
auto
7fb9a90
to
0958e72
Compare
EWS run on previous version of this PR (hash 0958e72) |
0958e72
to
1f6983c
Compare
EWS run on current version of this PR (hash 1f6983c) |
https://bugs.webkit.org/show_bug.cgi?id=254857 Reviewed by Darin Adler. Update HTMLFastPathParser::scanTagName() to return an ElementName enum instead of a Span<Character>. This simplifies the logic in the HTML fast parser and is slightly more efficient. * Source/WebCore/dom/make_names.pl: (printElementNameHeaderFile): (printElementNameCppFile): * Source/WebCore/html/parser/HTMLDocumentParserFastPath.cpp: (WebCore::HTMLFastPathParser::scanTagName): (WebCore::HTMLFastPathParser::parseSpecificElements): (WebCore::HTMLFastPathParser::parseElement): (WebCore::onlyContainsLowercaseASCIILetters): Deleted. (WebCore::tagNameHash): Deleted. Canonical link: https://commits.webkit.org/262484@main
1f6983c
to
8e6de78
Compare
Committed 262484@main (8e6de78): https://commits.webkit.org/262484@main Reviewed commits have been landed. Closing PR #12270 and removing active labels. |
8e6de78
1f6983c
π§ͺ gtk-wk2π§ͺ mac-wk2