bcarrella/cef
1
0
Fork 0
mirror of https://bitbucket.org/chromiumembedded/cef synced 2025-06-05 21:39:12 +02:00
Code Issues Packages Projects Releases Wiki Activity

alloy: Fix shutdown assert if OneShotTimer is deleted on the wrong thread

Browse Source
This commit is contained in:
Marshall Greenblatt
2022-01-25 16:15:23 -05:00
parent f07ce7b357
commit eb2c682768
2 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
libcef/browser/alloy/alloy_browser_host_impl.cc
View File

@ -765,7 +765,10 @@ void AlloyBrowserHostImpl::DestroyBrowser() {
menu_manager_.reset(nullptr);
// Delete the audio capturer
recently_audible_timer_.Stop();
if (recently_audible_timer_) {
recently_audible_timer_->Stop();
recently_audible_timer_.reset();
}
audio_capturer_.reset(nullptr);
CefBrowserHostBase::DestroyBrowser();
@ -1501,14 +1504,19 @@ void AlloyBrowserHostImpl::DidFinishNavigation(
void AlloyBrowserHostImpl::OnAudioStateChanged(bool audible) {
if (audible) {
recently_audible_timer_.Stop();
if (recently_audible_timer_)
recently_audible_timer_->Stop();
StartAudioCapturer();
} else if (audio_capturer_) {
if (!recently_audible_timer_)
recently_audible_timer_ = std::make_unique<base::OneShotTimer>();
// If you have a media playing that has a short quiet moment, web_contents
// will immediately switch to non-audible state. We don't want to stop
// audio stream so quickly, let's give the stream some time to resume
// playing.
recently_audible_timer_.Start(
recently_audible_timer_->Start(
FROM_HERE, kRecentlyAudibleTimeout,
base::BindOnce(&AlloyBrowserHostImpl::OnRecentlyAudibleTimerFired,
this));