236319 2022-02-08 12:14:36 -0800 Worker scripts should always be decoded as UTF-8 2022-02-09 10:52:43 -0800 1 1 1 Unclassified WebKit WebCore Misc. WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 230382 1 cdumez cdumez achristensen bfulgham darin ggaren sam webkit-bug-importer youennf oldest_to_newest 1838824 0 cdumez 2022-02-08 12:14:36 -0800 Worker scripts should always be decoded as UTF-8: - https://html.spec.whatwg.org/multipage/webappapis.html#fetch-a-classic-worker-script (Step 7) 1838825 1 451290 cdumez 2022-02-08 12:17:21 -0800 Created attachment 451290 Patch 1838842 2 webkit-bug-importer 2022-02-08 13:01:05 -0800 <rdar://problem/88647606> 1839257 3 451290 achristensen 2022-02-09 10:24:23 -0800 Comment on attachment 451290 Patch This change now matches Firefox and the spec but not Chrome. Do we care? 1839258 4 cdumez 2022-02-09 10:25:21 -0800 (In reply to Alex Christensen from comment #3) > Comment on attachment 451290 [details] > Patch > > This change now matches Firefox and the spec but not Chrome. Do we care? Given we have both the spec and Firefox on our side, I think we should at least try it. 1839259 5 bfulgham 2022-02-09 10:26:22 -0800 (In reply to Chris Dumez from comment #4) > (In reply to Alex Christensen from comment #3) > > Comment on attachment 451290 [details] > > Patch > > > > This change now matches Firefox and the spec but not Chrome. Do we care? > > Given we have both the spec and Firefox on our side, I think we should at > least try it. What does WPT say? Is there coverage for this? 1839260 6 cdumez 2022-02-09 10:27:12 -0800 (In reply to Brent Fulgham from comment #5) > (In reply to Chris Dumez from comment #4) > > (In reply to Alex Christensen from comment #3) > > > Comment on attachment 451290 [details] > > > Patch > > > > > > This change now matches Firefox and the spec but not Chrome. Do we care? > > > > Given we have both the spec and Firefox on our side, I think we should at > > least try it. > > What does WPT say? Is there coverage for this? Look at the patch? :) 1839262 7 bfulgham 2022-02-09 10:29:31 -0800 (In reply to Chris Dumez from comment #6) > (In reply to Brent Fulgham from comment #5) > > (In reply to Chris Dumez from comment #4) > > > (In reply to Alex Christensen from comment #3) > > > > Comment on attachment 451290 [details] > > > > Patch > > > > > > > > This change now matches Firefox and the spec but not Chrome. Do we care? > > > > > > Given we have both the spec and Firefox on our side, I think we should at > > > least try it. > > > > What does WPT say? Is there coverage for this? > > Look at the patch? :) :-P Good point! I say ship it. WPT is truth. 1839285 8 ews-feeder 2022-02-09 10:52:38 -0800 Committed r289489 (247027@main): <https://commits.webkit.org/247027@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 451290. 451290 2022-02-08 12:17:21 -0800 2022-02-09 10:52:40 -0800 Patch bug-236319-20220208121720.patch text/plain 4813 cdumez U3VidmVyc2lvbiBSZXZpc2lvbjogMjg5NDAyCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZmExYmFjNmE4M2Q1MzIw ODQ2Nzc5NWQxODFjNWQ0OGZhN2Y3NGY1Ny4uODUwMTg5Y2M1ZDQ0MDQwMjU3ZTRiYjI3OGJkMzBi ZjQzOTFlNGNiMiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDIyLTAyLTA4ICBDaHJp cyBEdW1leiAgPGNkdW1lekBhcHBsZS5jb20+CisKKyAgICAgICAgV29ya2VyIHNjcmlwdHMgc2hv dWxkIGFsd2F5cyBiZSBkZWNvZGVkIGFzIFVURi04CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMzYzMTkKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP RFkgKE9PUFMhKS4KKworICAgICAgICBXb3JrZXIgc2NyaXB0cyBzaG91bGQgYWx3YXlzIGJlIGRl Y29kZWQgYXMgVVRGLTg6CisgICAgICAgIC0gaHR0cHM6Ly9odG1sLnNwZWMud2hhdHdnLm9yZy9t dWx0aXBhZ2Uvd2ViYXBwYXBpcy5odG1sI2ZldGNoLWEtY2xhc3NpYy13b3JrZXItc2NyaXB0IChT dGVwIDcpCisKKyAgICAgICAgTm8gbmV3IHRlc3RzLCByZWJhc2VsaW5lZCBleGlzdGluZyB0ZXN0 cy4KKworICAgICAgICAqIHdvcmtlcnMvV29ya2VyU2NyaXB0TG9hZGVyLmNwcDoKKyAgICAgICAg KFdlYkNvcmU6OldvcmtlclNjcmlwdExvYWRlcjo6ZGlkUmVjZWl2ZURhdGEpOgorCiAyMDIyLTAy LTA4ICBTYW0gV2VpbmlnICA8d2VpbmlnQGFwcGxlLmNvbT4KIAogICAgICAgICBDb252ZXJzaW9u IHRvIGEgY29sb3Igc3BhY2Ugd2l0aCBhIHNtYWxsZXIgZ2FtdXQgc2hvdWxkIHBlcmZvcm0gZ2Ft dXQgbWFwcGluZwpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvd29ya2Vycy9Xb3JrZXJTY3Jp cHRMb2FkZXIuY3BwIGIvU291cmNlL1dlYkNvcmUvd29ya2Vycy9Xb3JrZXJTY3JpcHRMb2FkZXIu Y3BwCmluZGV4IDlkZmI0Yzk5MWE4Y2I1OTFlMTI4Nzg1ZTZhNmYxNGNkMmExNTI2MjguLjhhY2Ri ZTE4NjNiYWRkNGQ5YjE1Y2M1M2JiYTc4MWYwZTBlNjA0YjggMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X ZWJDb3JlL3dvcmtlcnMvV29ya2VyU2NyaXB0TG9hZGVyLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29y ZS93b3JrZXJzL1dvcmtlclNjcmlwdExvYWRlci5jcHAKQEAgLTE4OSw3ICsxODksNiBAQCB2b2lk IFdvcmtlclNjcmlwdExvYWRlcjo6ZGlkUmVjZWl2ZVJlc3BvbnNlKFJlc291cmNlTG9hZGVySWRl bnRpZmllciBpZGVudGlmaWVyLAogICAgIG1fcmVzcG9uc2VVUkwgPSByZXNwb25zZS51cmwoKTsK ICAgICBtX2NlcnRpZmljYXRlSW5mbyA9IHJlc3BvbnNlLmNlcnRpZmljYXRlSW5mbygpID8gKnJl c3BvbnNlLmNlcnRpZmljYXRlSW5mbygpIDogQ2VydGlmaWNhdGVJbmZvKCk7CiAgICAgbV9yZXNw b25zZU1JTUVUeXBlID0gcmVzcG9uc2UubWltZVR5cGUoKTsKLSAgICBtX3Jlc3BvbnNlRW5jb2Rp bmcgPSByZXNwb25zZS50ZXh0RW5jb2RpbmdOYW1lKCk7CiAgICAgbV9yZXNwb25zZVNvdXJjZSA9 IHJlc3BvbnNlLnNvdXJjZSgpOwogICAgIG1faXNSZWRpcmVjdGVkID0gcmVzcG9uc2UuaXNSZWRp cmVjdGVkKCk7CiAgICAgbV9jb250ZW50U2VjdXJpdHlQb2xpY3kgPSBDb250ZW50U2VjdXJpdHlQ b2xpY3lSZXNwb25zZUhlYWRlcnMgeyByZXNwb25zZSB9OwpAQCAtMjA1LDEyICsyMDQsOCBAQCB2 b2lkIFdvcmtlclNjcmlwdExvYWRlcjo6ZGlkUmVjZWl2ZURhdGEoY29uc3QgU2hhcmVkQnVmZmVy JiBidWZmZXIpCiAgICAgaWYgKG1fZmFpbGVkKQogICAgICAgICByZXR1cm47CiAKLSAgICBpZiAo IW1fZGVjb2RlcikgewotICAgICAgICBpZiAoIW1fcmVzcG9uc2VFbmNvZGluZy5pc0VtcHR5KCkp Ci0gICAgICAgICAgICBtX2RlY29kZXIgPSBUZXh0UmVzb3VyY2VEZWNvZGVyOjpjcmVhdGUoInRl eHQvamF2YXNjcmlwdCJfcywgbV9yZXNwb25zZUVuY29kaW5nKTsKLSAgICAgICAgZWxzZQotICAg ICAgICAgICAgbV9kZWNvZGVyID0gVGV4dFJlc291cmNlRGVjb2Rlcjo6Y3JlYXRlKCJ0ZXh0L2ph dmFzY3JpcHQiX3MsICJVVEYtOCIpOwotICAgIH0KKyAgICBpZiAoIW1fZGVjb2RlcikKKyAgICAg ICAgbV9kZWNvZGVyID0gVGV4dFJlc291cmNlRGVjb2Rlcjo6Y3JlYXRlKCJ0ZXh0L2phdmFzY3Jp cHQiX3MsICJVVEYtOCIpOwogCiAgICAgaWYgKGJ1ZmZlci5pc0VtcHR5KCkpCiAgICAgICAgIHJl dHVybjsKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3dvcmtlcnMvV29ya2VyU2NyaXB0TG9h ZGVyLmggYi9Tb3VyY2UvV2ViQ29yZS93b3JrZXJzL1dvcmtlclNjcmlwdExvYWRlci5oCmluZGV4 IDY5NGEwNjc2NGQzN2NhNzMwZmQxZTBiMWQyYjg3YjY1ZDAwZmMwYWYuLjVlMTI0ZjJiMmNhYzdk ZTc0YWM5MmQyMmNiZmViOTdjZTBhNTA1NTYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3dv cmtlcnMvV29ya2VyU2NyaXB0TG9hZGVyLmgKKysrIGIvU291cmNlL1dlYkNvcmUvd29ya2Vycy9X b3JrZXJTY3JpcHRMb2FkZXIuaApAQCAtOTksNyArOTksNiBAQCBwcml2YXRlOgogCiAgICAgV29y a2VyU2NyaXB0TG9hZGVyQ2xpZW50KiBtX2NsaWVudCB7IG51bGxwdHIgfTsKICAgICBSZWZQdHI8 VGhyZWFkYWJsZUxvYWRlcj4gbV90aHJlYWRhYmxlTG9hZGVyOwotICAgIFN0cmluZyBtX3Jlc3Bv bnNlRW5jb2Rpbmc7CiAgICAgUmVmUHRyPFRleHRSZXNvdXJjZURlY29kZXI+IG1fZGVjb2RlcjsK ICAgICBTY3JpcHRCdWZmZXIgbV9zY3JpcHQ7CiAgICAgVVJMIG1fdXJsOwpkaWZmIC0tZ2l0IGEv TGF5b3V0VGVzdHMvaW1wb3J0ZWQvdzNjL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL2ltcG9ydGVk L3czYy9DaGFuZ2VMb2cKaW5kZXggOWQ5MTRlMjUwNGQ0NzQ1OTI2MDNhNzE5MWEyNThiZjE5Zjgw NjE1MC4uOWRhZGJjODMwNTYwZmYzNmU5MjM5YTY1ZjgzNGE0MDE2NGQ1NmFjNiAxMDA2NDQKLS0t IGEvTGF5b3V0VGVzdHMvaW1wb3J0ZWQvdzNjL0NoYW5nZUxvZworKysgYi9MYXlvdXRUZXN0cy9p bXBvcnRlZC93M2MvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMjItMDItMDggIENocmlz IER1bWV6ICA8Y2R1bWV6QGFwcGxlLmNvbT4KKworICAgICAgICBXb3JrZXIgc2NyaXB0cyBzaG91 bGQgYWx3YXlzIGJlIGRlY29kZWQgYXMgVVRGLTgKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp dC5vcmcvc2hvd19idWcuY2dpP2lkPTIzNjMxOQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E WSAoT09QUyEpLgorCisgICAgICAgIFJlYmFzZWxpbmUgV1BUIHRlc3RzIHRoYXQgYXJlIG5vdyBw YXNzaW5nLgorCisgICAgICAgICogd2ViLXBsYXRmb3JtLXRlc3RzL3dvcmtlcnMvc2VtYW50aWNz L2VuY29kaW5ncy8wMDEtZXhwZWN0ZWQudHh0OgorICAgICAgICAqIHdlYi1wbGF0Zm9ybS10ZXN0 cy93b3JrZXJzL3NlbWFudGljcy9lbmNvZGluZ3MvMDAyLWV4cGVjdGVkLnR4dDoKKwogMjAyMi0w Mi0wOCAgSm9uIExlZSAgPGpvbmxlZUBhcHBsZS5jb20+CiAKICAgICAgICAgVW5yZXZpZXdlZCBn YXJkZW5pbmcuCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9pbXBvcnRlZC93M2Mvd2ViLXBsYXRm b3JtLXRlc3RzL3dvcmtlcnMvc2VtYW50aWNzL2VuY29kaW5ncy8wMDEtZXhwZWN0ZWQudHh0IGIv TGF5b3V0VGVzdHMvaW1wb3J0ZWQvdzNjL3dlYi1wbGF0Zm9ybS10ZXN0cy93b3JrZXJzL3NlbWFu dGljcy9lbmNvZGluZ3MvMDAxLWV4cGVjdGVkLnR4dAppbmRleCA1NzZlOWY4OWY0NTI3ZDk5ZmYy ODZlMjMzODZkM2VhYzFjMTA0NGNjLi5jMGJmNDlkODZmMTkxM2Q2NzU4YmQyZTYxZjM5MDhlMjc4 MDA4OTFhIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9pbXBvcnRlZC93M2Mvd2ViLXBsYXRmb3Jt LXRlc3RzL3dvcmtlcnMvc2VtYW50aWNzL2VuY29kaW5ncy8wMDEtZXhwZWN0ZWQudHh0CisrKyBi L0xheW91dFRlc3RzL2ltcG9ydGVkL3czYy93ZWItcGxhdGZvcm0tdGVzdHMvd29ya2Vycy9zZW1h bnRpY3MvZW5jb2RpbmdzLzAwMS1leHBlY3RlZC50eHQKQEAgLTEsMyArMSwzIEBACiAKLUZBSUwg ZW5jb2RpbmcsIGRlZGljYXRlZCB3b3JrZXIgYXNzZXJ0X2VxdWFsczogZXhwZWN0ZWQgIsOlIiBi dXQgZ290ICLDg8KlIgorUEFTUyBlbmNvZGluZywgZGVkaWNhdGVkIHdvcmtlcgogCmRpZmYgLS1n aXQgYS9MYXlvdXRUZXN0cy9pbXBvcnRlZC93M2Mvd2ViLXBsYXRmb3JtLXRlc3RzL3dvcmtlcnMv c2VtYW50aWNzL2VuY29kaW5ncy8wMDItZXhwZWN0ZWQudHh0IGIvTGF5b3V0VGVzdHMvaW1wb3J0 ZWQvdzNjL3dlYi1wbGF0Zm9ybS10ZXN0cy93b3JrZXJzL3NlbWFudGljcy9lbmNvZGluZ3MvMDAy LWV4cGVjdGVkLnR4dAppbmRleCA4ODI3YjVkMTlkODJiYmZmMWE1ZDJlYzYzMzM3Y2M0NjdjNDEy MjJjLi5jM2YyNTc0ZDBlMTQ0ZjQ2NGY4NmYyMTU4NjBiMjNiNmFkODBkYmRlIDEwMDY0NAotLS0g YS9MYXlvdXRUZXN0cy9pbXBvcnRlZC93M2Mvd2ViLXBsYXRmb3JtLXRlc3RzL3dvcmtlcnMvc2Vt YW50aWNzL2VuY29kaW5ncy8wMDItZXhwZWN0ZWQudHh0CisrKyBiL0xheW91dFRlc3RzL2ltcG9y dGVkL3czYy93ZWItcGxhdGZvcm0tdGVzdHMvd29ya2Vycy9zZW1hbnRpY3MvZW5jb2RpbmdzLzAw Mi1leHBlY3RlZC50eHQKQEAgLTEsMyArMSwzIEBACiAKLUZBSUwgZW5jb2RpbmcsIHNoYXJlZCB3 b3JrZXIgYXNzZXJ0X2VxdWFsczogZXhwZWN0ZWQgIsOlIiBidXQgZ290ICLDg8KlIgorUEFTUyBl bmNvZGluZywgc2hhcmVkIHdvcmtlcgogCg==