[
https://issues.apache.org/jira/browse/WICKET-6051?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15057919#comment-15057919
]
Rakesh A commented on WICKET-6051:
----------------------------------
Checked the same CSS file with latest CssUrlReplacer implementation (new
RegEx), I see it perform lot better
First request (no cache) after tomcat startup, it took 110ms, after that
subsequent requests took around 2ms.
But the loop to check for exclusions might impact performance - if we've
reasonably large set of files as exclusions.
> Improve performance of CssUrlReplacer
> -------------------------------------
>
> Key: WICKET-6051
> URL: https://issues.apache.org/jira/browse/WICKET-6051
> Project: Wicket
> Issue Type: Improvement
> Components: wicket
> Affects Versions: 6.20.0, 6.21.0, 7.1.0, 8.0.0-M1
> Reporter: Rakesh A
> Assignee: Tobias Soloschenko
> Fix For: 7.2.0, 8.0.0-M1, 6.22.0
>
> Attachments: CssUrlReplacer.java
>
>
> Implementation of org.apache.wicket.resource.CssUrlReplacer (if enabled),
> might affect performance, and increase the loading time of CSS files, if they
> contain many background-image properties with 'url()' values.
> In one instance, it took 18sec to process a CSS file with 663 instances of
> 'url()'.
> It might help, if we add option to specify 'exclusions' with certain CSS
> resources.
> PR: https://github.com/apache/wicket/pull/147
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
