1. 29 Aug, 2014 2 commits
  2. 27 Aug, 2014 2 commits
    • James Zern's avatar
      vp9: fix crash in inline loopfilter w/corrupt file · cde790c3
      James Zern authored
      attempting to decode a frame after the previous frame failed has the
      potential of interrupting an earlier loop filter task
      
      Change-Id: I6f2b1ddcdf5b89c3e2ee8caf5289dada2a087d66
      cde790c3
    • James Zern's avatar
      vp9: fix crash in mt loopfilter w/corrupt file · 4f27202d
      James Zern authored
      if the first frame was corrupt and loop filter not called, the next call
      would assume the necessary allocations had been done and segfault when
      accessing a NULL pointer
      
      Change-Id: Ib6ef505e5c594e6f0fe65ab0700172bcf06b92a6
      4f27202d
  3. 21 Aug, 2014 1 commit
  4. 18 Aug, 2014 1 commit
    • Minghai Shang's avatar
      [spatial svc]Add a few different encode frame tests. · e1b5d248
      Minghai Shang authored
      1. Clean the code for encode frame tests
      2. Add encode w/ and w/o alt reference frame test
      3. Add encode SNR layers test
      4. Add encode multiple layers but decode partial layers test
      
      Change-Id: Ibd2c9bc02525db584a6f931a98405f2d851b3cd6
      e1b5d248
  5. 14 Aug, 2014 2 commits
    • James Zern's avatar
      vp9_copy_reference_dec: check ref frame index before use · a07d09ff
      James Zern authored
      use get_ref_frame() to ensure a valid frame is retrieved if one exists
      
      Change-Id: I4b116d928e643ee62a991babd4b974037b6b3830
      a07d09ff
    • Adrian Grange's avatar
      Fix bug 837: realloc mode info buffers on resize · 4e30565a
      Adrian Grange authored
      The test to determine if the mode info buffers need
      to be resized when the frame size changes was
      incorrect, as per bug 837.
      
      By storing the size of the allocated data structure,
      a simple test determines whether to allocate more
      memory when the frame size changes.
      
      Change-Id: I1544698f2882cf958fc672485614f2f46e9719bd
      4e30565a
  6. 12 Aug, 2014 1 commit
  7. 08 Aug, 2014 2 commits
  8. 07 Aug, 2014 1 commit
    • Deb Mukherjee's avatar
      Changes hdr for profiles > 1 for intraonly frames · 09bf1d61
      Deb Mukherjee authored
      Specifies the bit-depth, color sampling and colorspace
      for intra only frames for profiles > 0
      
      Also adds checks to ensure that profile 1 and 3 are
      exclusively used for non 420 streams.
      
      Change-Id: Icfb15fa1acccbce8f757c78fa8a2f60591360745
      09bf1d61
  9. 28 Jul, 2014 1 commit
    • Jim Bankoski's avatar
      Fix reference frame size restrictions. · 899585eb
      Jim Bankoski authored
      The issue was introduced by commit g9f37d149 with adding explicit
      restrictions on reference-frame scale factors. The restriction
      is checked against aligned-by-8 frame dimensions, not against
      original ones. So, for example, frame of 35×35 actually can refer
      to frame of 70×70, but the new check won't allow this. It will
      compare 35 vs 72 (not 70), so 2x downscale limit will be exceeded.
      
      Change-Id: Ic663693034440f64ac8312cbff9e1e773a921060
      899585eb
  10. 24 Jul, 2014 2 commits
    • Yaowu Xu's avatar
      Changed validation of reference frame size · 9261e1aa
      Yaowu Xu authored
      A previous change, https://gerrit.chromium.org/gerrit/#/c/70632,
      introduced a size validation for reference frames to insuare the
      input stream is a valid VP9 stream. However, the logic requiring
      all reference frames have valid size turned out to be too strict.
      
      In this commit, we modify the validation to require one of the
      reference frame has valid dimension. In addition, the decoder
      reports error whenever it detects the use of reference frame
      with invalid scalig ratio.
      
      Change-Id: If8efc312244087556cfe00f1fcbdff811268ebad
      9261e1aa
    • Adrian Grange's avatar
      Fix allocation of context buffers on frame resize · 423e8a97
      Adrian Grange authored
      The patch:
      https://gerrit.chromium.org/gerrit/#/c/70814/
      changed the test that determined whether the context
      frame buffers needed to be reallocated or not.
      
      The code checked for a change in total frame area
      to signal the need to reallocate context buffers.
      However, the above_context buffer needs to be
      resized i:xf only the width of the frame has increased.
      
      Change-Id: Ib89d75651af252908144cf662578d84f16cf30e6
      423e8a97
  11. 21 Jul, 2014 2 commits
  12. 18 Jul, 2014 2 commits
  13. 17 Jul, 2014 2 commits
    • Adrian Grange's avatar
      Modified frame buffer handling · f68aaa38
      Adrian Grange authored
      This patch is the first step toward simplifying the
      frame buffer handling.
      
      The final goal is to have a common frame buffer handling
      framework for both encoder and decoder that incorporates
      the existing ability to use externally allocated memory.
      
      Change-Id: I2c378a4f54a39908915f46c4260e17a080db7ff1
      f68aaa38
    • Jim Bankoski's avatar
      allow config options to limit max size of decode · 943e4327
      Jim Bankoski authored
      This is a practical concern to allow us to fail in a decoder instance
      if the size of a file is bigger than we can reasonably handle.
      
      Change-Id: I0446b5502b1f8a48408107648ff2a8d187dca393
      943e4327
  14. 15 Jul, 2014 1 commit
  15. 11 Jul, 2014 2 commits
  16. 10 Jul, 2014 1 commit
  17. 09 Jul, 2014 1 commit
  18. 08 Jul, 2014 1 commit
  19. 07 Jul, 2014 1 commit
    • hkuang's avatar
      Move vp9_thread.* to common. · 337e8015
      hkuang authored
      Prepare for frame parallel decoding, the reference count buffers
      need to be protected by mutex. Move vp9_thread.* to common
      folder so that those buffers could use cross-platform mutex
      from vp9_thread.*.
      
      Change-Id: I541277cf15eefed6641555944f67f4a0bcdc8154
      337e8015
  20. 01 Jul, 2014 3 commits
    • Jim Bankoski's avatar
      validate uv block size when reading partition · abf0df08
      Jim Bankoski authored
      Change-Id: I74fc5f1a7bab3128cdd49441b83ec3a25aee65ca
      abf0df08
    • James Zern's avatar
      update vp9_thread.[hc] · e656f44c
      James Zern authored
      pull the latest from WebP, which adds a worker interface abstraction
      allowing an application to override init/reset/sync/launch/execute/end
      
      this has the side effect of removing a harmless, but annoying, TSan
      warning.
      
      Original source:
       http://git.chromium.org/webm/libwebp.git
        100644 blob 08ad4e1fecba302bf1247645e84a7d2779956bc3 src/utils/thread.c
        100644 blob 7bd451b124ae3b81596abfbcc823e3cb129d3a38 src/utils/thread.h
      
      Local modifications:
       - s/WebP/VP9/g
       - camelcase functions -> lower with _'s
       - associate '*' with the variable, not the type
      
      Change-Id: I875ac5a74ed873cbcb19a3a100b5e0ca6fcd9aed
      e656f44c
    • hkuang's avatar
      Revert "Fix a bug in VP9Worker which leads to unit test hang." · 1480ba6f
      hkuang authored
      The caller should reset the state instead of letting worker
      to reset.
      
      This reverts commit 34b2ce15.
      
      Change-Id: Idb546ea6386cffc44e98dee772900d21ab79710f
      1480ba6f
  21. 30 Jun, 2014 2 commits
  22. 29 Jun, 2014 1 commit
  23. 28 Jun, 2014 1 commit
  24. 27 Jun, 2014 1 commit
    • Jim Bankoski's avatar
      Better validation of invalid files · 9f37d149
      Jim Bankoski authored
      This patch checks that a decoder never tries to reference frame that's
      outside the range of 2x to 1/16th the size of this frame.  Any attempt
      to do so causes a failure.
      
      Change-Id: I5c98fa7bb95ac4f29146f29dd92b62fe96164e4c
      9f37d149
  25. 25 Jun, 2014 2 commits
    • hkuang's avatar
      Revert "Revert 3 patches from Hangyu to get Chrome to build:" · b3363561
      hkuang authored
      This patch reverts the previous revert from Jim and also add a
      variable user_priv in the FrameWorker to save the user_priv
      passed from the application. In the decoder_get_frame function,
      the user_priv will be binded with the img. This change is needed
      or it will fail the unit test added here:
      https://gerrit.chromium.org/gerrit/#/c/70610/
      
      This reverts commit 9be46e45.
      
      Change-Id: I376d9a12ee196faffdf3c792b59e6137c56132c1
      b3363561
    • James Zern's avatar
      vp9: check tile column count · b2b07755
      James Zern authored
      the max is 6. there are assumptions throughout the decode regarding
      this; fixes a crash with a fuzzed bitstream
      
      $ zzuf -s 5861 -r 0.01:0.05 -b 6- \
        < vp90-2-00-quantizer-00.webm.ivf \
        | dd of=invalid-vp90-2-00-quantizer-00.webm.ivf.s5861_r01-05_b6-.ivf \
          bs=1 count=81883
      
      Change-Id: I6af41bb34252e88bc156a4c27c80d505d45f5642
      b2b07755
  26. 21 Jun, 2014 1 commit
    • Jim Bankoski's avatar
      Revert 3 patches from Hangyu to get Chrome to build: · 9be46e45
      Jim Bankoski authored
      Avoids failures:
      MSE_ClearKey/EncryptedMediaTest.Playback_VP9Video_WebM/0
      MSE_ClearKey_Prefixed/EncryptedMediaTest.Playback_VP9Video_WebM/0
      MSE_ExternalClearKey_Prefixed/EncryptedMediaTest.Playback_VP9Video_WebM/0
      MSE_ExternalClearKey/EncryptedMediaTest.Playback_VP9Video_WebM/0
      MSE_ExternalClearKeyDecryptOnly/EncryptedMediaTest.Playback_VP9Video_WebM/0
      MSE_ExternalClearKeyDecryptOnly_Prefixed/EncryptedMediaTest.Playback_VP9Video_WebM/0
      SRC_ExternalClearKey/EncryptedMediaTest.Playback_VP9Video_WebM/0
      SRC_ExternalClearKey_Prefixed/EncryptedMediaTest.Playback_VP9Video_WebM/0
      SRC_ClearKey_Prefixed/EncryptedMediaTest.Playback_VP9Video_WebM/0
      
      Patches are
      This reverts commit 9bc04085
      This reverts commit 6f5aba06
      This reverts commit 9bc04085
      
      I1f250441	Revert "Refactor the vp9_get_frame code for frame parallel."
      Ibfdddce5	Revert "Delay decreasing reference count in frame-parallel decoding."
      I00ce6771	Revert "Introduce FrameWorker for decoding."
      
      Need better testing in libvpx for these commits
      
      Change-Id: Ifa1f279b0cabf4b47c051ec26018f9301c1e130e
      9be46e45
  27. 20 Jun, 2014 1 commit
    • hkuang's avatar
      Introduce FrameWorker for decoding. · 9bc04085
      hkuang authored
      When decoding in serial mode, there will be only
      one FrameWorker doing decoding. When decoding in
      parallel mode, there will be several FrameWorkers
      doing decoding in parallel.
      
      Change-Id: If53fc5c49c7a0bf5e773f1ce7008b8a62fdae257
      9bc04085