[Webkit-unassigned] [Bug 117488] Consider atomicizing inline scripts before execution

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 6 14:56:37 PDT 2023


https://bugs.webkit.org/show_bug.cgi?id=117488

Ahmad Saleem <ahmad.saleem792 at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ahmad.saleem792 at gmail.com

--- Comment #1 from Ahmad Saleem <ahmad.saleem792 at gmail.com> ---
I tried:

Source/WebCore/dom/CharacterData.h:

void atomize() { m_data = AtomString(m_data); }

__________

Source/WebCore/dom/TextNodeTraversal.cpp:


String childTextContent(const ContainerNode& root)
{
    StringBuilder result;
+    for (Text* text = TextNodeTraversal::firstChild(root); text; text = TextNodeTraversal::nextSibling(*text)) {
+        text->atomize();
        result.append(text->data());
+    }
    return result.toString();
}

__________

But again, it compiles but we might need AB testing to confirm any benefit on memory and no impact on performance from this. Any Apple engineer interested?

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20230706/94c62547/attachment.htm>


More information about the webkit-unassigned mailing list