Skip to content

chore: pass webviewID with initial version message to sw#244910

Merged
deepak1556 merged 1 commit intomainfrom
robo/fix_sw_reuse
Mar 27, 2025
Merged

chore: pass webviewID with initial version message to sw#244910
deepak1556 merged 1 commit intomainfrom
robo/fix_sw_reuse

Conversation

@deepak1556
Copy link
Copy Markdown
Collaborator

With #244144 we pass the webviewID information as part of the sw registration url, this causes a new worker to be registered for every webview. The ID information is only needed for the resource loads from the webview, so we can pass it as part of the initial version message and keep the sw controller unchanged.

@deepak1556 deepak1556 added this to the March 2025 milestone Mar 27, 2025
@deepak1556 deepak1556 requested a review from mjbvz March 27, 2025 19:11
@deepak1556 deepak1556 self-assigned this Mar 27, 2025
@deepak1556 deepak1556 enabled auto-merge (squash) March 27, 2025 19:12
@deepak1556 deepak1556 merged commit 1919326 into main Mar 27, 2025
8 checks passed
@deepak1556 deepak1556 deleted the robo/fix_sw_reuse branch March 27, 2025 20:01
deepak1556 added a commit that referenced this pull request Apr 1, 2025
deepak1556 added a commit that referenced this pull request Apr 1, 2025
deepak1556 added a commit that referenced this pull request Apr 1, 2025
…45234)

Revert "chore: pass webviewID with initial version message to sw (#244910)"

This reverts commit 1919326.
mustard-mh pushed a commit to gitpod-io/openvscode-server that referenced this pull request Apr 13, 2025
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators May 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants