diff options
author | qinmin <qinmin@chromium.org> | 2014-09-03 14:50:32 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-09-03 22:53:23 +0000 |
commit | 0b08967cc3c1a8e6bc17921201553bb6d8a7bb90 (patch) | |
tree | 0390496c0ff24a9d9255e48bcc83d809db58e646 /net/tools/quic/end_to_end_test.cc | |
parent | a29dd6ccf30bda316f32e17952dc2258103b2801 (diff) | |
download | chromium_src-0b08967cc3c1a8e6bc17921201553bb6d8a7bb90.zip chromium_src-0b08967cc3c1a8e6bc17921201553bb6d8a7bb90.tar.gz chromium_src-0b08967cc3c1a8e6bc17921201553bb6d8a7bb90.tar.bz2 |
Remove OnMediaResourcesReleased callback passed to MediaPlayerAndroid
This callback has the reentrance issue as calling MediaPlayerBridge::Release() will call into function.
The worst case happens when BrowserMediaPlayerManager dtor is called. In that case, this function will be called on each player, while BMPM is in a partially destroyed state.
Since BMPM controls the lifetime of the players, it can call OnMediaResourcesReleased() when it calls Release() on a player.
For MSE case, we also simplified the situation here since we can check MSP::IsSurfaceInUse(). This CL also fixes that call.
BUG=410096
Review URL: https://codereview.chromium.org/534063002
Cr-Commit-Position: refs/heads/master@{#293207}
Diffstat (limited to 'net/tools/quic/end_to_end_test.cc')
0 files changed, 0 insertions, 0 deletions