Skip to content
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

NW2: nw.App.crashRenderer() doesn't kill nw like NW1 does #7383

Open
bodo22 opened this issue Feb 19, 2020 · 8 comments
Open

NW2: nw.App.crashRenderer() doesn't kill nw like NW1 does #7383

bodo22 opened this issue Feb 19, 2020 · 8 comments
Labels

Comments

@bodo22
Copy link

bodo22 commented Feb 19, 2020

Via issue #7339

NWJS Version : nw1 vs nw2 behaviour, seemingly independent of nw.js version
Operating System : Ubuntu 16.4 LTS

Expected behavior

With nw1, when testing a renderer crash with nw.App.crashRenderer(), the whole nw process dies.

Actual behavior

With nw2, the nw process is not killed completely.

How to reproduce

  • Start the current nw.js version (0.44.2) with the chromium flag --disable-features=nw2.

  • Execute nw.App.crashRenderer();
    -> nw completely dies

  • Start the current nw.js version (0.44.2) without the chromium flag --disable-features=nw2.

  • Execute nw.App.crashRenderer();
    -> nw doesn't die, but shows an "Oh Snap" error screen

Related (future) thoughts:

There are probably 2 use cases when a nw renderer dies:

  1. don't let the user decide on anything, immediately just kill the whole process.
  2. let the user interact & decide and what to do, e.g. let him write some text and allow him to send debugging logs etc.

Maybe it makes sense to handle crashing differently in the future, maybe based on a manifest file parameter, e.g. if kiosk is set, or to make it more explicit, a new parameter? Another idea would be to just execute some "renderer died" middleware code, to let the developer decide what to do.

@rogerwang
Copy link
Member

This is fixed in git and will be available in the next nightly build.

@gpetrov
Copy link

gpetrov commented Feb 22, 2020

@rogerwang - but now the whole nwjs process just disappears on crash? How can we open a new popup for the user to report an issue? As discussed in #7339 (comment)

@rogerwang
Copy link
Member

This is just to keep the same behavior with NW1. The enhanced crash handling will be implemented later.

@arudnev
Copy link

arudnev commented Feb 25, 2020

@rogerwang, I just checked it with the following build on macOS:
http://dl.nwjs.io/live-build/nw44/20200225-162000/f721a1faf/v0.44.3/
It still shows "Aw, Snap!" with Close button.

@rogerwang rogerwang reopened this Feb 26, 2020
@rogerwang
Copy link
Member

@arudnev thanks. It seems to remain opening when devtools is opened. Will fix it soon.

@bodo22
Copy link
Author

bodo22 commented Sep 7, 2020

Any update on this, can i help somehow?

Same behaviour still happening with current build
nw.js Version: 0.48.1
OS: ubuntu 16 & ubuntu 20

@stale
Copy link

stale bot commented Jan 9, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jan 9, 2022
@bodo22
Copy link
Author

bodo22 commented Jan 10, 2022

Bumping.

Same behaviour still happening with current build
nw.js Version: 0.60.0
tested OSes: ubuntu 16 LTS & ubuntu 20 LTS

@stale stale bot removed the stale label Jan 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants