• Max Morin's avatar
    Reland "Fix yet another AudioOutputDevice crash." · e1411c5f
    Max Morin authored
    This is a reland of a7c1c185
    
    Clusterfuzz issues were (hopefully) fixed by
    https://chromium-review.googlesource.com/c/chromium/src/+/962124
    
    Original change's description:
    > Fix yet another AudioOutputDevice crash.
    >
    > Bug is identical to the last :/. No idea why it's much more frequent with mojo,
    > possibly a race is likely to happen when destroying a frame: |callback_| is
    > destroyed due to frame being destroyed, and authorization is failed for the
    > same reason. With the per process message filter, authorization wouldn't fail
    > due to the frame being destructed.
    >
    > Bug: 819277
    > Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
    > Change-Id: Ic17dc96d2c83a08732b43094bac2f2c3fa7035ad
    > Reviewed-on: https://chromium-review.googlesource.com/951774
    > Commit-Queue: Max Morin <maxmorin@chromium.org>
    > Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#541801}
    
    Bug: 819277
    Cq-Include-Trybots: luci.chromium.try:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
    Change-Id: Ia6aca4d8f93d1534a364abe9d467829bde928de7
    Reviewed-on: https://chromium-review.googlesource.com/962721
    Commit-Queue: Max Morin <maxmorin@chromium.org>
    Reviewed-by: default avatarOlga Sharonova <olka@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#543352}
    e1411c5f
audio_output_device.h 9.09 KB