Fix loopfilter delta zero transitions
authorJohn Koleszar <jkoleszar@google.com>
Wed, 29 Sep 2010 17:04:04 +0000 (13:04 -0400)
committerJohn Koleszar <jkoleszar@google.com>
Wed, 29 Sep 2010 17:04:04 +0000 (13:04 -0400)
commitb9be7a464f065c3700f937eea01d9298002b37eb
treef9d9d22690afe087df44733543fe29a68ef66e31
parent7288cdf79dd179d5bbf927db6240e3b9a4da412b
Fix loopfilter delta zero transitions

Loopfilter deltas are initialized to zero on keyframes in the decoder.
The values then persist from the previous frame unless an update bit
is set in the bitstream. This data is not included in the entropy
data saved by the 'refresh entropy' bit in the bitstream, so it is
effectively an additional contextual element beyond the 3 ref-frames
and the entropy data.

The encoder was treating this delta update bit as update-if-nonzero,
meaning that the value would be refreshed even if it hadn't changed,
and more significantly, if the correct value for the delta changed
to zero, the update wouldn't be sent, and the decoder would preserve
the last (presumably non-zero) value.

This patch updates the encoder to send an update only if the value
has changed from the previously transmitted value. It also forces the
value to be transmitted in error resilient mode, to account for lost
context in the event of lost frames.

Change-Id: I56671d5b42965d0166ac226765dbfce3e5301868
vp8/common/blockd.h
vp8/encoder/bitstream.c
vp8/encoder/onyx_if.c