Crash on close about agi HOT 6 CLOSED

Thank you for the bug report. I'm failing to reproduce the issue. When you say you ignored the dialog, do you mean you hit escape, closed it by clicking the red close button, or just clicked on the "Get started" button? How did you close the App? Cmd-q, the menu, the red close button? I've tried all these and couldn't replicate.

Although, looking at the stack, there could be some kind of race going on, which would explain why I can't replicate. Does this keep happening for you all the time?

from agi.

I have reproduced this 2 times, first closed the ADB dialog with close button then closed the main window again with the close button. The second time specified the adb location then get "Get started", then closed the app with top panel options "Quit the android gpu inspector". I tried these steps about 6 times got this crash 2 times. I looked at the code and I thought about race condition, too. I will try to find time to investigate this crash and will provide a fix for this :)

from agi.

HI @artyomdangizyan! Did you have some time to investigate this issue? Could you please provide the logs so we can take a look here? Thank you very much!

from agi.

Hi @pau-baiget I attached the log files. I have already reproduced this crash about 10-15 times but I still can't reproduce this from the first try. I tried to build the project to debug the crash, but building this project in my environment is too painful (the bazel version is mismatched, some externals libs are failing because of some warnings, etc..) But I have looked at the sources and my hypothesis is that this crash is a result fo some race condition, a thread is asynchronously closing the application and calling destroy on the window after which the application member in windows is becoming null and the other thread is looping and it has already successfully passed the validation that window is not destroyed. But unfortunately, I couldn't find any asynchrony all to the MainWindow close method. Anyway, do you need any more info from me for the investigation of this crash?
gapic.log
gapis.log

from agi.

@artyomdangizyan , apologies for the lack of follow-up!! We have published loads of fixes and updated UI libraries (PR) so this issue might not be reproducible anymore. Can you please try the latest dev release?

Thanks! We'll monitor this bug more closely so it doesn't get stuck again!

from agi.

Closing this issue as we're not able to reproduce this issue anymore. @artyomdangizyan please add a comment and we'll reopen if necessary, thanks!

from agi.