Skip to content

fix: loading dedicated/shared worker scripts over custom protocol#24750

Merged
jkleinsc merged 4 commits into
9-x-yfrom
trop/9-x-y-bp-fix-loading-dedicated-shared-worker-scripts-over-custom-protocol-1595900935098
Aug 17, 2020
Merged

fix: loading dedicated/shared worker scripts over custom protocol#24750
jkleinsc merged 4 commits into
9-x-yfrom
trop/9-x-y-bp-fix-loading-dedicated-shared-worker-scripts-over-custom-protocol-1595900935098

Conversation

@trop

@trop trop Bot commented Jul 28, 2020

Copy link
Copy Markdown
Contributor

Backport of #20625

See that PR for details.

Notes:

  • fix loading shared worker scripts over custom protocol
  • fix crash when loading worker scripts with nodeIntegration enabled

@trop
trop Bot requested a review from a team as a code owner July 28, 2020 01:49
@electron-cation electron-cation Bot added the new-pr 🌱 PR opened recently label Jul 28, 2020
@trop
trop Bot requested a review from deepak1556 July 28, 2020 01:49
@trop trop Bot added 9-x-y backport This is a backport PR labels Jul 28, 2020
@electron-cation electron-cation Bot removed the new-pr 🌱 PR opened recently label Jul 28, 2020
[worker_global_scope.cc(111)] Check failed: url_.IsValid().
@codebytere
codebytere force-pushed the trop/9-x-y-bp-fix-loading-dedicated-shared-worker-scripts-over-custom-protocol-1595900935098 branch from 011483d to e0176da Compare August 6, 2020 15:27

@jkleinsc jkleinsc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like the new test is failing on WOA

@jkleinsc
jkleinsc merged commit e2c4e34 into 9-x-y Aug 17, 2020
@release-clerk

release-clerk Bot commented Aug 17, 2020

Copy link
Copy Markdown

Release Notes Persisted

  • fix loading shared worker scripts over custom protocol
  • fix crash when loading worker scripts with nodeIntegration enabled

@jkleinsc
jkleinsc deleted the trop/9-x-y-bp-fix-loading-dedicated-shared-worker-scripts-over-custom-protocol-1595900935098 branch August 17, 2020 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

9-x-y backport This is a backport PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants