commit a3f71ccff6c514a7396a1a329194e5cda18b6825
author Paul Wilkins <paulwilkins@google.com> Wed Jan 26 12:52:31 2011 +0000
committer Paul Wilkins <paulwilkins@google.com> Wed Jan 26 12:52:31 2011 +0000
tree 6aeba91215162b499eedbc167942fb94cbed49fe
parent 2caa36aa4fce2bb7e3f8e43624cab94b78bc8950

Correction to buffer update for non-viewable frames.

The code previously tested cpi->common.refresh_alt_ref_frame
but there are situations where this flag may be set for viewable frames.

The correct test should be !cm->show_frame.

Change-Id: Ia1a600622992a4a68fe1d38ac23bf6b34b133688

vp8/encoder/onyx_if.c

diff --git a/vp8/encoder/onyx_if.c b/vp8/encoder/onyx_if.c
index 9007853..d60bf8b 100644
--- a/vp8/encoder/onyx_if.c
+++ b/vp8/encoder/onyx_if.c
@@ -4614,7 +4614,8 @@
     }
 
     // Update the buffer level variable.
-    if (cpi->common.refresh_alt_ref_frame)
+    // Non-viewable frames are a special case and are treated as pure overhead.
+    if ( !cm->show_frame )
         cpi->bits_off_target -= cpi->projected_frame_size;
     else
         cpi->bits_off_target += cpi->av_per_frame_bandwidth - cpi->projected_frame_size;