22952 2008-12-21 04:53:12 -0800 Cached CSS stylesheet reuses already-decoded form, even if it was decoded with the wrong charset 2010-05-24 15:24:30 -0700 1 1 1 Unclassified WebKit Page Loading 528+ (Nightly build) Mac OS X 10.5 NEW HasReduction, InRadar P2 Normal --- 1 ddkilzer webkit-unassigned ap darin ian koivisto oldest_to_newest 103106 0 ddkilzer 2008-12-21 04:53:12 -0800 * SUMMARY When a CSS stylesheet with no charset is first loaded, it gets the charset of the referring HTML document. If a second HTML document includes the exact same CSS stylesheet, the stylesheet will continue using the charset of the first document instead of the second. * STEPS TO REPRODUCE 1. Open the soon-to-be-attached test case on thie bug. 2. Open URL: http://simon.html5.org/test/css/syndata/representation/001.htm * RESULTS No matter how many times the page is reloaded, <http://simon.html5.org/test/css/syndata/representation/001.htm> always fails until the WebCore cache is cleared. * REGRESSION On shipping Safari 3.2.1, subsequent reloads of <http://simon.html5.org/test/css/syndata/representation/001.htm> appear to fix the encoding. On ToT WebKit, it doesn't change until the WebCore cache is cleared. * NOTES See Bug 11850 for the original analysis. 103107 1 26182 ddkilzer 2008-12-21 04:53:55 -0800 Created attachment 26182 Test case (Step 1) 103108 2 ddkilzer 2008-12-21 04:56:38 -0800 (In reply to comment #0) > * RESULTS > No matter how many times the page is reloaded, > <http://simon.html5.org/test/css/syndata/representation/001.htm> always fails > until the WebCore cache is cleared. Actually, clearing the WebCore cache doesn't seem to have an effect (nor does Cmd-Opt-E). The only way to fix the charset is to restart the browser. 103110 3 ddkilzer 2008-12-21 04:59:10 -0800 When this bug is fixed, consider implementing (and thoroughly testing) this charset fallback mechanism: http://www.w3.org/TR/CSS21/syndata.html#charset 103116 4 ap 2008-12-21 09:18:35 -0800 A somewhat more practical manifestation of the same problem is tracked as: <rdar://problem/5809774> Safari doesn't reload scripts once Text Encoding is changed. (In reply to comment #3) > When this bug is fixed, consider implementing (and thoroughly testing) this > charset fallback mechanism: > > http://www.w3.org/TR/CSS21/syndata.html#charset We have it implemented and tested already - are you aware of any bugs? 103122 5 ddkilzer 2008-12-21 10:19:21 -0800 (In reply to comment #4) > (In reply to comment #3) > > When this bug is fixed, consider implementing (and thoroughly testing) this > > charset fallback mechanism: > > > > http://www.w3.org/TR/CSS21/syndata.html#charset > > We have it implemented and tested already - are you aware of any bugs? Great! I guess I was looking for one method where the fallback logic was contained, but couldn't find it upon initial inspection. 103123 6 koivisto 2008-12-21 10:33:14 -0800 Using shift-reload fixes the problem in TOT. The bug here is that an existing cache entry should not be used if charset differs. The bug has been around for ever so it probably does not occur much in real world (though the recent reloading changes mean that it now needs shift-reload to fix instead of a regular reload). 103128 7 darin 2008-12-21 11:06:54 -0800 This seems like a minor issue since it only affects cases where the same stylesheet is used with different encodings. If this really was a common issue, we would want to detect cases where the stylesheet was entirely ASCII and so not affected by the encoding, because it would be a shame not to share the cached stylesheet in that case. For testing and software development purposes, there could be some value to fixing this. 229941 8 ap 2010-05-24 15:24:30 -0700 See also: bug 27802. 26182 2008-12-21 04:53:55 -0800 2008-12-21 04:53:55 -0800 Test case (Step 1) 001.html text/html;charset=shift-jis 435 ddkilzer PCFkb2N0eXBlIGh0bWw+CjxtZXRhIGh0dHAtZXF1aXY9IkNvbnRlbnQtVHlwZSIgY29udGVudD0i dGV4dC9odG1sO2NoYXJzZXQ9c2hpZnQtamlzIj4KPHRpdGxlPnV0Zi04IGluIGV4dGVybmFsIHN0 eWxlc2hlZXQ8L3RpdGxlPgo8bGluayByZWw9InN0eWxlc2hlZXQiIGhyZWY9Imh0dHA6Ly9zaW1v bi5odG1sNS5vcmcvdGVzdC9jc3Mvc3luZGF0YS9yZXByZXNlbnRhdGlvbi9zdXBwb3J0LzAwMS5j c3MiPgo8bGluayByZWw9ImhlbHAiIGhyZWY9Imh0dHA6Ly93d3cudzMub3JnL1RSL0NTUzIxL3N5 bmRhdGEuaHRtbCNxMjMiPgo8IS0tIHdlYmtpdCAjMTEwMTEgLS0+CjxwPlRoZSBmb2xsb3dpbmcg dHdvIGxpbmVzIHNob3VsZCBiZSBlcXVpdmFsZW50Ogo8ZGl2PknDsXTDq3Juw6J0acO0bsOgbGl6 w6Z0acO4bjwvZGl2Pgo8ZGl2IGlkPSJ0ZXN0Ij48L2Rpdj4K