Add mode info arrays and mode info index.

In non frame-parallel decoding, this works the same way as
current decoding scheme. Every time after decoder finish
decoding a frame, it will swap the current mode info pointer
and  previous mode info pointer if the decoded frame needs
to be shown. Both mode info pointer and previous mode info
pointer are from mode info arrays.

In frame-parallel decoding, this will become more complicated
as current frame's mode info pointer will be shared with next
frame as previous mode info pointer. But when one decoder
thread finishes decoding one frame and starts to work on next
available frame, it needs to retain the decoded frame's mode
info pointers until next frame finishes decoding. The mode info
index will serve this purpose. The decoder will use different
buffer in the mode info arrays and use the other buffer to save
previous decoded frame’s mode info.

Change-Id: If11d57d8eb0ee38c8876158e5482177fcb229428

Merge "Use small transform size in non-rd real-time mode"

Merge "Removing unused tt_activity_measure()."

Use small transform size in non-rd real-time mode

In non-rd real-time mode, choosing smaller transform size in
encoding gives better video quality and good speed gain than
choosing larger transform size. This patch set tx size search
method to ALLOW_8X8, which is better than using 4x4 or other
larger sizes.

Borg tests on rtc set at speed 6 showed significant gain on quality.
PSNR gain: 11.034% and SSIM gain: 15.466%.

The speed gain is 5% - 12% for <720p clips, and 2% - 7% for
720p clips.

Change-Id: If4dc74ed2df359346b059f47fb73b4a0193ec548

Merge "Fix internal stats printing"

Merge "gen_msvs_*proj.sh: speed up file generation"

gen_msvs_*proj.sh: speed up file generation

execute fix_path once on the source file list rather than once per entry

Change-Id: Ibc8226e391b3028c1b0bcfeab83c790387c9fe23

Merge "Revert "Removing this_frame_stats member from TWO_PASS struct.""

Revert "Removing this_frame_stats member from TWO_PASS struct."

Use of stack frame variable "fps" beyond the lifetime of the function.

fps is sent as a paremeter to output_stats and stored in the
packet holding this encoded frame. This has scope beyond the
lifetime of the calling function.

This reverts commit 3f95a230c7e54104201b3f6c577dfa9251b4642a

Change-Id: Icd8e14b3d7dd733590ada12e619b9dce95b6b0f5

Merge "Adding encode_tiles() function."

Merge "Fixes qindex for first frame in 1-pass cq/q modes"

Merge "README: add a note about the test vectors"

Merge changes I0e4d807f,Ia5ff575c,Ie4a1f313

* changes:
  gen_msvs_*proj.sh: strip SRC_PATH_BARE from obj names
  *.mk: pass SRC_PATH_BARE to all GEN_VCPROJ invocations
  build/msvs: fix builds in source dirs with spaces

Adding encode_tiles() function.

Change-Id: Ib8187c8f2556e1e9268b0683cd2b6ff3489f0205

Merge "Neon match to vp8 temporal denoiser fix"

Fixes qindex for first frame in 1-pass cq/q modes

Produces sane qindex for the first frame in 1-pass constant and
constrained qualirty modes.

Change-Id: Ib2a5091df15a23489e9bb5534a2019cf2689755e

Merge changes I59ef7c49,I0156d856

* changes:
  Made MACROBLOCK.increase_denoising cond-compiled
  s/INT_MAX/UINT_MAX/ where appropriate

Fix internal stats printing

Change-Id: I61bd0b127164a591b1c983bfcebd64ba7617f796

Add x86-iphonesimulator-gcc target.

macho32 i386 with:
- -miphoneos-version-min=5.0
- -isysroot from xcrun --sdk iphonesimulator

Change-Id: I6bb43eebab39a71cdb76264644eba14c22d736c2

Removing unused tt_activity_measure().

Change-Id: Ifcb46e6904730d14b9ef76b648b4d0dc3cd5d0c5

Removing unused alt_freq field from VP9EncoderConfig.

Change-Id: I9b683c8647a864e74073161f4aa6f2911b7825e3

Made MACROBLOCK.increase_denoising cond-compiled

Change-Id: I59ef7c49f72d2d40bbe5b56af11bdf5f9ae2f1b8

s/INT_MAX/UINT_MAX/ where appropriate

Change-Id: I0156d85671305326525c4644510e240021eca461

Merge "Enable unit test for partial 16x16 inverse 2D-DCT"

Using 2 instead of 3 elements for avg_frame_qindex array.

The third array element was unused. 2 elements now: key- and interframe.

Change-Id: I5b8b9f5d889cc96a204cedfc432059293256298e

Enable unit test for partial 16x16 inverse 2D-DCT

This commit enables unit test for SSSE3 16x16 inverse 2D-DCT with
10 non-zero coefficients. It includes a new test condition to
cover the potential overflow issue due to extremely coarse quantization.

Change-Id: I945e16f05dfbe19500f0da5f15990feba8e26d99

Merge "Re-enable unit test for SSSE3 forward 8x8 2D-DCT"

Merge "Enable SSSE3 inverse 2D-DCT with 10 non-zero coeffs"

Re-enable unit test for SSSE3 forward 8x8 2D-DCT

This commit enables the unit test for SSSE3 forward 8x8 2D-DCT.

Change-Id: Ie4aae84ac74f6eb1a80b62285c91632bd6db29f1

Merge "Removing lossless field from VP9EncoderConfig."

Merge "Fix potential overflow issue in SSSE3 forward 8x8 2D-DCT"

Merge "Rework unit test for 8x8 transformation"

Fix potential overflow issue in SSSE3 forward 8x8 2D-DCT

The SSSE3 implementation might find a potential overflow issue in
its second 1-D transform, if all input residual pixels are close to
255. This commit fixes the issue and re-enables the unit test on
the SSSE3 version.

Change-Id: I0520478abdab7afd3ff2842516bec951111e9b3c

Rework unit test for 8x8 transformation

This commit reworks the unit test for 8x8 forward/inverse
transformation. It adds extreme input value test to detect overflow
issues in the intermediate steps.

It temporarily disables unit test for the SSSE3 version, which
showed overflow failure in the new test conditions.

Change-Id: I7caf10bba4b6db031add65d8c0eb99426b38aa42

Adding buffer levels to RATE_CONTROL struct.

Change-Id: Ib35ff854378764dc3c6745844c67a33dee545663

Removing lossless field from VP9EncoderConfig.

Right now there is just one place to check: xd->lossless and for the first
pass there is a function is_lossless_requested().

Change-Id: I949a6834e64ce51e422e2892f097f2b871b5429a

Merge "Cleaning up full_pixel_search()."

Merge "Moving first pass related functions to vp9_firstpasss.c."

Merge "Reusing existing vp9_get{8x8, 16x16}var() instead of new ones."

Fix AQ mode 2 bug where delta causes Q 0.

In Aq mode 2 for kf/arf/gf the segment q delta
is calculated and then applied by re-quantization without
going through the rd loop again. If the base Q != 0
but the segment Q == 0 (lossless) this can could give rise
to a situation where we have an illegal combination of
transform size and Q. (Q == 0 requires that all blocks
are coded 4x4 WHT).

Change-Id: I241a58c6494ed442e9e4630070b0cde0fb99ae45

Merge "Remove Wextra warnings from vp9_sad.c"

Merge "Remove an attempt to handle SEG_LVL_SKIP sub8x8."

gen_msvs_*proj.sh: strip SRC_PATH_BARE from obj names

reduces the risk of exceeding msbuild command line length limits

Change-Id: I0e4d807f42c090a709a3dc365a02b929a3071688

*.mk: pass SRC_PATH_BARE to all GEN_VCPROJ invocations

this will enable stripping of the path from the object file name

Change-Id: Ia5ff575cfa4fe946200878b0aded05832609f316

build/msvs: fix builds in source dirs with spaces

...when configured below the path containing spaces. configuring outside
the path containing spaces still won't work due to issues with the
makefiles, e.g.,
/path with spaces/git
/path with spaces/build1
/build2
configure/make in build1 will work, build2 will not

Change-Id: Ie4a1f313596d7457cadd67476ac1dbd3273ad46e

Remove Wextra warnings from vp9_sad.c

As a side-effect, the sad unit tests for VP8 and VP9
had to be separated.

Fixes a bug in original patch:
(https://gerrit.chromium.org/gerrit/#/c/70163/8)
that was reverted due to a nightly test failure.

Change-Id: Ia2a4e9e278fd3c89d6c3c82fcc6381320ec2a8a6

Fixing failed ARM build.

Change-Id: I3f74418f07c2dfdd7725a5b4a8ef5c5f4aca6289

Merge "Fixed OUTPUT_YUV_SRC behavior for VP8"

Merge "seeing a 10x slowing down, revert now for investigation"

seeing a 10x slowing down, revert now for investigation

Revert "Fix a problem of using an uninitialized parameter"

This reverts commit 538af7db5ffe2ab560511a3bc0ae74dae9316c14

Change-Id: I071aa9b7068ef515abb8ae9584df15067706ccb5

Merge "Revert "Remove Wextra warnings from vp9_sad.c""

Revert "Remove Wextra warnings from vp9_sad.c"

This reverts commit 916550428db803c54c993ff9d3c34b9b0bcebb7c

Change-Id: I500822b03f09c64ff6ec5396c68edee9ca3b75cb

Merge "Converting disable_inter_mode_mask to inter_mode_mask."

README: add a note about the test vectors

Change-Id: I8b3299bf1579962c6dbb29b61c16801fef2460fd

Merge "Add overflow check unit test for 16x16 inverse DCT/ADST transform"

Merge "Fix a potential overflow issue in inverse 16x16 full 2D-DCT"

Merge "Fix unused variable warning."

Merge "Refactor the vp9_get_frame code for frame parallel."

Add overflow check unit test for 16x16 inverse DCT/ADST transform

This commit applies quantization process with coarse quantization
step size to the forward transform coefficients and tests all the
inverse 16x16 DCT and ADST implementation versions with the
dequantized coefficients as input, to verify that the outcomes
match the prototype.

Change-Id: I68034a6126b45192c87d8c642155290e89bff8fa

Merge "Fix a problem of using an uninitialized parameter"

Merge "Removing unused ref_frame_mask local var."

Refactor the vp9_get_frame code for frame parallel.

In frame parallel decoding mode,  there will be still several frames inside
the decoder when application stop calling vpx_codec_decode to decode frames.
The application will need to keep calling vpx_codec_get_frame to get all the
remaining decoded frames in the decoder.

Change-Id: I2ce8260a91282f045bb9a6093ff8a606b1990f14

Merge "Do not export non-existant symbols"

Fix unused variable warning.

Change-Id: I52f73943194039697da6880d2da54821b2498336

Fix a problem of using an uninitialized parameter

This commit added a call to set speed feature before initializing
motion search, fixed the problem where unintialized search method
is used before its value being set.

Change-Id: I537e4612bf0d00fd6f51396fd222d4b3bd6fde58

Fixed OUTPUT_YUV_SRC behavior for VP8

By enabling the OUTPUT_YUV_SRC compiler flag, the encoder will write the raw
input to bd.yuv.

The functionality was mostly implemented, but in its previous state did not
compile.

Change-Id: Ia331ad0f4c6e6f9f51e8d42cd33ba8cc146b3dbf

Merge "Re-factor some duplicate code."

Do not export non-existant symbols

Change-Id: Ic0fb8e5016d064c2227cf5d87ffba86a4303c82b

Removing unused ref_frame_mask local var.

Change-Id: Ie11558c076a0161cc9608788e050b1b16e31c490

Merge "Consistent names for intra mask flags."

Merge "Don't update encoder skip count for SEG_LVL_SKIP."

Converting disable_inter_mode_mask to inter_mode_mask.

Making this consistent with intra mode masks: you need to specify
allowed inter/intra modes to use.

Change-Id: Iaecd28bf79047259707d8e7a59a57bb7b856383e

Consistent names for intra mask flags.

Change-Id: Ibdd5255d37200fb8a1d50f71a2a49c6089ae21e7

Remove an attempt to handle SEG_LVL_SKIP sub8x8.

SEG_LEVEL_SKIP requires the block size to be at least 8x8. Attempting to
use it on smaller partitions causes the decoder to reject the bitstream.

Change-Id: Ia7188cdf8ae5ac1df6bd29f3f80dbb0610e1f7b1

Merge "Making speed checks consistent in set_rt_speed_feature()."

Fix a potential overflow issue in inverse 16x16 full 2D-DCT

An overflow issue could potentially happen in the second round 1-D
transform of the SSSE3 full inverse 16x16 2D-DCT. This commit fixes
this issue.

Change-Id: Ia19e4888fda1cc929a28a5f89a5beec612d628dc

Don't update encoder skip count for SEG_LVL_SKIP.

This aligns the encoder behavior with the decoder.

Change-Id: Ifa0840e4b07b19309e0bf1d1182498883249ec45

Merge "Moving itxm_add pointer from MACROBLOCKD to MACROBLOCK."

Reusing existing vp9_get{8x8, 16x16}var() instead of new ones.

Change-Id: I87b7c657d8813d7fb383ab519d150c0ffb1dd377

Making speed checks consistent in set_rt_speed_feature().

Change-Id: Id3d0a49836fe996b806707d29a8130acf9d7ea0e

Merge "Fixing -Wextra warnings in vp9_{cx, dx}_iface.c."

Merge "Fixed a crash windows build"

Fixed a crash windows build

Change-Id: I58baa1da1f3bfc8a6da454399139fe6a7473ff10

Neon match to vp8 temporal denoiser fix

Now match the "C" version of "Fix to reduce block
artifacts from vp8 temporal denoiser."
(see change id Id9b56e59e33f3c22e79d2f89f763bdde246fdf3f)

Change-Id: I99e569bb6af4ae3532621127e12bf917a48ba08e

Merge "vp8 denoiser: fix to zero_mv mode selection."

vp8 denoiser: fix to zero_mv mode selection.

In the current logic, if the sse for zero motion is smaller
than the sse for new_mv (i.e., best_sse), we may still end up
using the non-zero mv for denoising (if the magnitude of new_mv is above threshold).
This can happen for very noisy content, and can lead to artifacts.

This change ensures that we always use zero_mv (over new_mv) for
denoisng if sse_zero_mv <= best_sse.

Change-Id: I8ef9294d837b077013b77a46c9a71d17c648b48a

Fixing -Wextra warnings in vp9_{cx, dx}_iface.c.

Change-Id: I0abad32551dc534d3db27424c118e4b2f6b50f37

Merge "Using 2 instead of 3 elements for last_q array."

Merge "Removing redundant vp9_zero() call."

Enable SSSE3 inverse 2D-DCT with 10 non-zero coeffs

This commit enables SSSE3 implementation of the inverse 2D-DCT
with only first 10 coefficients non-zero. It reduces the runtime
of SSE2 version from 745 cycles to 538 cycles, i.e., 27% speed-up.

Change-Id: I18ba4128859b09c704a6ee361d69a86c09fe8dfe

Merge "Cleaning up vp9_variance_sse2.c."

Merge "Always allow ZEROMV when SEG_LVL_SKIP is on."

Merge "Fix the all intra modes mask constant."

Merge "Removing this_frame_stats member from TWO_PASS struct."

Re-factor some duplicate code.

Change-Id: I89a1dbea39c50c7633f746d9c93fec3a289f1b42

Merge "Remove brightness weighting in two pass."

Removing undeclared identifier - build fix

Fixes build with --enable-internal-stats

Change-Id: I137169c859f561478e45891defe976d595454166

Cleaning up full_pixel_search().

Change-Id: Ie517ac06385133ffb3bbc449d9f23240f245976d

Using 2 instead of 3 elements for last_q array.

Change-Id: I2c6950e7d79fc89c6f97e6dcf47317ef66c453a5

Always allow ZEROMV when SEG_LVL_SKIP is on.

Change-Id: I6db1dc82f66438ac48f571d2f1a2ac7c39a97a1a