cef icon indicating copy to clipboard operation
cef copied to clipboard

Audio handler broken with a crash on shutdown

Open pkviet opened this issue 1 year ago • 2 comments

Describe the bug Cef crashes on browser close due to Audio Handler. Here's the stack trace: devenv_2024-04-27_18-21-36

There an error message in OnAudioStreamError which is: Socket closed unexpectedly

The ceftests also fail for AudioOutputTest.AudioCloseBrowserTest with the same error message:

[ RUN      ] AudioOutputTest.AudioCloseBrowserTest
[0427/092425.012:WARNING:audio_output_unittest.cc(937)] OnAudioStreamError: message = Socket closed unexpectedly.
..\..\cef\tests\ceftests\audio_output_unittest.cc(948): error: Value of: got_on_audio_stream_stopped_
  Actual: false
Expected: true
Stack trace:
        `anonymous namespace'::AudioTestHandler::DestroyTest [0x00007FF65CCDEB3A+1018]
        `anonymous namespace'::AudioTestHandler::OnAudioStreamError [0x00007FF65CCE088D+253]
        CefAudioHandlerCppToC::CefAudioHandlerCppToC [0x00007FF65D2DAF90+2016]
        CefAudioHandlerCToCpp::OnAudioStreamError [0x00007FF9657EF78E+238]
        CefAudioCapturer::OnCaptureError [0x00007FF96586E8FB+203]
        media::AudioInputDevice::AudioThreadCallback::OnSocketError [0x00007FF969693C38+72]

..\..\cef\tests\ceftests\audio_output_unittest.cc(949): error: Value of: got_on_audio_stream_error_
  Actual: true
Expected: false
Stack trace:
        `anonymous namespace'::AudioTestHandler::DestroyTest [0x00007FF65CCDEC44+1284]
        `anonymous namespace'::AudioTestHandler::OnAudioStreamError [0x00007FF65CCE088D+253]
        CefAudioHandlerCppToC::CefAudioHandlerCppToC [0x00007FF65D2DAF90+2016]
        CefAudioHandlerCToCpp::OnAudioStreamError [0x00007FF9657EF78E+238]
        CefAudioCapturer::OnCaptureError [0x00007FF96586E8FB+203]
        media::AudioInputDevice::AudioThreadCallback::OnSocketError [0x00007FF969693C38+72]

To Reproduce Steps to reproduce the behavior:

  1. Just run ceftests --gtest_filter=AudioOutputTest

Expected behavior No crash on shutdown.

Screenshots

Versions (please complete the following information):

  • CEF Version: master (6367)

pkviet avatar Apr 27 '24 16:04 pkviet

@shagkur fyi

pkviet avatar Apr 27 '24 16:04 pkviet

It looks like it might related to this chromium change https://chromium-review.googlesource.com/c/chromium/src/+/5331283

tytan652 avatar Apr 30 '24 18:04 tytan652