[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


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:


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



String childTextContent(const ContainerNode& root)
    StringBuilder result;
+    for (Text* text = TextNodeTraversal::firstChild(root); text; text = TextNodeTraversal::nextSibling(*text)) {
+        text->atomize();
+    }
    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