Worker scripts should always be decoded as UTF-8: - https://html.spec.whatwg.org/multipage/webappapis.html#fetch-a-classic-worker-script (Step 7)
Created attachment 451290 [details] Patch
<rdar://problem/88647606>
Comment on attachment 451290 [details] Patch This change now matches Firefox and the spec but not Chrome. Do we care?
(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.
(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?
(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? :)
(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.
Committed r289489 (247027@main): <https://commits.webkit.org/247027@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 451290 [details].