24399 2009-03-05 12:32:25 -0800 Imported user style sheets fail to display 2010-09-28 09:30:11 -0700 1 1 1 Unclassified WebKit CSS 528+ (Nightly build) Mac OS X 10.5 ASSIGNED InRadar P2 Normal --- 1 pg hyatt aroben hyatt jberlin oldest_to_newest 112427 0 pg 2009-03-05 12:32:25 -0800 If a user defined style sheet imports additional style sheets, the imported style sheets will only be displayed on first load. To reproduce: (Please see the example attached html and css file) 1) Create a style sheet that does nothing but import another style sheet 2) Define a second style sheet imported by the first style sheet with some simple style definitions 3) In Safari, set your user style sheet to be the first style sheet which import the second one 4) Quit and relaunch Safari 4) Create simple html page which will show the style definitions you added to the second style sheet 5) Load the simple html page in Safari - the styles will be show 6) Reload the page - the styles will vanish Though this is not likely to often be a major issue for average users, the real problem is when a non-webbrowser based app is using WebKit and user defined style sheets to display content. In some cases the content will be rendered completely blank after first being displayed. The work around for this has been to override the cache policy to reload ignoring cache when loading content. This was recently broken and still is in the version of WebKit shipping with Safari 4 beta. 112429 1 28318 pg 2009-03-05 12:34:52 -0800 Created attachment 28318 test html file 112431 2 28319 pg 2009-03-05 12:35:24 -0800 Created attachment 28319 first test css file 112432 3 28320 pg 2009-03-05 12:35:44 -0800 Created attachment 28320 second test css file 145502 4 aroben 2009-09-08 12:45:37 -0700 <rdar://problem/7206000> 145504 5 aroben 2009-09-08 12:49:34 -0700 Hyatt thinks this is happening because "user stylesheets don't hold up the display of the document or prevent onload from firing, but they need to". 150175 6 40148 hyatt 2009-09-25 15:13:44 -0700 Created attachment 40148 Patch 150180 7 40148 andersca 2009-09-25 15:20:12 -0700 Comment on attachment 40148 Patch r=me 150184 8 hyatt 2009-09-25 15:25:28 -0700 Fixed in r48773. 240817 9 hyatt 2010-06-21 12:43:44 -0700 Pat says this actually regressed by the time we shipped Safari 5, so it's broken again. :( 273552 10 40148 eric 2010-09-02 13:55:55 -0700 Comment on attachment 40148 Patch Clearing Anders' review since this was landed. 28318 2009-03-05 12:34:52 -0800 2009-03-05 12:34:52 -0800 test html file csstest.html text/plain 127 pg PGh0bWw+Cgo8aGVhZD4KPC9oZWFkPgoKPGJvZHk+Cgo8aDE+VGhpcyBpcyBoZWFkZXIgMTwvaDE+ CjxoMj5UaGlzIGlzIGhlYWRlciAyPC9oMj4KPHA+VGhpcyBpcyBhIHBhcmFncmFwaDwvcD4KCjwv Ym9keT4KPC9odG1sPg== 28319 2009-03-05 12:35:24 -0800 2009-03-05 12:35:24 -0800 first test css file style.css text/plain 22 pg QGltcG9ydCAic3R5bGUyLmNzcyI7Cg== 28320 2009-03-05 12:35:44 -0800 2009-03-05 12:35:44 -0800 second test css file style2.css text/plain 134 pg Ym9keSB7YmFja2dyb3VuZC1jb2xvcjogeWVsbG93fQpoMSB7YmFja2dyb3VuZC1jb2xvcjogIzAw ZmYwMH0KaDIge2JhY2tncm91bmQtY29sb3I6IHRyYW5zcGFyZW50fQpwIHtiYWNrZ3JvdW5kLWNv bG9yOiByZ2IoMjUwLDAsMjU1KX0= 40148 2009-09-25 15:13:44 -0700 2010-09-02 13:55:55 -0700 Patch patch.txt text/plain 6093 hyatt SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA0ODc3MikKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMjUgQEAKKzIwMDktMDktMjUgIERhdmUgSHlhdHQgIDxoeWF0dEBhcHBsZS5jb20+ CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgaHR0cHM6 Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI0Mzk5CisgICAgICAgIE1ha2UgQGlt cG9ydCB3b3JrIGluIHVzZXIgc3R5bGVzaGVldHMuICBUaGUgZmlyc3QgYnVnIHdhcyB0aGF0IHRo ZSBVUkwgd2Fzbid0IGJlaW5nIHNldCBvbiB0aGUKKyAgICAgICAgdXNlciBzaGVldHMgdGhlbXNl bHZlcywgc28gcmVsYXRpdmUgQGltcG9ydCBVUkxzIGNvdWxkbid0IHJlc29sdmUgcHJvcGVybHku ICBUaGUgc2Vjb25kIGJ1ZworICAgICAgICB3YXMgdGhhdCB0aGUgbG9hZHMgd291bGQgYmUgZGVu aWVkLiAgVGhpcyBpcyBmaXhlZCBieSB1c2luZyB0aGUgcmVxdWVzdFVzZXJDU1NTdHlsZVNoZWV0 IG1ldGhvZAorICAgICAgICBpbnN0ZWFkIG9mIHRoZSBub3JtYWwgcmVxdWVzdCBtZXRob2QuICBJ biBvcmRlciB0byBrbm93IHdoZW4gdG8gZG8gdGhpcywgQ1NTU3R5bGVTaGVldHMgbm93IGhhdmUK KyAgICAgICAgYSBwcm9wZ2F0ZWQgYm9vbGVhbiwgbV9pc1VzZXJTdHlsZVNoZWV0LCB0aGF0IGxl dHMgdGhlbSBrbm93IGlmIHRoZXkgYXJlIHVzZXIgc3R5bGVzaGVldHMgb3Igbm90LgorCisgICAg ICAgICogY3NzL0NTU0ltcG9ydFJ1bGUuY3BwOgorICAgICAgICAoV2ViQ29yZTo6Q1NTSW1wb3J0 UnVsZTo6aW5zZXJ0ZWRJbnRvUGFyZW50KToKKyAgICAgICAgKiBjc3MvQ1NTU3R5bGVTaGVldC5j cHA6CisgICAgICAgIChXZWJDb3JlOjpDU1NTdHlsZVNoZWV0OjpDU1NTdHlsZVNoZWV0KToKKyAg ICAgICAgKiBjc3MvQ1NTU3R5bGVTaGVldC5oOgorICAgICAgICAoV2ViQ29yZTo6Q1NTU3R5bGVT aGVldDo6c2V0SXNVc2VyU3R5bGVTaGVldCk6CisgICAgICAgIChXZWJDb3JlOjpDU1NTdHlsZVNo ZWV0Ojppc1VzZXJTdHlsZVNoZWV0KToKKyAgICAgICAgKiBkb20vRG9jdW1lbnQuY3BwOgorICAg ICAgICAoV2ViQ29yZTo6RG9jdW1lbnQ6OnBhZ2VVc2VyU2hlZXQpOgorICAgICAgICAoV2ViQ29y ZTo6RG9jdW1lbnQ6OnBhZ2VHcm91cFVzZXJTaGVldHMpOgorCiAyMDA5LTA5LTI1ICBTaW1vbiBG cmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERh cmluIEFkbGVyLgpJbmRleDogV2ViQ29yZS9jc3MvQ1NTSW1wb3J0UnVsZS5jcHAKPT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PQotLS0gV2ViQ29yZS9jc3MvQ1NTSW1wb3J0UnVsZS5jcHAJKHJldmlzaW9uIDQ4NzYzKQorKysg V2ViQ29yZS9jc3MvQ1NTSW1wb3J0UnVsZS5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTExMSw3ICsx MTEsMTAgQEAgdm9pZCBDU1NJbXBvcnRSdWxlOjppbnNlcnRlZEludG9QYXJlbnQoKQogICAgICAg ICByb290ID0gY3VycjsKICAgICB9CiAKLSAgICBtX2NhY2hlZFNoZWV0ID0gZG9jTG9hZGVyLT5y ZXF1ZXN0Q1NTU3R5bGVTaGVldChhYnNIcmVmLCBwYXJlbnRTaGVldC0+Y2hhcnNldCgpKTsKKyAg ICBpZiAocGFyZW50U2hlZXQtPmlzVXNlclN0eWxlU2hlZXQoKSkKKyAgICAgICAgbV9jYWNoZWRT aGVldCA9IGRvY0xvYWRlci0+cmVxdWVzdFVzZXJDU1NTdHlsZVNoZWV0KGFic0hyZWYsIHBhcmVu dFNoZWV0LT5jaGFyc2V0KCkpOworICAgIGVsc2UKKyAgICAgICAgbV9jYWNoZWRTaGVldCA9IGRv Y0xvYWRlci0+cmVxdWVzdENTU1N0eWxlU2hlZXQoYWJzSHJlZiwgcGFyZW50U2hlZXQtPmNoYXJz ZXQoKSk7CiAgICAgaWYgKG1fY2FjaGVkU2hlZXQpIHsKICAgICAgICAgLy8gaWYgdGhlIGltcG9y dCBydWxlIGlzIGlzc3VlZCBkeW5hbWljYWxseSwgdGhlIHNoZWV0IG1heSBiZQogICAgICAgICAv LyByZW1vdmVkIGZyb20gdGhlIHBlbmRpbmcgc2hlZXQgY291bnQsIHNvIGxldCB0aGUgZG9jIGtu b3cKSW5kZXg6IFdlYkNvcmUvY3NzL0NTU1N0eWxlU2hlZXQuY3BwCj09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdl YkNvcmUvY3NzL0NTU1N0eWxlU2hlZXQuY3BwCShyZXZpc2lvbiA0ODc2MykKKysrIFdlYkNvcmUv Y3NzL0NTU1N0eWxlU2hlZXQuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC00MCwyMCArNDAsMjIgQEAg Q1NTU3R5bGVTaGVldDo6Q1NTU3R5bGVTaGVldChDU1NTdHlsZVNoZQogICAgICwgbV9jaGFyc2V0 KGNoYXJzZXQpCiAgICAgLCBtX2xvYWRDb21wbGV0ZWQoZmFsc2UpCiAgICAgLCBtX3N0cmljdFBh cnNpbmcoIXBhcmVudFNoZWV0IHx8IHBhcmVudFNoZWV0LT51c2VTdHJpY3RQYXJzaW5nKCkpCisg ICAgLCBtX2lzVXNlclN0eWxlU2hlZXQocGFyZW50U2hlZXQgPyBwYXJlbnRTaGVldC0+aXNVc2Vy U3R5bGVTaGVldCgpIDogZmFsc2UpCiB7CiB9CiAKLUNTU1N0eWxlU2hlZXQ6OkNTU1N0eWxlU2hl ZXQoTm9kZSAqcGFyZW50Tm9kZSwgY29uc3QgU3RyaW5nJiBocmVmLCBjb25zdCBTdHJpbmcmIGNo YXJzZXQpCitDU1NTdHlsZVNoZWV0OjpDU1NTdHlsZVNoZWV0KE5vZGUqIHBhcmVudE5vZGUsIGNv bnN0IFN0cmluZyYgaHJlZiwgY29uc3QgU3RyaW5nJiBjaGFyc2V0KQogICAgIDogU3R5bGVTaGVl dChwYXJlbnROb2RlLCBocmVmKQogICAgICwgbV9kb2MocGFyZW50Tm9kZS0+ZG9jdW1lbnQoKSkK ICAgICAsIG1fbmFtZXNwYWNlcygwKQogICAgICwgbV9jaGFyc2V0KGNoYXJzZXQpCiAgICAgLCBt X2xvYWRDb21wbGV0ZWQoZmFsc2UpCiAgICAgLCBtX3N0cmljdFBhcnNpbmcoZmFsc2UpCisgICAg LCBtX2lzVXNlclN0eWxlU2hlZXQoZmFsc2UpCiB7CiB9CiAKLUNTU1N0eWxlU2hlZXQ6OkNTU1N0 eWxlU2hlZXQoQ1NTUnVsZSAqb3duZXJSdWxlLCBjb25zdCBTdHJpbmcmIGhyZWYsIGNvbnN0IFN0 cmluZyYgY2hhcnNldCkKK0NTU1N0eWxlU2hlZXQ6OkNTU1N0eWxlU2hlZXQoQ1NTUnVsZSogb3du ZXJSdWxlLCBjb25zdCBTdHJpbmcmIGhyZWYsIGNvbnN0IFN0cmluZyYgY2hhcnNldCkKICAgICA6 IFN0eWxlU2hlZXQob3duZXJSdWxlLCBocmVmKQogICAgICwgbV9uYW1lc3BhY2VzKDApCiAgICAg LCBtX2NoYXJzZXQoY2hhcnNldCkKQEAgLTYyLDYgKzY0LDcgQEAgQ1NTU3R5bGVTaGVldDo6Q1NT U3R5bGVTaGVldChDU1NSdWxlICpvdwogewogICAgIENTU1N0eWxlU2hlZXQqIHBhcmVudFNoZWV0 ID0gb3duZXJSdWxlID8gb3duZXJSdWxlLT5wYXJlbnRTdHlsZVNoZWV0KCkgOiAwOwogICAgIG1f ZG9jID0gcGFyZW50U2hlZXQgPyBwYXJlbnRTaGVldC0+ZG9jKCkgOiAwOworICAgIG1faXNVc2Vy U3R5bGVTaGVldCA9IHBhcmVudFNoZWV0ID8gcGFyZW50U2hlZXQtPmlzVXNlclN0eWxlU2hlZXQo KSA6IGZhbHNlOwogfQogCiBDU1NTdHlsZVNoZWV0Ojp+Q1NTU3R5bGVTaGVldCgpCkluZGV4OiBX ZWJDb3JlL2Nzcy9DU1NTdHlsZVNoZWV0LmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9jc3MvQ1NT U3R5bGVTaGVldC5oCShyZXZpc2lvbiA0ODc2MykKKysrIFdlYkNvcmUvY3NzL0NTU1N0eWxlU2hl ZXQuaAkod29ya2luZyBjb3B5KQpAQCAtOTMsNiArOTMsOSBAQCBwdWJsaWM6CiAgICAgdm9pZCBz ZXRTdHJpY3RQYXJzaW5nKGJvb2wgYikgeyBtX3N0cmljdFBhcnNpbmcgPSBiOyB9CiAgICAgYm9v bCB1c2VTdHJpY3RQYXJzaW5nKCkgY29uc3QgeyByZXR1cm4gbV9zdHJpY3RQYXJzaW5nOyB9CiAK KyAgICB2b2lkIHNldElzVXNlclN0eWxlU2hlZXQoYm9vbCBiKSB7IG1faXNVc2VyU3R5bGVTaGVl dCA9IGI7IH0KKyAgICBib29sIGlzVXNlclN0eWxlU2hlZXQoKSBjb25zdCB7IHJldHVybiBtX2lz VXNlclN0eWxlU2hlZXQ7IH0KKwogcHJpdmF0ZToKICAgICBDU1NTdHlsZVNoZWV0KE5vZGUqIG93 bmVyTm9kZSwgY29uc3QgU3RyaW5nJiBocmVmLCBjb25zdCBTdHJpbmcmIGNoYXJzZXQpOwogICAg IENTU1N0eWxlU2hlZXQoQ1NTU3R5bGVTaGVldCogcGFyZW50U2hlZXQsIGNvbnN0IFN0cmluZyYg aHJlZiwgY29uc3QgU3RyaW5nJiBjaGFyc2V0KTsKQEAgLTEwNiw2ICsxMDksNyBAQCBwcml2YXRl OgogICAgIFN0cmluZyBtX2NoYXJzZXQ7CiAgICAgYm9vbCBtX2xvYWRDb21wbGV0ZWQgOiAxOwog ICAgIGJvb2wgbV9zdHJpY3RQYXJzaW5nIDogMTsKKyAgICBib29sIG1faXNVc2VyU3R5bGVTaGVl dCA6IDE7CiB9OwogCiB9IC8vIG5hbWVzcGFjZQpJbmRleDogV2ViQ29yZS9kb20vRG9jdW1lbnQu Y3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvZG9tL0RvY3VtZW50LmNwcAkocmV2aXNpb24gNDg3 NjMpCisrKyBXZWJDb3JlL2RvbS9Eb2N1bWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTE5MzIs NiArMTkzMiwxMCBAQCBDU1NTdHlsZVNoZWV0KiBEb2N1bWVudDo6cGFnZVVzZXJTaGVldCgpCiAg ICAgaWYgKG1fcGFnZVVzZXJTaGVldCkKICAgICAgICAgcmV0dXJuIG1fcGFnZVVzZXJTaGVldC5n ZXQoKTsKICAgICAKKyAgICBTZXR0aW5ncyogcGFnZVNldHRpbmdzID0gc2V0dGluZ3MoKTsKKyAg ICBpZiAoIXBhZ2VTZXR0aW5ncykKKyAgICAgICAgcmV0dXJuIDA7CisgICAgCiAgICAgUGFnZSog b3duaW5nUGFnZSA9IHBhZ2UoKTsKICAgICBpZiAoIW93bmluZ1BhZ2UpCiAgICAgICAgIHJldHVy biAwOwpAQCAtMTk0MSw3ICsxOTQ1LDggQEAgQ1NTU3R5bGVTaGVldCogRG9jdW1lbnQ6OnBhZ2VV c2VyU2hlZXQoKQogICAgICAgICByZXR1cm4gMDsKICAgICAKICAgICAvLyBQYXJzZSB0aGUgc2hl ZXQgYW5kIGNhY2hlIGl0LgotICAgIG1fcGFnZVVzZXJTaGVldCA9IENTU1N0eWxlU2hlZXQ6OmNy ZWF0ZSh0aGlzKTsKKyAgICBtX3BhZ2VVc2VyU2hlZXQgPSBDU1NTdHlsZVNoZWV0OjpjcmVhdGUo dGhpcywgcGFnZVNldHRpbmdzLT51c2VyU3R5bGVTaGVldExvY2F0aW9uKCkpOworICAgIG1fcGFn ZVVzZXJTaGVldC0+c2V0SXNVc2VyU3R5bGVTaGVldCh0cnVlKTsKICAgICBtX3BhZ2VVc2VyU2hl ZXQtPnBhcnNlU3RyaW5nKHVzZXJTaGVldFRleHQsICFpbkNvbXBhdE1vZGUoKSk7CiAgICAgcmV0 dXJuIG1fcGFnZVVzZXJTaGVldC5nZXQoKTsKIH0KQEAgLTE5NzMsNyArMTk3OCw4IEBAIGNvbnN0 IFZlY3RvcjxSZWZQdHI8Q1NTU3R5bGVTaGVldD4gPiogRG8KICAgICAgICAgY29uc3QgVXNlclN0 eWxlU2hlZXRWZWN0b3IqIHNoZWV0cyA9IGl0LT5zZWNvbmQ7CiAgICAgICAgIGZvciAodW5zaWdu ZWQgaSA9IDA7IGkgPCBzaGVldHMtPnNpemUoKTsgKytpKSB7CiAgICAgICAgICAgICBjb25zdCBV c2VyU3R5bGVTaGVldCogc2hlZXQgPSBzaGVldHMtPmF0KGkpLmdldCgpOwotICAgICAgICAgICAg UmVmUHRyPENTU1N0eWxlU2hlZXQ+IHBhcnNlZFNoZWV0ID0gQ1NTU3R5bGVTaGVldDo6Y3JlYXRl KGNvbnN0X2Nhc3Q8RG9jdW1lbnQqPih0aGlzKSk7CisgICAgICAgICAgICBSZWZQdHI8Q1NTU3R5 bGVTaGVldD4gcGFyc2VkU2hlZXQgPSBDU1NTdHlsZVNoZWV0OjpjcmVhdGUoY29uc3RfY2FzdDxE b2N1bWVudCo+KHRoaXMpLCBzaGVldC0+dXJsKCkpOworICAgICAgICAgICAgcGFyc2VkU2hlZXQt PnNldElzVXNlclN0eWxlU2hlZXQodHJ1ZSk7CiAgICAgICAgICAgICBwYXJzZWRTaGVldC0+cGFy c2VTdHJpbmcoc2hlZXQtPnNvdXJjZSgpLCAhaW5Db21wYXRNb2RlKCkpOwogICAgICAgICAgICAg aWYgKCFtX3BhZ2VHcm91cFVzZXJTaGVldHMpCiAgICAgICAgICAgICAgICAgbV9wYWdlR3JvdXBV c2VyU2hlZXRzLnNldChuZXcgVmVjdG9yPFJlZlB0cjxDU1NTdHlsZVNoZWV0PiA+KTsK