[webkit-reviews] review granted: [Bug 117202] Avoid multiple copies of inline script & style strings : [Attachment 203874] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Arunprasad Rajkumar
<arurajku at cisco.com>'s request for review:
Bug 117202: Avoid multiple copies of inline script & style strings
https://bugs.webkit.org/show_bug.cgi?id=117202

Attachment 203874: Patch
https://bugs.webkit.org/attachment.cgi?id=203874&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=203874&action=review


> Source/WebCore/html/parser/HTMLConstructionSite.cpp:500
> +    unsigned lengthLimit = shouldUseLengthLimit(task.parent.get()) ?
Text::defaultLengthLimit : std::numeric_limits<unsigned>::max();

Too bad we need to compute this limit ahead of time. Sure would be nice to only
do the work when we actually get close enough to hitting a limit.