Introduce an LRU cache around CSSParser::parseValue to prevent variables from needing to re-parse unnecessarily.
Created attachment 148239 [details] Patch
Comment on attachment 148239 [details] Patch Attachment 148239 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12974598
Comment on attachment 148239 [details] Patch Attachment 148239 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/12979349
Comment on attachment 148239 [details] Patch Attachment 148239 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/12974599
Comment on attachment 148239 [details] Patch Attachment 148239 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12968962
Comment on attachment 148239 [details] Patch This is a lot of new code without any rationale. If this improves performance, please tell what your benchmark is.
As Antti said on bug 85580: "Parsing CSS values on style resolve time is going to be horribly slow. A good approach might be to cache the parsed values to the variable declaration value itself (with id/type key)." This patch implements that request.
(In reply to comment #7) > As Antti said on bug 85580: > "Parsing CSS values on style resolve time is going to be horribly slow. A good approach might be to cache the parsed values to the variable declaration value itself (with id/type key)." > This patch implements that request. This patch does not resemble my comment at all.