Commit 6d0e2690 authored by aconverse's avatar aconverse Committed by Commit bot

Revert of Use tiles when encoding with VP9. (patchset #1 id:1 of...

Revert of Use tiles when encoding with VP9. (patchset #1 id:1 of https://codereview.chromium.org/977663004/)

Reason for revert:
Breaks linux valgrind,
needs libvpx side fixes and roll, http://build.chromium.org/p/chromium.memory.fyi/builders/Linux%20Tests%20%28valgrind%29%283%29/builds/44420

Original issue's description:
> Use tiles when encoding with VP9.
>
> VP9's tile encoding feature allows for multithreaded parallelism on both the
> encoder and decoder side.
>
> TEST=Run remoting_perftests under VTune and see that two logical CPUs are used
> BUG=134202
>
> Committed: https://crrev.com/e67fefbf4f5a5a83ca8d487f422b09664dc4c8fe
> Cr-Commit-Position: refs/heads/master@{#319167}

TBR=wez@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=134202

Review URL: https://codereview.chromium.org/978133002

Cr-Commit-Position: refs/heads/master@{#319185}
parent c9e732f0
......@@ -133,10 +133,6 @@ void SetVp9CodecOptions(vpx_codec_ctx_t* codec, bool lossless_encode) {
ret = vpx_codec_control(
codec, VP9E_SET_TUNE_CONTENT, VP9E_CONTENT_SCREEN);
DCHECK_EQ(VPX_CODEC_OK, ret) << "Failed to set screen content mode";
// VP9 tiles allow for parallelism on both encode and decode
ret = vpx_codec_control(codec, VP9E_SET_TILE_COLUMNS, 1);
DCHECK_EQ(VPX_CODEC_OK, ret) << "Failed to set tile columns";
}
void CreateImage(bool use_i444,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment