Is Electron workaround still necessary?

[christianbundy]

There's an Electron workaround in index.js that was added in July 2016, which was before Electron 1.3 was released (Chrome 52, Node 6). Four years later Electron is on version 10.11, and I've tested all the way back to Electron 7 and I can't find the problem. Does anyone have more context on what platform the problem was happening on? The only context I have is 50dbe3b, but that doesn't tell us much.

If the underlying problem is gone, I'd love to remove the workaround, but I thought I should make an issue before making a PR.

[arj03]

I don't see why that should be there still. Lets Get It Removed :-)

[austinfrey]

@christianbundy I did some tests going all the way back to electron v1. Things break around v3 but not due to the detached methods I don't think.