• servolk's avatar
    Fix processing of multiple stream status changes by renderer · bff54a24
    servolk authored
    Typically renderer needs to do Flush + StartPlaying in response to
    stream status change (i.e. when stream gets enabled or disabled). But
    since Flush is an async operation we might get another status change
    of the same stream, while the renderer is still handling the previous
    one. In the past renderer has simply ignored status changes while
    another status change was processed. But that was problematic, since
    we couldn't guarantee that the renderer status will be correct after
    stream has been enabled/disabled very quickly a few times. This CL
    fixes that issue by postponing status changes handling if necessary,
    instead of dropping those notifications.
    
    BUG=678031
    
    Review-Url: https://codereview.chromium.org/2605473002
    Cr-Commit-Position: refs/heads/master@{#442762}
    bff54a24
renderer_impl.cc 27.4 KB