search

nwjs / nw.js

Call all Node.js modules directly from DOM/WebWorker and enable a new way of writing applications with all Web technologies.
https://nwjs.io
MIT License
40.35k stars 3.88k forks source link

NWjs app crashing on closing a window it creates with dev tools opened #7051

Open JeFawk opened 5 years ago

JeFawk commented 5 years ago

======== TEMPLATE BUG FORM ========

NWJS Version : 0.38.2 Operating System : Windows 10

Expected behavior

The app to not crash no matter if the new window created has dev tools opened or not.

Actual behavior

Once the app creates a new window, if the window has dev tools opened and the said window is closed the app will crash. Sometimes the debug log updates, sometimes it doesn't.

How to reproduce

The app in question is Construct 3. You can download it from here: https://downloads.scirra.com/c3-desktop/win64/construct3-win64-c74-stable.zip It was tested internally with 0.38.2

  1. Download the app, open it, close the Buy dialogue
  2. In the starting screen open the premade project called Demonoir image
  3. Once fully loaded press F4 to start previewing the game
  4. Once the new window appeared and the menu of the game shows up, open the dev tools for the new window that was created, as if you're debugging the game
  5. Close the game's window
  6. Notice how the whole app crashed (reproduced on 3 separate machines)

This was my latest debug log:

[0512/193023.309:WARNING:minidump_to_upload_parameters.cc(67)] duplicate annotation name metrics_client_id, discarding value 56873513dff244cb87afde70fd200f39
[0512/193023.316:ERROR:http_transport_win.cc(178)] WinHttpCrackUrl: The operation completed successfully. (0x0)
[0512/194216.353:WARNING:minidump_to_upload_parameters.cc(67)] duplicate annotation name metrics_client_id, discarding value 56873513dff244cb87afde70fd200f39
[0512/194216.358:ERROR:http_transport_win.cc(178)] WinHttpCrackUrl: The operation completed successfully. (0x0)
[0512/194354.046:WARNING:minidump_to_upload_parameters.cc(67)] duplicate annotation name metrics_client_id, discarding value 56873513dff244cb87afde70fd200f39
[0512/194354.051:ERROR:http_transport_win.cc(178)] WinHttpCrackUrl: The operation completed successfully. (0x0)
[0512/195024.167:WARNING:minidump_to_upload_parameters.cc(67)] duplicate annotation name metrics_client_id, discarding value 56873513dff244cb87afde70fd200f39
[0512/195024.173:ERROR:http_transport_win.cc(178)] WinHttpCrackUrl: The operation completed successfully. (0x0)
[0512/220520.989:WARNING:minidump_to_upload_parameters.cc(67)] duplicate annotation name metrics_client_id, discarding value 56873513dff244cb87afde70fd200f39
[0512/220520.994:ERROR:http_transport_win.cc(178)] WinHttpCrackUrl: The operation completed successfully. (0x0)
[0512/220700.213:WARNING:minidump_to_upload_parameters.cc(67)] duplicate annotation name metrics_client_id, discarding value 56873513dff244cb87afde70fd200f39
[0512/220700.220:ERROR:http_transport_win.cc(178)] WinHttpCrackUrl: The operation completed successfully. (0x0)
ffanny commented 5 years ago

I can reproduce this using a simple demo with nwjs-sdk-win-v0.38.1&v0.37.3. But it works well with v0.36.5.

mamged commented 5 years ago

@ffanny +1