The classic script's source text isn't defined in HTML any more

[jungkees]

Update algorithm step 8 compares the updating script's source text against the installed script's source text. This check is to return early when the two scripts are byte identical.

I noticed HTML removed the definition of the source text from the classic script struct and added a record (Script Record) to the script struct instead.

I wonder if comparing the record.[[ECMAScriptCode]] of the two scripts would make sense or we should add the source text to the script struct again. From #1023 (comment), the latter seems to be more appropriate I suppose.

/cc @domenic

[domenic]

Oh, sorry about this. It does sound like we need to add the source text back to the script record. I will try to add it back soon, per the comment you cited.

[jungkees]

That sounds good. I presume you meant to add it back to the script struct, right? We need non-parsed raw script texts for both classic scripts and module scripts.

[domenic]

Yep.

[jungkees]

8e25c26 closes this issue by comparing response bodies instead of source texts.