Summary: | De-template ContainerNodeAlgorithms | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Antti Koivisto <koivisto> | ||||
Component: | DOM | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | calvaris, commit-queue, eric.carlson, esprehn+autocc, gyuyoung.kim, kangil.han, philipj, rniwa | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | 149400 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Antti Koivisto
2015-09-17 14:18:13 PDT
Created attachment 261425 [details]
patch
Comment on attachment 261425 [details]
patch
Nice. Next make it iterative? :)
Comment on attachment 261425 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=261425&action=review > Source/WebCore/html/track/WebVTTParser.cpp:544 > + auto child = Text::create(document, m_token.characters()); > + m_currentNode->parserAppendChild(WTF::move(child)); If we did it on one line without the local variable then we wouldn’t need to call WTF::move. > Source/WebCore/xml/parser/XMLDocumentParserLibxml2.cpp:1031 > + auto newNode = CDATASection::create(m_currentNode->document(), toString(s, len)); > + m_currentNode->parserAppendChild(WTF::move(newNode)); If we did it on one line without the local variable then we wouldn’t need to call WTF::move. > Source/WebCore/xml/parser/XMLDocumentParserLibxml2.cpp:1047 > + auto newNode = Comment::create(m_currentNode->document(), toString(s)); > + m_currentNode->parserAppendChild(WTF::move(newNode)); If we did it on one line without the local variable then we wouldn’t need to call WTF::move. |