vp9_systemdependent.h: relocate system includes

avoid wrapping msvc includes with extern "C"; this breaks some visual
studio builds of the (c++) tests.

Change-Id: Ie8062d55d4f4c049f6cd360a36da6a67607df132

Merge "Removing DBG_PRNT_SEGMAP."

Merge "libmkv: constify codec_id."

Merge "Removing unused estimate_cq() function."

Merge "Moving MBGRAPH_{MB, FRAME}_STATS structs to vp9_mbgraph.h."

Merge "Renaming vp9_variance_c.c to vp9_variance.c."

Merge "Replacing int_mv with MV."

Merge "Inlinging get_next_chkpts macro in vp9_mcomp.c."

Merge "Removing SECTIONBITS_OUTPUT."

libmkv: constify codec_id.

This lets the caller pass a string literal for the codec id.

From https://bugzilla.mozilla.org/show_bug.cgi?id=966044

Change-Id: I345bd7a5943f0c33b3fb368d4280100ac5038a3d

Removing DBG_PRNT_SEGMAP.

Change-Id: I71d85e3455545960938e525ae8aa0a667e1db94c

Merge "Ignore missing newline message"

Ignore missing newline message

git diff adds the following line to diffs:
\ No newline at end of file

which interferes with diff.py parsing. diff.py only looks for '+', '-'
and ' ' at the beginning of the line.

Issue seen on https://gerrit.chromium.org/gerrit/68611

Change-Id: I0d7b4485c470e0b409f2c9cddde6c9aceba0152e

Merge "Moving x1 & y1 calculation under if condition."

Merge "With on demand border extension, clamping the MV is not longer needed."

Renaming vp9_variance_c.c to vp9_variance.c.

Change-Id: I7b29cb18ad36d79e1c6329c7de88496059f49db4

With on demand border extension, clamping the MV
is not longer needed.

Change-Id: I40c37ef18c67ab27fc336694dfca3c43a87c47ca

Removing unused estimate_cq() function.

Change-Id: If3a1af1a54f3a3c0d9c966fdc0721981070c7f28

Merge "One-pass rate control fixes and cleanups"

Merge "Remove duplicated code"

One-pass rate control fixes and cleanups

Fixes rate control partially in one-pass non-cbr case to achieve a
bitrate close to the one desired. Previous version was way off at
the high bitrate end.

Also includes several one-pass rate control cleanups and refactoring.

On derfraw300, one-pass encoding is now 19% off from two-pass speed
0 encoding, down from 35%.

Change-Id: I6f0dcdb7f8aa85a7e7cd3a3155d4f9d2a4d2f4f4

Remove duplicated code

Change-Id: I1b7c8165162e835e22cf164cce989c0cebd7ca95

Merge "vp9_filter.h: rename interp_kernel type"

Merge "vp9/encoder: fix function prototypes"

Merge changes I9ebfeb49,I2e2f9bb6

* changes:
  resize_test: use const_iterator where appropriate
  resize_test: silence a warning

Merge changes Ic362b088,I1c65c2aa

* changes:
  vpxenc: find_mismatch/compare_img: normalize types
  vpxenc: cosmetics

Moving x1 & y1 calculation under if condition.

Change-Id: Iae787d491f7cfe24855ef8f2d04e2c6c19350378

Replacing int_mv with MV.

Change-Id: I36b8b419be92171857e77a881da066424ff47af5

Merge "Removing ENC_DEBUG."

Merge "Build fix with config internal stats"

vp9_filter.h: rename interp_kernel type

-> InterpKernel
avoids conflicts in variable names, fixing the build with various
toolchains.

broken since:
8691565 Removing subpix_fn_table struct.

Change-Id: Ib5f6fdbcb494a97b62c75b99d4d826ff25d4c981

Merge "INLINE and reimplement get_unsigned_bits()."

Build fix with config internal stats

Fixes a build issue when internal stats is enabled

Change-Id: I822cc60274e34b5f29ccbaa1f986fb9da6a8de4b

Merge "Removing "_short" suffix from arm transform file names."

Inlinging get_next_chkpts macro in vp9_mcomp.c.

Change-Id: I38ca35dccfb9051d08b026d75ba2f3a35adb0259

Moving MBGRAPH_{MB, FRAME}_STATS structs to vp9_mbgraph.h.

Change-Id: I424c79735a7f3bbf6bf9c1391ff53aa78abae624

Merge "Optimize bilinear sub-pixel filters in sse2"

Merge "Propgate constants through subexp routines."

Merge "Revert "Layer based rate control for CBR mode.""

Revert "Layer based rate control for CBR mode."

This reverts commit 6be2b750b8968a0b19180a6474864c0b6a41db13

Change-Id: Ic52acd98b37c3ba49d4999b463389eb564f49c4b

INLINE and reimplement get_unsigned_bits().

The new implementation disagrees when the argument is equal to 2**n but
that is never called in practice and based on how it is used the new
implementation is correct in that case.

Change-Id: Ifbac4ad87d459fe6bd2fd0f400c0340f96617342

Propgate constants through subexp routines.

This avoids calls to get_unsigned_bits() with constants and
replaces hard to trace loops with simpler structures.

Change-Id: Ic1afc5a17d7df5bcfc85b76efda316b0bf118467

Merge "Cleaning up vp9_mcomp.{c, h}."

Optimize bilinear sub-pixel filters in sse2

Using bilinear filters could speed up the codec in real-time mode.
This patch added sse2 optimizations of bilinear filters that
operate on different-sized blocks.

Tests showed that the real-time encoder was speeded up by 3%.

Change-Id: If99a7ee4385fcc225c3ee7445d962d5752e57c3f

Layer based rate control for CBR mode.

This patch adds a buffer-based rate control for temporal layers,
under CBR mode.

Added vpx_temporal_scalable_patters.c encoder for testing temporal
layers, for both vp9 and vp8 (replaces the old vp8_scalable_patterns).

Updated datarate unittest with tests for temporal layer rate-targeting.

Change-Id: I9cb6cce2494390ae6096ee17774af7fb9308bde7

vp9/encoder: fix function prototypes

silences warnings about declarations not matching prototype

Change-Id: I30b9a24f78ebe3b9cc2bbfcd3a7363ba7c328e4d

Merge "static function convert to inline or global vp9_blockd.h"

resize_test: use const_iterator where appropriate

Change-Id: I9ebfeb4947e7d7bc8af175730b0b25b2880ee26c

resize_test: silence a warning

the timebase of pts is 1/fps, so casting down to unsigned is safe here.

Change-Id: I2e2f9bb68604c3c5539804685e6166beb54fe83e

Merge "Set proper rtc interp_filter at block level"

Merge "Typo & comment corrections"

static function convert to inline or global vp9_blockd.h

Change-Id: Ifdd951f24932839f06d1c700371662511dde6ebe

Merge "libmkv: remove unused variable."

Merge "libmkv: Remove spurious declaration."

Merge "Rename a loopfilter parameter"

Merge "Cleaning up vp9_get_second_pass_params() function."

Cleaning up vp9_mcomp.{c, h}.

Change-Id: Ib71d9ed3f98e9468ad951bdc24c9ab565216eb38

Merge "Cleaning up motion compensation code."

Merge "remove duplicate const from vpx_codec_iface_t use"

Removing "_short" suffix from arm transform file names.

Change-Id: Iefe118f61a335e88821a21a9f50fb919212c1507

Cleaning up vp9_get_second_pass_params() function.

Change-Id: Ie7b295190d8def447408e575ea0f3fa3c7eb7dd2

remove duplicate const from vpx_codec_iface_t use

the public typedef already includes a const, quiets
'same type qualifier used more than once' warnings

Change-Id: Ib118b3b116fba59d4c6ead84d85b26e5d3ed363d

Set proper rtc interp_filter at block level

Based on frame level settings.

Change-Id: I922c09c195ab055585eaa4f544c51aceb1203a7a

Rename a loopfilter parameter

As pointed out by Dmitry and James, "partial" is a Microsoft-
specific c++ keyword, and it is renamed.

Change-Id: Ia0fc11ceb89e54b3195287f89f7e26edbbe9beb8

Merge "bsize problem 360p"

vpxenc: find_mismatch/compare_img: normalize types

use uint32_t where appropriate, silences a few warnings

Change-Id: Ic362b08868b00f4828982757a7bcee081d50308d

vpxenc: cosmetics

find_mismatch() / compare_img(): const, whitespace

Change-Id: I1c65c2aac378535001720aca31e7437ee5492739

Typo & comment corrections

Change-Id: I64569afe61a2679afdd263d3e226a89b9f73cc61

libmkv: remove unused variable.

Fixes a warning.

Change-Id: I7f98ea7847cf8e776a4bbd65d273036280bfb5dd

Merge "Combining fb_idx_ref_cnt[] and yv12_fb[] arrays."

Merge "Renaming "mbskip" to "skip"."

libmkv: Remove spurious declaration.

Change-Id: I5a83f6651a42df8387aa8996f9353a20e60483df

Merge "Only allow interp_filter change in SWITCHABLE mode"

Merge "fix IOCs"

Merge "Properly merge two different real time modes"

bsize problem 360p

Fixes an assert that crashed for 360p..

Change-Id: I2faf15c93cbdb0e62a27a3b663f0d09ba62774a8

Only allow interp_filter change in SWITCHABLE mode

This commit added a logic to prevent the inter_filter type from being
changed if the default interp_filter mode is not switchable. Also, it
sets the default interp_filter to BILINEAR at very and super fast rtc
encoding modes

Change-Id: Ic41e6d31de29795a4ce536ec79afb01cab6daad3

fix IOCs

Left shift of negative values caused IOC warnings.

Change-Id: I6f41b020ca0ff51f7861719d41393e9460b75d4e

Properly merge two different real time modes

--rt --cpu-used=-5 uses the progressive rtc mode
--rt --cpu-used=-6 uses the new super fast rtc mode

Change-Id: Id6469ca996100cdf794a0e42d76430161f22f976

vp9 decoder: row-based multi-threaded loopfilter

Implemented parallel loopfiltering, which uses existing tile-
decoding threads. Each thread works on one row, and when that row
is loopfiltered, it moves to next unattended row. To ensure the
correct filtering order, threads are synchronized and one
superblock is filtered only if the superblocks it depends on are
filtered already.

To reduce synchronization overhead and speed up the decoder, we use
nsync > 1 for high resolution.

Performance tests:
1. on desktop:
8-tile 4k video using 8 threads, speedup: 70% - 80%
4-tile HD video using 4 threads, speedup: ~35%
2. on mobile device(Nexus 7):
4-tile 1080p video using 4 threads, speedup: 18% - 25%
4-tile 1080p video using 2 threads, speedup: 10% - 15%

Change-Id: If54b4a11960dd706c22d5ad145ad94156031f36a

Cleanup block_rd_txfm.

* Avoid unnecessary type erasure
* Prune unused/duplicate fields from struct rdcost_block_args
* Make struct rdcost_block_args a local

Change-Id: I4f1fd4837ccd028bbfe727191ee8d69f0463b7e5

Merge "Add constant to represent minimum KF boost"

Disable update of last_show_frame for existing frame

When showing a previously decoded frame, i.e. when
show_existing_frame=1, the update of the
last_show_frame flag must be disabled.

This is to ensure that the last_show_frame flag
reflects the state of the flag for the immediately
previously decoded frame rather then the value that
was forced to ensure that a previously decoded frame
would be displayed.

This patch also adds a test vector to verify that the
display_existing_frame flag works as expected. Code
for generating the test vector can be found in this
patch:
https://gerrit.chromium.org/gerrit/#/c/68581/

(Bug originally reported by Alexander Voronov
<ru.xalba@gmail.com>).

Change-Id: I731d288fba02088959f7fcc87707137fffc6acf5

remove confusing compressor_speed

use mode instead

Change-Id: I419d7a2dc4b0714ca6ff723c5e824521c150c460

Add constant to represent minimum KF boost

Added a constant to represent the minimum KF boost
rather than using the magic number 2000 in the code.

Change-Id: I9428b61f47d26312caff81c6f9ae8587df004791

Merge "Replace inline with INLINE"

Merge "tests: fix a few msvc warnings"

Replace inline with INLINE

So x86_64-win64-vs11 can build successfully.

Change-Id: If354c2ea3921fac8c9b413ed39223e70bc20c535

Merge "Enforce the use of profile 1 for non-420 content."

Merge "Fixing out of bounds access in frame_refs[] array."

Merge "Fix some automerge artifacts"

Merge "Fix a build issue for --enable-intern-stats"

Fix some automerge artifacts

Some changes in 1ca1186 were mistakenly reverted by a later merge,
this commit re-instated the chanages from values to enums.

Change-Id: Ia6b01c31da584a1f612996e6432612c1295b9eaf

Fix a build issue for --enable-intern-stats

Change-Id: Iea7c9fa0726dbf9792eea79e6a05eb8a3c718d45

Merge "Changing ivf_write_frame_header() function signature."

Fixing out of bounds access in frame_refs[] array.

Change-Id: I08f45573e0b2195c09fb6aecacb4c57431a711ea

Merge "obj_int_extract (msvs): add missing project dep"

Changing ivf_write_frame_header() function signature.

Replacing vpx_codec_cx_pkt argument with two separate pts and frame_size.

Change-Id: I7b37e379ee71342520cf08f03acfb4b499b2cbe4

obj_int_extract (msvs): add missing project dep

obj_int_extract.bat
this project and target still need some work to allow for concurrent
builds to succeed from the command line.

Change-Id: Ieb3bddc54636e77519083c48573909616257eb23

Merge "create super fast rtc mode"