Bug 216134
| Summary: | Each step in Page::updateRendering() should use the same set of documents | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> |
| Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | cdumez, graouts, rniwa, simon.fraser, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | Safari Technology Preview | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 202843 | ||
Simon Fraser (smfr)
https://html.spec.whatwg.org/multipage/webappapis.html#event-loop-processing-model implies that once you've collected the set of documents in step 11.1, you use that same set for all the subsequent steps.
We don't follow this; we call forEachDocument() at each step, so potentially we'll pick up a new document that was added by the previous step.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ryosuke Niwa
(In reply to Simon Fraser (smfr) from comment #0)
> https://html.spec.whatwg.org/multipage/webappapis.html#event-loop-processing-
> model implies that once you've collected the set of documents in step 11.1,
> you use that same set for all the subsequent steps.
>
> We don't follow this; we call forEachDocument() at each step, so potentially
> we'll pick up a new document that was added by the previous step.
Right, I wanna make this code change with a test.
Radar WebKit Bug Importer
<rdar://problem/68654053>