MediaRecorder: Tune complexity settings for VP9 encoding

MediaRecorder: Tune complexity settings for VP9 encoding

The first landing on this branch got reverted due to a silly
mistake: it needed a previous CL (the first go). Now both are
considered below.


--- description for the second CL -----------------------------
This code was landed but -silly me- I was setting the complexity
_before_ creating the encoder :P

Also, the values the VP8E_SET_CPUUSED parameter are:
 - up to 8 for VP9 (and not up to 16, that's for VP8)
 - Real Time values are 5 - 8 [1]

BUG=591342, 589162
[1] https://code.google.com/p/chromium/codesearch#chromium/src/third_party/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc&sq=package:chromium&type=cs&l=35&rcl=1458143321

--- description for the first CL ------------------------------

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

Cr-Commit-Position: refs/heads/master@{#381572}

VideoTrackRecorder: limit VP9 cpu usage and quality

Also this CL simplifies the way VideoTrackRecorderTests
deals with the expectations, in the same way as
MediaRecorderHandlerTest was simplified before in
https://codereview.chromium.org/1579693006

BUG=589162
Review URL: https://codereview.chromium.org/1732083002

Cr-Commit-Position: refs/heads/master@{#378085}



TBR=niklase@chromium.org
NOTRY=true
NOPRESUBMIT=true

[mcasas, 2016-03-18 01:56:26]

Description was changed from

==========
MediaRecorder: Tune complexity settings for VP9 encoding

The first landing on this branch got reverted due to a silly
mistake: it needed a previous CL (the first go). Now both are
considered below.


--- description for the second CL -----------------------------
This code was landed but -silly me- I was setting the complexity
_before_ creating the encoder :P

Also, the values the VP8E_SET_CPUUSED parameter are:
 - up to 8 for VP9 (and not up to 16, that's for VP8)
 - Real Time values are 5 - 8 [1]

BUG=591342, 589162

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/webrtc...

--- description for the first CL ------------------------------

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

Cr-Commit-Position: refs/heads/master@{#381572}

VideoTrackRecorder: limit VP9 cpu usage and quality

Also this CL simplifies the way VideoTrackRecorderTests
deals with the expectations, in the same way as
MediaRecorderHandlerTest was simplified before in
https://codereview.chromium.org/1579693006

BUG=589162

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

Cr-Commit-Position: refs/heads/master@{#378085}
==========

to

==========
MediaRecorder: Tune complexity settings for VP9 encoding

The first landing on this branch got reverted due to a silly
mistake: it needed a previous CL (the first go). Now both are
considered below.


--- description for the second CL -----------------------------
This code was landed but -silly me- I was setting the complexity
_before_ creating the encoder :P

Also, the values the VP8E_SET_CPUUSED parameter are:
 - up to 8 for VP9 (and not up to 16, that's for VP8)
 - Real Time values are 5 - 8 [1]

BUG=591342, 589162

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/webrtc...

--- description for the first CL ------------------------------

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

Cr-Commit-Position: refs/heads/master@{#381572}

VideoTrackRecorder: limit VP9 cpu usage and quality

Also this CL simplifies the way VideoTrackRecorderTests
deals with the expectations, in the same way as
MediaRecorderHandlerTest was simplified before in
https://codereview.chromium.org/1579693006

BUG=589162

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

Cr-Commit-Position: refs/heads/master@{#378085}



TBR=niklase@chromium.org
NOTRY=true
NOPRESUBMIT=true
==========

[mcasas, 2016-03-18 16:49:30]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2016-03-18 16:49:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1814853006/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1814853006/1

[commit-bot: I haz the power, 2016-03-18 16:51:33]

Description was changed from

==========
MediaRecorder: Tune complexity settings for VP9 encoding

The first landing on this branch got reverted due to a silly
mistake: it needed a previous CL (the first go). Now both are
considered below.


--- description for the second CL -----------------------------
This code was landed but -silly me- I was setting the complexity
_before_ creating the encoder :P

Also, the values the VP8E_SET_CPUUSED parameter are:
 - up to 8 for VP9 (and not up to 16, that's for VP8)
 - Real Time values are 5 - 8 [1]

BUG=591342, 589162

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/webrtc...

--- description for the first CL ------------------------------

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

Cr-Commit-Position: refs/heads/master@{#381572}

VideoTrackRecorder: limit VP9 cpu usage and quality

Also this CL simplifies the way VideoTrackRecorderTests
deals with the expectations, in the same way as
MediaRecorderHandlerTest was simplified before in
https://codereview.chromium.org/1579693006

BUG=589162

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

Cr-Commit-Position: refs/heads/master@{#378085}



TBR=niklase@chromium.org
NOTRY=true
NOPRESUBMIT=true
==========

to

==========
MediaRecorder: Tune complexity settings for VP9 encoding

The first landing on this branch got reverted due to a silly
mistake: it needed a previous CL (the first go). Now both are
considered below.


--- description for the second CL -----------------------------
This code was landed but -silly me- I was setting the complexity
_before_ creating the encoder :P

Also, the values the VP8E_SET_CPUUSED parameter are:
 - up to 8 for VP9 (and not up to 16, that's for VP8)
 - Real Time values are 5 - 8 [1]

BUG=591342, 589162

[1]
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/webrtc...

--- description for the first CL ------------------------------

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

Cr-Commit-Position: refs/heads/master@{#381572}

VideoTrackRecorder: limit VP9 cpu usage and quality

Also this CL simplifies the way VideoTrackRecorderTests
deals with the expectations, in the same way as
MediaRecorderHandlerTest was simplified before in
https://codereview.chromium.org/1579693006

BUG=589162

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

Cr-Commit-Position: refs/heads/master@{#378085}



TBR=niklase@chromium.org
NOTRY=true
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-03-18 16:51:34]

Committed patchset #1 (id:1)