• Jeffrey Kardatzke's avatar
    Revert "media/gpu/vaapiWrapper: Add WARN_UNUSED_RESULT to critical functions" · 241d3ebe
    Jeffrey Kardatzke authored
    This reverts commit 8a1b573d.
    
    Reason for revert: Incorrect condition on return value from CreateContext in ApplyResolutionChange.
    
    Original change's description:
    > media/gpu/vaapiWrapper: Add WARN_UNUSED_RESULT to critical functions
    >
    > This adds WARN_UNUSED_RESULT to VaapiWrapper functions whose
    > result should have the critical impact enough to regard its
    > failure as video processing fatal.
    > VaapiVideoDecoder actually doesn't handle the failure of
    > VaapiWrapper::CreateContext. This CL also fixes it.
    >
    > Bug: b:171536540
    > Test: Build media_unittest and vaapi_unittest
    > Change-Id: Iba6924849bfe8627fd328c3541c7a2c5cb0f0371
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2493742
    > Commit-Queue: Hirokazu Honda <hiroh@chromium.org>
    > Reviewed-by: Andres Calderon Jaramillo <andrescj@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#820164}
    
    TBR=hiroh@chromium.org,andrescj@chromium.org
    
    Change-Id: If3f3255475735f81786f64d686524947ec042a65
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: b:171536540
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2495980Reviewed-by: default avatarJeffrey Kardatzke <jkardatzke@google.com>
    Commit-Queue: Jeffrey Kardatzke <jkardatzke@google.com>
    Cr-Commit-Position: refs/heads/master@{#820424}
    241d3ebe
vaapi_video_decoder.cc 25.3 KB