get_ref_frame: check ref_frame_map value

'ref_frame_map' is initialized to -1. avoids using an invalid index  if
VP9_GET_REFERENCE/VP8_COPY_REFERENCE controls are issued after a decode
error.

Change-Id: I4599762c4d0b07a5943a72bf4a86ccb596cc062a

VP8D_GET_FRAME_CORRUPTED: check frame pointer

if the decode of the first frame fails, frame_to_show may not be set.
fixes a crash in vpxdec with corrupt data.

Change-Id: I5ab9476d005778a13fd42a39d05876bb6c90a93c

vpx_codec_decode: check data size

When a valid data pointer is given make sure the size is greater than
zero.
A previous check for vp9 was incorrectly removed in:
7050074 Make the api behavior conform to api spec.

No semantics for valid pointers + 0-sized frames are defined for VPx
codecs, so move the check to vpx_codec_decode(). This avoids an assert
in vp9.

+ add some basic invalid param testing for decoder init/decode/destroy

Change-Id: I99f9cef6076d15874fd72ac973f2685d8a2353c3

Merge "Removing unised OUTPUT_YUV_SRC define."

Merge "Integrate fast txfm and quant path into skip_recode system"

Merge "Extend skip_txfm flag into array to cover YUV planes"

Merge "Adds spatial-svc macros to code to allow disabling"

Merge "test/vpxenc.sh: Convert vpxenc() to a simple wrapper."

Merge "vp8: Code cleanup for control of denoiser mode."

Removing unised OUTPUT_YUV_SRC define.

Change-Id: Ia88eebf10c8620dadaa66722543db291e44fe104

vp8: Code cleanup for control of denoiser mode.

Change-Id: Icb9918dd38d15061d62852e6a2d905e8ceb2c1ac

Adds spatial-svc macros to code to allow disabling

Adds a couple of CONFIG_SPATIAL_SVC macros to allow compilation
when spatial-svc is disabled.

Change-Id: I67f84406d1d3fa26f225d9c6518161a0841f2d29

Merge "Moving pass from VP9_COMP to VP9EncoderConfig."

Merge "Common encode/decode function to get reference frame"

test/vpxenc.sh: Convert vpxenc() to a simple wrapper.

- Split vpxenc() into vpxenc() and vpxenc_pipe().
- Drop all but one positional param (the input file) in favor
  of passing args directly to vpxenc.
- Add an extra lossless test that explicitly sets min-q and
  max-q to 0.

Change-Id: I7d5f7b495f8b9447388c5f459bc9f6de2214caf2

Merge "Removing legacy XMA features from libvpx."

Moving pass from VP9_COMP to VP9EncoderConfig.

We had a very complicated way to initialize cpi->pass from
cfg->g_pass:
switch (cfg->g_pass) {
  case VPX_RC_ONE_PASS:
    oxcf->mode = ONE_PASS_GOOD;
    break;
  case VPX_RC_FIRST_PASS:
    oxcf->mode = TWO_PASS_FIRST;
    break;
  case VPX_RC_LAST_PASS:
    oxcf->mode = TWO_PASS_SECOND_BEST;
    break;
}

cpi->pass = get_pass(oxcf->mode).

Now pass is moved to VP9EncoderConfig and initialization is simple:
switch (cfg->g_pass) {
  case VPX_RC_ONE_PASS:
    oxcf->pass = 0;
    break;
  case VPX_RC_FIRST_PASS:
    oxcf->pass = 1;
    break;
  case VPX_RC_LAST_PASS:
    oxcf->pass = 2;
    break;
}

Change-Id: I8f582203a4575f5e39b071598484a8ad2b72e0d9

Merge "Cleaning up vp9_encodeframe.c."

Common encode/decode function to get reference frame

Replaced encoder and decoder functions to get a pointer
to a reference frame with a common function, vp9_get_ref_frame,
and simplified it.

Change-Id: Icb206fcce8caace3bfd1db3dbfa318dde79043ee

Remove coding_use_prev_mi member from VP9_COMMON

This was shadowing the use of error_resilient_mode, but with
the opposite sense.

Change-Id: Ie4d30263a304fe4b3e94f0c7741db6888cc6afd8

Merge "Fix active_map speed 6."

Merge "Cleanup SEG_LVL_SKIP handling in encode_superblock."

Merge "Fix bug 807"

Fix bug 807

in the sub_pixel_*variance* function the dst is aligned to 16 bytes and not
to 32 bytes - now load unaligned data

Change-Id: I2e0b9745543697efc56fefa32857ea10117af135

test/vpxenc.sh: Enable the VP9 lossless test.

Change-Id: Iff403dfb4c8face0494f0dba840ce91f51e9ca04

Merge "vp9 ssvc test: Remove disabled tests."

Merge "vpxenc tests: Relocate vpxenc wrapper."

Merge "Fix bug 806"

Fix active_map speed 6.

Fix the interaction between active map and reuse_inter_pred_sby. The
reuse_inter_pred_sby feature expects inter predictors to already be
built, but blocks with active map on skip this step.

Change-Id: Ibb2bf0d228f678935d82a0ede9cb0919ab7c8878

Cleanup SEG_LVL_SKIP handling in encode_superblock.

Change-Id: Ib7497ba08696765cbc1b2cc4218d37f4298f278c

vp9 ssvc test: Remove disabled tests.

The disabled tests require a mode parameter that has been removed
from the ssvc example.

BUG=https://code.google.com/p/webm/issues/detail?id=833

Change-Id: I1a5d67628ed769cc5372c4a864f33d5569b6eb13

Merge "tools_common.c: unsigned signed mismatch addressed"

Cleaning up vp9_encodeframe.c.

Change-Id: Ia3001ae5c44faee3978fc3eb7a027cd9712a0373

Merge "vp8: for spatial resize, reallocate denoiser in vp8_allocate_comp."

Fix bug 806

in the function sad32x32x4d and sad64x64x4d the source is aligned to 16 bytes
and not to 32 bytes - the load is now unaligned.

Change-Id: I922fdba56d0936b5cf72e4503519f185645a168c

Merge "Removing direct references to VP9_COMP."

vp8: for spatial resize, reallocate denoiser in vp8_allocate_comp.

Change-Id: I1827efe2126501a4a5637ee3ae85233d7526bc0e

Merge "vp9_bitstream: make some function params const"

vpxenc tests: Relocate vpxenc wrapper.

Move vpxenc() from tools_common.sh to vpxenc.sh.

Change-Id: If3bc1e8616f7e0334d39b560946e467482fe5cf4

Merge "Changes hdr for profiles > 1 for intraonly frames"

vp9_bitstream: make some function params const

Change-Id: Ic49e290c7c88c55e730a0b2b0eb16ee3d0f120e7

Merge "vp9_cx_iface.c: warning missing default"

Merge "Adding ticks <-> timebase units conversion functions."

Changes hdr for profiles > 1 for intraonly frames

Specifies the bit-depth, color sampling and colorspace
for intra only frames for profiles > 0

Also adds checks to ensure that profile 1 and 3 are
exclusively used for non 420 streams.

Change-Id: Icfb15fa1acccbce8f757c78fa8a2f60591360745

Merge "configure: add --enable-coefficient-range-checking"

tools_common.c: unsigned signed mismatch addressed

Change-Id: I919ca42794f13751ca099868e73892d9df64e45f

vp9_cx_iface.c: warning missing default

Change-Id: I18cbdad6031d14d4c1a818c0ab33bbedf7b33e19

Merge "vp9_rd.c: cleanup ugly expression"

Merge "Make the api behavior conform to api spec."

configure: add --enable-coefficient-range-checking

This commit adds a configure time option used to enable strict error
checking in decoder to make sure intermediate stage cofficients of
inverse transforms are within valid range of signed 16 bit integer.

For valid VP9 input streams, intermediate stage coefficients should
always stay within the range of a signed 16 bit integer. Coefficients
can go out of this range for invalid/corrupt VP9 streams. However,
strictly checking this range for every intermediate coefficient can
be a burden for decoder, therefore such validation is only enabled
with configure option --enable-coefficient-range-checking.

Change-Id: I47d47c8c4e48a922c3d223ca59064f51b3f0f5ed

vp9_rd.c: cleanup ugly expression

Change-Id: I066763c2e3ae69ab772bb986da97af9b504f0329

Merge "[spatial svc]Add is_spatial_svc() helper function."

Integrate fast txfm and quant path into skip_recode system

This commit integrates the fast transform and quantization process
into skip_recode scheme in the rate-distortion optimization loop.
Previously the fast transform and quantization process was only
enabled for non-RD coding flow.

Change-Id: Ib7db4d39b7033f1495c75897271f769799198ba8

Merge "vp8 bugfix: Reallocate denoiser when spatially resizing."

[spatial svc]Add is_spatial_svc() helper function.

Change-Id: Ice5376100d8e27cbdaddfd3cd06898cedd2720fe

Adding ticks <-> timebase units conversion functions.

Change-Id: I75abd57367a7974a9fab8a727b2bbc54dea428c3

vp8 bugfix: Reallocate denoiser when spatially resizing.

This is needed to update the width/height and stride parameters
for the reference buffers that the denoiser uses.

Change-Id: Id51b3bdcb56bbbc8187865544ccd3d872a0d51fe

Merge "Create function vp9_preserve_existing_gf"

Removing direct references to VP9_COMP.

Change-Id: Ic37624d807884e71f08b50fd04892f03f2708ba7

Merge "Write VP9E_SET_TUNE_CONTENT to the correct location."

Merge "vp9_subexp: remove vp9_compute_update_table()"

Make the api behavior conform to api spec.

When no more data is available, vpx_codec_decode should
be called with NULL as data and 0 as data_sz.

vpx_codec_get_frame iterates over a list of the frames
available for display. The iterator storage should be initialized
to NULL to start the iteration. Iteration is complete when this
function returns NULL.

Also change the unit test to conform to the api spec.

Change-Id: I4b258b309f5df3d37d10c82f01492c0394181c2a

Write VP9E_SET_TUNE_CONTENT to the correct location.

Change-Id: Ie8c976ececc79fd58c6eb16b7add053dc58e3bb8

Merge "Use INTER_ALL for VAR based partitions for screencast material."

Merge "Add a codec control option to tune for screen content."

Merge "Remove vp9_postproc_x86.h"

Merge "Directly split the block in partition search"

Removing legacy XMA features from libvpx.

Change-Id: I60ffee7cf78e748792fa6d109322ec32f92e40a9

Directly split the block in partition search

This patch allows the encoder to directly split the block
in partition search, therefore skip searching NONE. It
computes a score which measures whether 16x16 motion vectors
from the first pass in the current block are consistent with
each others. If they are inconsistent and we have enough Q
to encode, split the block directly, and skip searching NONE.

This feature is under flag CONFIG_FP_MB_STATS. In speed 2,
it further gives a speedup of 3-8% on sample yt clips as
compared to the previous version under the same flag. Overall,
the features under the flag will give 7-15% on typical yt
clips at up to 6000kbps data rate. The speedup at very high
data rate is not significant.

For hard stdhd clips:
park_joy_1080p @ 15000kbps:       504541ms -> 506293ms (-0.35%)
pedestrian_area_1080p @ 2000kbps: 326610ms -> 290090ms (+11.2%)

The compression performance using the features under the flag:
derf: -0.068%
yt:   -0.189%
hd:   -0.318%
stdhd:-0.183%

To use the feature, set CONFIG_FP_MB_STATS and turn on
cpi->use_fp_mb_stats.

Change-Id: Iad58a2966515c8861aa9eb211565b1864048d47f

Remove vp9_postproc_x86.h

This configuration has moved to vp9_rtcd_defs.pl

Change-Id: I71a31dbb8d79df226b60dd834324a5af69956c51

Extend skip_txfm flag into array to cover YUV planes

Change-Id: Ieae182d72d625d0d3fd4ed7c7d24cb521a0f21b0

Create function vp9_preserve_existing_gf

This code was being called from two places and
difficult to parse. I rationalized it in to a
function to improve readability.

Change-Id: I154b8fe0b84e6c01e69601e78e67bd47c954d8b6

Use INTER_ALL for VAR based partitions for screencast material.

This offers 25% more compression on my HD screencast testset.

Change-Id: I85eaef95fd8f2e03e326443e9514482b2ee35cef

Add a codec control option to tune for screen content.

Change-Id: I12b0b0a06750cb9a5f1bdc3368deb44137acc9a3

vp9_subexp: remove vp9_compute_update_table()

makes update_bits[] a constant table

Change-Id: If105bf0522487f54a31e12a7cc73aa24671b19d4

Merge "[spatial svc]Enabl golden frame for base layer and fix wrong ref_frame_flag for upper layers in first frame"

Merge changes I4e2aeaef,Ie3e478ef

* changes:
  vpxdec warning errors addressed
  cast enums to int to avoid gcc warning in pred_common

Merge "Store first pass motion vector directions"

Store first pass motion vector directions

Re-organize the one-byte structure for 16x16 first pass
block. Add bits to indicate motion vector directions.

Change-Id: Id10754ba343dfc712c7fed5bcc85c67fa0bbcb89

Merge "Cosmetic indent fix in vp9_rd_pick_inter_mode_sb"

[spatial svc]Enabl golden frame for base layer and fix wrong ref_frame_flag for upper layers in first frame

Change-Id: I1fec9c084b499b2f445b79726066d01b136b59fa

Cosmetic indent fix in vp9_rd_pick_inter_mode_sb

Change-Id: I303213fbb0ad5fbf28f0ca17271624b75e9d6319

vpxdec warning errors addressed

Change-Id: I4e2aeaef3e6efd5be3cb963d2eba5e08dc89b803

cast enums to int to avoid gcc warning in pred_common

Change-Id: Ie3e478ef4fa565225d9e19a14d2f40aad966c2b6

energy -> int to avoid unsigned / signed mismatch

Change-Id: Idd1327852f0df0eab0ea3b33959f2b8292b77301

vp9_firstpass.c: fix unused static function

Change-Id: Ia989a01d79f1f02defbeccb1b8eee23fdf5e764e

set but unused var in vp9_rdopt

Change-Id: Ie813af24d23c7d72b38749b3382b315daf516be5

break at the end of clauses with assert(0) to avoid gcc warning

Change-Id: I1b3c5337f018dde27dc819ab18bd081d169a91e8

uint8_t segment and skip to avoid signed / unsigned warnings

Change-Id: I2e2765b851fb0a1b15351c2aa0e079197cbee373

avoid signed mismatch warning in gcc

Change-Id: I9d995178fd764be252e571fa23901da0f991bbb8

Merge "frame_size_tests: cosmetics"

Merge "vp9_entropy: inline comes first to avoid warning."

Merge "signed unsigned mismatch - warning error"

vp9_entropy: inline comes first to avoid warning.

Change-Id: I5b050122e6ed183a5b33c1f38e4fbf63b6721062

Merge "removed sign mismatch warning"

Merge "Added vp9_sad8x8_neon()"

frame_size_tests: cosmetics

- output DecodeError() on failure
- remove unused includes

Change-Id: Ib5059137f7033089d4010cbdfd633861b9028718

Neon version of vp9_sub_pixel_variance8x8(),

vp9_variance8x8(), and vp9_get8x8var().

On a Nexus 7, vpxenc (in realtime mode, speed -12)
reported a performance improvement of ~1.2%.

Change-Id: I8a66ac2a0f550b407caa27816833bdc563395102

Merge "Neon version of vp9_sub_pixel_variance32x32(),"

Added vp9_sad8x8_neon()

Change-Id: I3be8911121ef9a5f39f6c1a2e28f9e00972e0624