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    

Description Simon Fraser (smfr) 2020-09-03 11:23:48 PDT
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.
Comment 1 Ryosuke Niwa 2020-09-03 19:56:24 PDT
(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.
Comment 2 Radar WebKit Bug Importer 2020-09-10 11:24:13 PDT
<rdar://problem/68654053>