Skip to content

Configurable resource prefetching #70

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

camillobruni
Copy link
Contributor

This addresses issue #58 and is useful for local profiling.

If the prefetchResources is disabled, we now no longer preload them eagerly, but rather use the live URL and either script-inject it or use it directly with workers.
This means we can easily get the original source positions for profiling.

  • --no-prefetch command ilne support
  • &prefetchResources=0 URLParam support

Drive-by-fix:

  • Move FileLoader out of the IIFE

--no-prefetch will directly use `load(...)` instead of
just injecting the script sources which makes it easier to navigate
and investigate profiles.

- Support more complex command line args and flags
- Add --help to cli.js

Bug: 411303884
Change-Id: I43a032342a6da42bfb61ebd9415e62f9d9597ecc
Reviewed-on: https://chromium-review.googlesource.com/c/external/github.com/WebKit/JetStream/+/6468518
Reviewed-by: Leszek Swirski <[email protected]>
Reviewed-by: Marja Hölttä <[email protected]>
Copy link

netlify bot commented Apr 23, 2025

Deploy Preview for webkit-jetstream-preview ready!

Name Link
🔨 Latest commit 1f4b223
🔍 Latest deploy log https://app.netlify.com/sites/webkit-jetstream-preview/deploys/68092ff5729592000834b60f
😎 Deploy Preview https://deploy-preview-70--webkit-jetstream-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@camillobruni camillobruni changed the title 2025 04 23 prefetch option Make resource prefetching optional Apr 23, 2025
@camillobruni camillobruni changed the title Make resource prefetching optional Configurable resource prefetching Apr 23, 2025
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

Successfully merging this pull request may close these issues.

1 participant