commit 7a9bb78320f29b7f4da2f81c4db6cc955ac65247
author Sarah Parker <sarahparker@google.com> Thu Oct 11 14:52:42 2018 -0700
committer Sarah Parker <sarahparker@google.com> Fri Oct 12 17:23:47 2018 +0000
tree 654e7dd877108ed3fb67962b7420129a1786c189
parent 0b3531a7b7c4feb119702bb9372669341fdcf5cb

Properly handle show existing internal ALTREF

The reference buffer virtual indices are being incorrectly
shifted on the encoder side when encoding
an error resilient ALTREF2 that is also a show_existing_frame.
This was causing a mismatch and noticeable artifacts.

BUG=aomedia:2159

Change-Id: I47c0fa8d6e8267aeb7f984e12aa3fd5889d14eb1

av1/encoder/encoder.c

diff --git a/av1/encoder/encoder.c b/av1/encoder/encoder.c
index 53c60a7..b9dbed9 100644
--- a/av1/encoder/encoder.c
+++ b/av1/encoder/encoder.c
@@ -3544,7 +3544,7 @@
 
     // TODO(zoeliu): Do we need to copy cpi->interp_filter_selected[0] over to
     // cpi->interp_filter_selected[GOLDEN_FRAME]?
-  } else if (cpi->rc.is_src_frame_ext_arf && cm->show_existing_frame) {
+  } else if (cpi->rc.is_src_frame_ext_arf && encode_show_existing_frame(cm)) {
 #if CONFIG_DEBUG
     const GF_GROUP *const gf_group = &cpi->twopass.gf_group;
     assert(gf_group->update_type[gf_group->index] == INTNL_OVERLAY_UPDATE);