#505: HTML Horizontal Review: misc / reorg / cleanup
Discussions
Discussed
Apr 27, 2020 (See Github)
Tess: a ... collection of [small] changes.
... it ends up being a huge diff as it impacts every constructor, but the actual meat of the change is fairly boring. the interesting change is the html constructor change itself...
... next one 5190 - there's a separate DOM parsing spec - this change moves that definition into the HTML spec. Results in some changes to the DOM parser. Mostly nits. This fixes defining what the URL of the document is. It fixes something that the previous DOM parser spec didn't do clearly. ... when there's an error what happens? This is also [uncontroversial].
... lastly: "Cleanups to "execute a script block” Signpost the debate on some script mismatches" - when we chunk out work for people to review.. this last one is an example - there was another, bigger changes that got split into 2 - the other change is the interesting one and this is the editorial stuff that got pulled out to clean up review of the interesting one.
th..th..that's all folks.
Comment by @hober Apr 27, 2020 (See Github)
We looked at these three changes in a breakout today, and we don't think there's anything of concern here.
OpenedApr 13, 2020
This is a subtask of #499. Please review these three HTML PRs:
HTMLConstructor
DOMParser
class to HTML