Content-Length: 286589 | pFad | https://github.com/w3c/longtasks/issues/75

12 Top-level browsing context scoping · Issue #75 · w3c/longtasks · GitHub
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

Top-level browsing context scoping #75

Closed
annevk opened this issue Nov 22, 2019 · 10 comments
Closed

Top-level browsing context scoping #75

annevk opened this issue Nov 22, 2019 · 10 comments

Comments

@annevk
Copy link
Member

annevk commented Nov 22, 2019

Why does this use top-level browsing context scoping? If A nests B and we run the event loop for B, we end up passing A's browsing context to the Long Tasks API from HTML's event loop algorithm. This seems rather weird. (And that in turn will do something with both A and B.)

@npm1
Copy link
Contributor

npm1 commented Nov 22, 2019

Yea, I agree that it's a bit weird. What we want is to report longtasks to browsing contexts sharing the same renderer process (is this equivalent to sharing the same event loop, or can there be more than one event loop per renderer process?). Is there a concept for this in the spec we may use?

@annevk
Copy link
Member Author

annevk commented Nov 24, 2019

A browsing context can encompass multiple processes so perhaps using them at all is a problem here?

@npm1
Copy link
Contributor

npm1 commented Nov 27, 2019

Ok, so we can change it to use the docs that are defined in the Update the rendering step.

@nicjansma
Copy link

@plehegar mentions we may want to also figure out if other specs are affected by this same discrepancy

@annevk
Copy link
Member Author

annevk commented Apr 14, 2020

Well, this is the only specification that update the rendering invokes with browsing contexts, right?

@yoavweiss
Copy link
Contributor

@plehegar - could you clarify your comment from the last call?

@npm1
Copy link
Contributor

npm1 commented Apr 14, 2020

Re-adding the requires-discussion label since I think we need to agree on the scope on which longtasks are reported. Right now, if If A nests B and B has a longtask then it may be reported in A when they share a process, regardless of origen (sure, implementation detail, but I think that's the current state).

@annevk
Copy link
Member Author

annevk commented Apr 15, 2020

That sounds very bad and indeed a bug.

@domenic
Copy link
Contributor

domenic commented Apr 15, 2020

It's not obvious it's bad, although it's bad that it wasn't discussed. Performance APIs are most useful when they measure all the things that affect you, and for long tasks, I think that would be everything in the same thread at least. (Which reduces to same process if we're ignoring tasks in workers, which I think this API does.)

But, we're currently moving toward restricting these sorts of process-wide APIs behind COOP+COEP. I'm not sure what attack exactly would be possible by detecting cross-site long tasks though...

@npm1
Copy link
Contributor

npm1 commented Apr 15, 2020

I imagine this was discussed at some point before this API was shipped. @spanicker can you provide some context? Oh I found this relevant document but not clear to me who it was shared with outside of Google at the time. https://docs.google.com/document/d/1tIMI1gau_q6X5EBnjDNiFS5NWV9cpYJ5KKA7xPd3VB8/edit

@npm1 npm1 closed this as completed in 1dadf85 Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/w3c/longtasks/issues/75

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy