Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Intervention: Throttle Loading in background #60

Closed
toyoshim opened this issue Oct 27, 2017 · 4 comments
Closed

Intervention: Throttle Loading in background #60

toyoshim opened this issue Oct 27, 2017 · 4 comments

Comments

@toyoshim
Copy link

The idea is to limit the number of in-flight resource loading requests in background tabs or frames.

Chrome is experimenting with this idea on m61+.

Rough algorithm

  • manage the in-flight request number per-frame basis
  • if the in-flight requests reaches the limit, and new resource request is needed, the request is pushed into a pending list.
  • if the frame gets to be in foreground, or in-flight request finishes, search the pending list, and issue new requests if the limit allows.

I'm exploring the best limit number for the main frame, and subframes respectively.

#33 and #59 is related, and #59 would overrides this throttling behavior after the N mins

/cc @skyostil @altimin @spanicker

@bzbarsky
Copy link

It's not clear to me why this is an intervention, in the sense of "spec violation that improves things". This behavior is allowed already by existing specs.

@toyoshim
Copy link
Author

Oops. That's right. I just thought it would help to share this because #59 's implementation depends on this. But you are right, this won't be a spec violation.

I may keep this issue open if this is still useful or worth sharing here. But if this is not a right place, just closing now is fine.

Thanks!

@ojanvafai
Copy link
Member

It fits under the broader category of "change from long-standing behavior that web developers might be affected by." Whether specs allow it or not doesn't seem like the critical distinction to me.

But, I agree this is borderline.

@domenic
Copy link
Collaborator

domenic commented Apr 18, 2022

As Boris says, this is already allowed by specs. Similar to #59 (comment), we can work on speccing this if we have some evidence sites have started to depend on it, but in the meantime we'll lean on the fact that Fetch does not mandate any particular connection speed, and close this issue.

@domenic domenic closed this as completed Apr 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants