| commit | 3ba0480e95aac446e3a0f54bdaef9168469f0749 | |
| author | dalecurtis <dalecurtis@chromium.org> | |
| Tue, 9 Sep 2014 20:21:30 +0000 (9 13:21 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Tue, 9 Sep 2014 20:30:37 +0000 (9 20:30 +0000) | ||
| tree | aaea141e6111769160a0061349c98ffd6e8b8310 | treesnapshot (tar.gz zip) |
| parent | 18533307aef4422946f1f242d6aced1b7aea870b | commitdiff |
Reorder AudioMessageFilter shutdown. Remove spammy DCHECK.
Fixes NULL pointer dereference when the RenderThreadImpl is
destructed but outstanding HTMLMediaElement objects have not
yet been garbage collected.
Reordering will keep audio streams alive slightly longer, but
that should be of no consequence since they would still respond
to graceful shutdown attempts by any living AudioRendererImpls.
BUG=408345
TEST=none
Review URL: https://codereview.chromium.org/517793003
Cr-Commit-Position: refs/heads/master@{#293999}
Fixes NULL pointer dereference when the RenderThreadImpl is
destructed but outstanding HTMLMediaElement objects have not
yet been garbage collected.
Reordering will keep audio streams alive slightly longer, but
that should be of no consequence since they would still respond
to graceful shutdown attempts by any living AudioRendererImpls.
BUG=408345
TEST=none
Review URL: https://codereview.chromium.org/517793003
Cr-Commit-Position: refs/heads/master@{#293999}
| content/renderer/media/audio_renderer_mixer_manager.cc | diffblobblamehistory | |
| content/renderer/render_thread_impl.cc | diffblobblamehistory |