KEMBAR78
Add Node.js integration to WebWorker by zcbenz · Pull Request #8852 · electron/electron · GitHub
Skip to content

Conversation

@zcbenz
Copy link
Contributor

@zcbenz zcbenz commented Mar 7, 2017

  • Add node integration to WebWorker
  • Fix thread unsafe code in Electron
  • Add an option in webPreferences
  • Tests
  • Docs

@@ -0,0 +1 @@
unload No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this file used in any specs?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was mistakenly commited.

@zcbenz zcbenz merged commit e539129 into master Mar 22, 2017
@zcbenz zcbenz deleted the web-worker-hook branch March 22, 2017 16:01
jviotti pushed a commit to balena-io/etcher that referenced this pull request Apr 25, 2017
Some changes that we're particularly interested in:

- electron/electron#8590
- electron/electron#8852
- electron/electron#7631

Note that the `electron-prebuilt` packaged has been renamed to
`electron`.

Change-Type: patch
Changelog-Entry: Upgrade Electron to v1.6.6.
Signed-off-by: Juan Cruz Viotti <jviotti@openmailbox.org>
jviotti pushed a commit to balena-io/etcher that referenced this pull request Apr 25, 2017
Some changes that we're particularly interested in:

- electron/electron#8590
- electron/electron#8852
- electron/electron#7631

Note that the `electron-prebuilt` packaged has been renamed to
`electron`.

Change-Type: patch
Changelog-Entry: Upgrade Electron to v1.6.6.
Signed-off-by: Juan Cruz Viotti <jviotti@openmailbox.org>
@mubaidr
Copy link

mubaidr commented May 3, 2018

Requireing a third party module from a worker scripts fails with an error message 'Cannot find blahblah module'. Happens to all installed modules. (FS, Path loads correctly)

Is this intended behavior?

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.

4 participants