changcheng / wro4j

Automatically exported from code.google.com/p/wro4j
0 stars 0 forks source link

Content length is not set #420

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Issue reported on mailing list: 
https://groups.google.com/forum/?hl=en&fromgroups#!topic/wro4j/Umfftk6RGCM

I think there might be some performance benefit to setting the content length 
of the response. 
I can't really get any appreciable benchmarks that shows this makes a 
difference but in theory it does (and perhaps its more noticeable on a mobile 
device).
It appears that Wro4j does not do this.

I was looking at WroManager in:
ro.isdc.wro.manager.WroManager.serveProcessedBundle()

It would be trivial to do something like: response.setContentLength(length).

We have the data so we might as well set the length.

Original issue reported on code.google.com by alex.obj...@gmail.com on 20 Apr 2012 at 8:58

GoogleCodeExporter commented 9 years ago

Original comment by alex.obj...@gmail.com on 20 Apr 2012 at 8:58

GoogleCodeExporter commented 9 years ago
Fixed in 1.4.x branch.

Original comment by alex.obj...@gmail.com on 26 Apr 2012 at 10:07