Only parse PPS up to PPS and SPS ids in the depacketizater.

Only parse PPS up to PPS and SPS ids in the depacketizater.

BUG=
R=danilchap@webrtc.org

Committed: https://crrev.com/126ee727a083fd99a2bfcc654062478dcccf3a3a
Cr-Commit-Position: refs/heads/master@{#14100}

[stefan-webrtc, 2016-09-07 09:56:34]

stefan@webrtc.org changed reviewers:
+ danilchap@webrtc.org

[stefan-webrtc, 2016-09-07 09:58:34]

The CQ bit was checked by stefan@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 09:58:37]

Dry run: 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, 2016-09-07 10:17:40]

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

[commit-bot: I haz the power, 2016-09-07 10:17:41]

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

[danilchap, 2016-09-07 10:38:51]

lgtm

https://codereview.webrtc.org/2316993002/diff/1/webrtc/common_video/h264/pps_...
File webrtc/common_video/h264/pps_parser.cc (right):

https://codereview.webrtc.org/2316993002/diff/1/webrtc/common_video/h264/pps_...
webrtc/common_video/h264/pps_parser.cc:31: bool PpsParser::ParsePpsIds(const
uint8_t* data,
Function declaration order should match function definition order.
i.e. this one should be after ParsePps

https://codereview.webrtc.org/2316993002/diff/1/webrtc/common_video/h264/pps_...
webrtc/common_video/h264/pps_parser.cc:45: bool
PpsParser::ParsePpsIdsInternal(rtc::BitBuffer* bit_buffer,
and this one after ParseInternal

[stefan-webrtc, 2016-09-07 10:44:55]

Comment addressed.

[stefan-webrtc, 2016-09-07 10:46:35]

Description was changed from

==========
Only parse PPS up to PPS and SPS ids in the depacketizater.

BUG=
==========

to

==========
Only parse PPS up to PPS and SPS ids in the depacketizater.

BUG=
R=danilchap@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/126ee727a083fd99a2bfcc654...
==========

[stefan-webrtc, 2016-09-07 10:46:36]

Committed patchset #2 (id:20001) manually as
126ee727a083fd99a2bfcc654062478dcccf3a3a (presubmit successful).

[commit-bot: I haz the power, 2016-09-07 10:46:37]

Description was changed from

==========
Only parse PPS up to PPS and SPS ids in the depacketizater.

BUG=
R=danilchap@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/126ee727a083fd99a2bfcc654...
==========

to

==========
Only parse PPS up to PPS and SPS ids in the depacketizater.

BUG=
R=danilchap@webrtc.org

Committed: https://crrev.com/126ee727a083fd99a2bfcc654062478dcccf3a3a
Cr-Commit-Position: refs/heads/master@{#14100}
==========