tree 775eb2fdf3f2b144f24f2bf21143e3b48cfaa4cb
parent 042af92489c7b8910b21c365984255d8cfc1baff
author David Turner <david.turner@argondesign.com> 1548349224 +0000
committer David Turner <david.turner@argondesign.com> 1549653980 +0000

Simplify buffer update decision code

* Remove check_src_altref() which was pointless in pass 2 (the
  parameters are reset by av1_rc_get_second_pass_params())
* Decide on FRAME_UPDATE_TYPE and only call
  av1_configure_buffer_updates() once
* Remove direct reads and writes to the refresh_*_frame flags from
  av1_rc_get_one_pass_{cbr,vbr}_params().
* Remove overrides of rc.is_src_frame_alt_ref and rc.is_src_frame_ext_arf
  from check_show_existing_frame(): these will be handled by
  av1_configure_buffer_updates()
* Move overriding refresh flags from set_ext_overrides() to
  av1_configure_buffer_updates()
* Move refresh_*_frame flag overrides from encode_frame_to_data_rate()
  and setup_frame()

This forms part of wider restructuring and refactoring in order to
achieve a clean API separation at the entry to the low-level encoder and
to make the high-level encoding strategy easier to modify.

BUG=aomedia:2244

Change-Id: Idf1e75ac59319e16cbefbba56038a21042666d33