Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HTML Processing Model: Event Loop step is wrong #174

Open
smfr opened this issue Oct 30, 2016 · 2 comments
Open

HTML Processing Model: Event Loop step is wrong #174

smfr opened this issue Oct 30, 2016 · 2 comments

Comments

@smfr
Copy link

smfr commented Oct 30, 2016

The proposed "HTML Processing Model: Event Loop" step is incomplete:

For each fully active Document in docs, Run the update intersection observations steps for each IntersectionObserver whose root is in the DOMtree of that Document.

This doesn't catch scrolling a frame having a subframe with a targeted element, i.e observers in subframes with an implicit root need to run when any ancestor frame gets scrolled or laid out.

@szager-chromium
Copy link
Collaborator

The implicit root is, by definition, in the DOMtree of the top-level document.

But leaving that aside, I'm wondering why this would be an issue. If the HTML event loop is running, then every active document will run these steps. Does it matter whether the IO steps run when the the top-level document is processed, or when the embedded document is processed?

@miketaylr
Copy link
Member

@smfr any thoughts? Or should we close this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants