| Summary: | HTML5 Parser should continue parsing after script execution | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||
| Component: | New Bugs | Assignee: | Eric Seidel (no email) <eric> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | abarth, tonyg | ||||
| Priority: | P2 | ||||||
| Version: | 528+ (Nightly build) | ||||||
| Hardware: | Other | ||||||
| OS: | OS X 10.5 | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 39259 | ||||||
| Attachments: |
|
||||||
|
Description
Eric Seidel (no email)
2010-06-10 03:34:12 PDT
Created attachment 58356 [details]
Patch
Comment on attachment 58356 [details]
Patch
Nice, but can you add your reduced test case? That's a lot easier to understand than the crazy JavaScript test that this actually fixes.
Here's the reduction eric sent me in email:
<pre id="log"></pre>
<script>
function log(string) {
document.getElementById("log").innerHTML += string + "\n";
}
log("1");
document.write("<script>log('2')<"+"/script>");
document.write("<script>log('3')<"+"/script>");
log("4");
</script>
I've converted it to a layout test locally. I just need to build and test it and I can land this for him.
Committed r60966: <http://trac.webkit.org/changeset/60966> |