• dalecurtis's avatar
    Log a non-crash dump for 10% of Spitzer pipeline decode failures. · 6e3901e3
    dalecurtis authored
    Spitzer is currently enabled as a 40% experiment on dev (soon on
    beta as well), but we have abnormally high video-only decode
    failure rates, so add some logging to try and track this down.
    
    ~10% of the time, this change will log the 'subresource_url' crash
    key as a non-crash dump when a decode error is encountered on
    Android where Spitzer is enabled.
    
    This will allow us to investigate and triage these crashes. We're
    already logging this key for normal crash dumps during DoLoad().
    
    This is a temporary change which will be removed shortly after
    landing once any issues are identified.
    
    BUG=595076
    TEST=none
    
    Review URL: https://codereview.chromium.org/1808473002
    
    Cr-Commit-Position: refs/heads/master@{#381532}
    6e3901e3
webmediaplayer_impl.cc 50.7 KB