Implement QP parsing in VideoDecoderWrapper.

Implement QP parsing in VideoDecoderWrapper.

BUG=webrtc:7760
NOTRY=True

Review-Url: https://codereview.webrtc.org/3000993002
Cr-Commit-Position: refs/heads/master@{#19378}
Committed: https://chromium.googlesource.com/external/webrtc/+/4dee3444936f27c08e736c58c36fa37802298a6f

[sakal, 2017-08-16 10:44:13]

Patchset #1 (id:1) has been deleted

[sakal, 2017-08-16 10:45:19]

sakal@webrtc.org changed reviewers:
+ kthelgason@webrtc.org

[sakal, 2017-08-16 10:45:19]

PTAL

[kthelgason, 2017-08-16 13:57:58]

https://codereview.webrtc.org/3000993002/diff/40001/webrtc/sdk/android/src/jn...
File webrtc/sdk/android/src/jni/videodecoderwrapper.cc (right):

https://codereview.webrtc.org/3000993002/diff/40001/webrtc/sdk/android/src/jn...
webrtc/sdk/android/src/jni/videodecoderwrapper.cc:75: qp_parsing_enabled_ =
true;
It's quite unclear what this depends on, can we try to just set it in one place
and make it clearer on what the decision is based? If it's not possible to do
that  in code consider adding a comment.

[sakal, 2017-08-16 14:47:07]

https://codereview.webrtc.org/3000993002/diff/40001/webrtc/sdk/android/src/jn...
File webrtc/sdk/android/src/jni/videodecoderwrapper.cc (right):

https://codereview.webrtc.org/3000993002/diff/40001/webrtc/sdk/android/src/jn...
webrtc/sdk/android/src/jni/videodecoderwrapper.cc:75: qp_parsing_enabled_ =
true;
On 2017/08/16 13:57:58, kthelgason wrote:
> It's quite unclear what this depends on, can we try to just set it in one
place
> and make it clearer on what the decision is based? If it's not possible to do
> that  in code consider adding a comment.

I added comments.

[kthelgason, 2017-08-17 07:31:28]

lgtm

[sakal, 2017-08-17 07:32:41]

The CQ bit was checked by sakal@webrtc.org

[commit-bot: I haz the power, 2017-08-17 07:32:44]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-08-17 08:02:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-08-17 08:02:34]

Try jobs failed on following builders:
  linux_arm64_dbg on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_arm64_dbg/builds/...)

[sakal, 2017-08-17 08:49:52]

The CQ bit was checked by sakal@webrtc.org

[commit-bot: I haz the power, 2017-08-17 08:50:03]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[sakal, 2017-08-17 08:59:33]

Description was changed from

==========
Implement QP parsing in VideoDecoderWrapper.

BUG=webrtc:7760
==========

to

==========
Implement QP parsing in VideoDecoderWrapper.

BUG=webrtc:7760
NOTRY=True
==========

[commit-bot: I haz the power, 2017-08-17 09:01:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-08-17 09:01:13]

Try jobs failed on following builders:
  linux_arm64_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_arm64_rel/builds/...)

[sakal, 2017-08-17 09:15:29]

The CQ bit was checked by sakal@webrtc.org

[commit-bot: I haz the power, 2017-08-17 09:15:33]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-08-17 09:18:05]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1502961329393810,
"parent_rev": "7441827b610a9f7a91105780a2ba08728cb21968", "commit_rev":
"4dee3444936f27c08e736c58c36fa37802298a6f"}

[commit-bot: I haz the power, 2017-08-17 09:18:09]

Description was changed from

==========
Implement QP parsing in VideoDecoderWrapper.

BUG=webrtc:7760
NOTRY=True
==========

to

==========
Implement QP parsing in VideoDecoderWrapper.

BUG=webrtc:7760
NOTRY=True

Review-Url: https://codereview.webrtc.org/3000993002
Cr-Commit-Position: refs/heads/master@{#19378}
Committed:
https://chromium.googlesource.com/external/webrtc/+/4dee3444936f27c08e736c58c...
==========

[commit-bot: I haz the power, 2017-08-17 09:18:10]

Committed patchset #3 (id:60001) as
https://chromium.googlesource.com/external/webrtc/+/4dee3444936f27c08e736c58c...