36399 2010-03-19 16:55:05 -0700 WorkerScriptLoader always (and unfortunately sometimes) inherits caching policy from the parent context. 2012-05-09 16:38:50 -0700 1 1 1 Unclassified WebKit WebCore JavaScript 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 dimich webkit-unassigned ap atwilson fishd japhet levin tonyg oldest_to_newest 202105 0 dimich 2010-03-19 16:55:05 -0700 Probably since we switched from using CachedScript to ThreadableLoader in WorkerScriptLoader (http://trac.webkit.org/changeset/44726), the caching behavior changed in somewhat unfortunate way: when the page that creates more then 1 worker from the same script is "reloaded", it re-validates all the resources via server roundtrip and if multiple workers are created from the same url, they get revalidated every time. Here is the test created by Drew Wilson: http://www.corp.google.com/~atwilson/worker-latency.html Click the link, it'll create several workers. Note the printed time. Hit refresh. It'll create several workers again, this time taking longer for each one, for roundtrip to the server. 202175 1 fishd 2010-03-19 22:56:48 -0700 See http://code.google.com/p/chromium/issues/detail?id=34092#c10. I don't think this bug is unique to workers, and the fix probably can be made to SubresourceLoader in a manner that would work for all resource loads made after the page has loaded. 202176 2 ap 2010-03-19 23:02:14 -0700 See also: bug 32423, bug 30862. We seem to be having two opposite bugs at once. 203303 3 51470 atwilson 2010-03-23 18:47:54 -0700 Created attachment 51470 HTML file that shows this bug 203304 4 51471 atwilson 2010-03-23 18:50:25 -0700 Created attachment 51471 Associated JS file (worker script) that shows the bug. Added worker-latency.html and worker.js files as attachments for this bug. Download both files into a single directory, load worker-latency.html in a browser, then hit reload to reproduce the issue. 620004 5 tonyg 2012-05-09 16:30:52 -0700 This no longer reproduces using atwilson's test case. Worker loaded scripts come from the cache even after a reload. Please reopen with explanation if I'm missing something. 620015 6 levin 2012-05-09 16:38:43 -0700 Likely it works now due to Nate's changes here http://trac.webkit.org/changeset/98380 and subsequent related changes later. 51470 2010-03-23 18:47:54 -0700 2010-03-23 18:47:54 -0700 HTML file that shows this bug worker-latency.html text/html 1134 atwilson PGh0bWw+Cjxib2R5Pgo8ZGl2IGlkPXJlc3VsdD48L2Rpdj4KPHNjcmlwdD4KICBmdW5jdGlvbiBs b2cobWVzc2FnZSkKICB7CiAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgicmVzdWx0IikuaW5u ZXJIVE1MICs9IG1lc3NhZ2UgKyAiPGJyLz4iOwogIH0KCiAgbG9nKCJQcmltaW5nIGNhY2hlIGJ5 IHN0YXJ0aW5nIHVwIGluaXRpYWwgd29ya2VyIik7CiAgd29ya2VyID0gbmV3IFdvcmtlcigid29y a2VyLmpzIik7CiAgd29ya2VyLm9ubWVzc2FnZSA9IGZ1bmN0aW9uKGV2dCkgewogICAgbG9nKCJD YWNoZSBwcmltZWQiKTsKICB9CgogIHZhciBzdGFydDsKICB2YXIgbnVtV29ya2VycyA9IDEwOwog IHZhciB3b3JrZXJzID0gW107CiAgZnVuY3Rpb24gc3RhcnRUZXN0KCkgewogICAgc3RhcnQgPSBu ZXcgRGF0ZSgpOwogICAgY3JlYXRlV29ya2VyKCk7CiAgfQogIGZ1bmN0aW9uIGNyZWF0ZVdvcmtl cigpIHsKICAgIHZhciB3b3JrZXIgPSBuZXcgV29ya2VyKCJ3b3JrZXIuanMiKTsKICAgIHdvcmtl cnNbd29ya2Vycy5sZW5ndGhdID0gd29ya2VyOwogICAgd29ya2VyLm9ubWVzc2FnZSA9IGZ1bmN0 aW9uKGV2dCkgewogICAgICBsb2coZXZ0LmRhdGEpOwogICAgICBpZiAod29ya2Vycy5sZW5ndGgg Pj0gbnVtV29ya2VycykgewogICAgICAgIGxvZyhudW1Xb3JrZXJzICsgIiB3b3JrZXJzIGNyZWF0 ZWQiKTsKICAgICAgICB2YXIgZWxhcHNlZCA9IChuZXcgRGF0ZSgpKS5nZXRUaW1lKCkgLSBzdGFy dC5nZXRUaW1lKCk7CiAgICAgICAgbG9nKCJUb3RhbCBlbGFwc2VkIHRpbWU6ICIgKyBlbGFwc2Vk ICsgIiBtcyIpOwogICAgICAgIGxvZygiQXZlcmFnZTogIiArIChlbGFwc2VkL251bVdvcmtlcnMp ICsgIiBtcyIpOwogICAgICAgIGZvciAodmFyIGkgPSAwIDsgaSA8IHdvcmtlcnMubGVuZ3RoIDsg aSsrKSB7CiAgICAgICAgICB3b3JrZXJzW2ldLnRlcm1pbmF0ZSgpOwogICAgICAgIH0KICAgICAg ICB3b3JrZXJzID0gW107CiAgICAgIH0gZWxzZSB7CiAgICAgICAgY3JlYXRlV29ya2VyKCk7CiAg ICAgIH0KICAgIH0KICB9CiA8L3NjcmlwdD4KCjxidXR0b24gb25jbGljaz0ic3RhcnRUZXN0KCki PlN0YXJ0IFdvcmtlciBUaW1pbmcgVGVzdDwvYnV0dG9uPgo8L2JvZHk+CjwvaHRtbD4K 51471 2010-03-23 18:50:25 -0700 2010-03-23 18:50:25 -0700 Associated JS file (worker script) that shows the bug. worker.js application/javascript 769 atwilson b25tZXNzYWdlID0gZnVuY3Rpb24oZXZ0KSB7CiAgaWYgKGV2dC5kYXRhID09ICJwaW5nIikgewog ICAgcG9zdE1lc3NhZ2UoInBvbmciKTsKICB9IGVsc2UgaWYgKGV2dC5kYXRhLmluZGV4T2YoImJ1 bGtzZW5kICIpID09IDApIHsKICAgIHZhciBudW1JdGVtcyA9IHBhcnNlSW50KGV2dC5kYXRhLnNw bGl0KCIgIiwgMilbMV0pOwogICAgZm9yICh2YXIgaSA9IDAgOyBpIDwgbnVtSXRlbXMgOyBpKysp IHsKICAgICAgcG9zdE1lc3NhZ2UoImJ1bGtzZW5kIik7CiAgICB9CiAgfSBlbHNlIGlmIChldnQu ZGF0YSA9PSAicG9ydCIpIHsKICAgIHZhciBwb3J0ID0gZXZ0LnBvcnRzWzBdOwogICAgcG9ydC5v bm1lc3NhZ2UgPSBmdW5jdGlvbihldnQpIHsKICAgICAgaWYgKGV2dC5kYXRhID09ICJwaW5nIikg ewogICAgICAgIHBvcnQucG9zdE1lc3NhZ2UoInBvbmciKTsKICAgICAgfSBlbHNlIGlmIChldnQu ZGF0YS5pbmRleE9mKCJidWxrc2VuZCAiKSA9PSAwKSB7CiAgICAgICAgdmFyIG51bUl0ZW1zID0g cGFyc2VJbnQoZXZ0LmRhdGEuc3BsaXQoIiAiLCAyKVsxXSk7CiAgICAgICAgZm9yICh2YXIgaSA9 IDAgOyBpIDwgbnVtSXRlbXMgOyBpKyspIHsKICAgICAgICAgIHBvcnQucG9zdE1lc3NhZ2UoImJ1 bGtzZW5kIik7CiAgICAgICAgfQogICAgICB9CiAgICB9OwogIH0gZWxzZSB7CiAgICBwb3N0TWVz c2FnZSgid29ya2VyIHJlY2VpdmVkIGludmFsaWQgbWVzc2FnZTogIiArIGV2dC5kYXRhKTsKICB9 Cn07Cgpwb3N0TWVzc2FnZSgic3RhcnRlZCIpOw==