Crash at CefSharp.Internals.ClientAdapter.OnProcessMessageReceived

[eadan-y]

Hi
This happens on version 86 as well as the latest (90)
added the libcef PDB.
This is the stack:

System.NullReferenceException: Object reference not set to an instance of an object. at
CefSharp.Internals.ClientAdapter.OnProcessMessageReceived(ClientAdapter* , scoped_refptr* browser, scoped_refptr* frame, cef_process_id_t source_process, scoped_refptr* message) in c:\projects\cefsharp\cefsharp.core.runtime\internals\clientadapter.cpp:line 1356

It happens after I hide a WPF window containing the browser.
I am communicating between my C# application and the browser page (mostly browser calls C# methods).

I think this happens when the browser gets disposed while there is a call from javascript to C# (via JavascriptObjectRepository) still running.

[amaitland]

This is the stack:

Which version corresponds to the stack trace?

[amaitland]

Also, the speculative changes does seem to help (although I tested with 91.1.10-CI4048).

Can you clarify help in this context?` Are you still seeing the problem just less often or does the problem appear to be resolved?

Question - is there any chance we could use your help as a consultant?

If the problem still reproduces after the changes I made in commit 2e87516 then I'm happy to discuss that option.

Generally speaking I would prefer that people/organisations fund my efforts via GitHub Sponsors

Is there any chance to create a hotfix for version 90 containing that fix?

I'll likely release a new version at the end of next week. If your company would like to sponsor me (or if someone from your company is already sponsoring) then I'd be happy to prioritise your request and release a new version ASAP.

[eadan-y]

The problem seems to be resolved in 91.1.10-CI4048.
I have now compiled my own version from 90 branch (that contains the new commit done after the tag) to a locally hosted nuget package to see if it fixes the issue as well, as we don't want to use the CI version.
Will the new version (~next week) be of 91 or of 90?

As for the sponsoring - We would love to support you while maintaining this project. We're running it within the organization.

[amaitland]

Will the new version (~next week) be of 91 or of 90?

I'll release an updated version of 90 including update to Chromium 90.0.4430.212. I'll release a 91-pre release a few days after.

[amaitland]

Version 90.6.70 is now available on Nuget.org.

[raziele]

Hi @amaitland, nice to e-meet. I'm @eadan-y's manager.
I'd like to thank you for your donation and work. We started supporting the project via the sponsership channel here on github (I hope you can see it already).

We'd love to discuss expanding this support for the project as well as exploring additional partnership options we've been thinking of.

If you're interested, please reach out to me at raziel.einhorn(at)walkme.com and we'll set up a call.