Fix signed integers in track_run_iterator.cc

Fix signed integers in track_run_iterator.cc

Sample sizes and durations in MP4 TRUN box are unsigned 32-bit integers
per ISO BMFF Byte Stream Format spec (see ISO/IEC 14496-12 section
8.8.8.2 describing syntax of TrackRunBox). They are correctly declared
as unsigned ints in box_definitions.h, but incorrectly declared as
signed integers in track_run_iterator.cc

Review-Url: https://codereview.chromium.org/2932853002
Cr-Commit-Position: refs/heads/master@{#478396}
Committed: https://chromium.googlesource.com/chromium/src/+/15d1b34174529968cac05a8902c96368687a8fd4

[servolk, 2017-06-09 18:07:06]

The CQ bit was checked by servolk@chromium.org to run a CQ dry run

[servolk, 2017-06-09 18:07:17]

Description was changed from

==========
Fix signed integers in track_run_iterator.cc

Sample sizes and durations in MP4 TRUN box are unsigned 32-bit integers
per ISO BMFF Byte Stream Format spec (see ISO/IEC 14496-12 section
8.8.8.2 describing syntax of TrackRunBox). They are correctly declared
as unsigned ints in box_definitions.h, but incorrectly declared as
signed integers in track_run_iterator.cc
==========

to

==========
Fix signed integers in track_run_iterator.cc

Sample sizes and durations in MP4 TRUN box are unsigned 32-bit integers
per ISO BMFF Byte Stream Format spec (see ISO/IEC 14496-12 section
8.8.8.2 describing syntax of TrackRunBox). They are correctly declared
as unsigned ints in box_definitions.h, but incorrectly declared as
signed integers in track_run_iterator.cc
==========

[servolk, 2017-06-09 18:07:17]

servolk@chromium.org changed reviewers:
+ wolenetz@chromium.org

[commit-bot: I haz the power, 2017-06-09 18:07:46]

Dry run: CQ is trying da patch.

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

[wolenetz, 2017-06-09 18:27:14]

lgtm -- are there any tests that were failing or a related crbug?

[servolk, 2017-06-09 19:34:25]

On 2017/06/09 18:27:14, wolenetz wrote:
> lgtm -- are there any tests that were failing or a related crbug?

No, I just stumbled on this randomly, while investigating an unrelated
Chromecast issue.

[servolk, 2017-06-09 19:34:31]

The CQ bit was unchecked by servolk@chromium.org

[servolk, 2017-06-09 19:34:32]

The CQ bit was checked by servolk@chromium.org

[commit-bot: I haz the power, 2017-06-09 19:34:42]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-06-09 20:29:15]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497036872367180, "parent_rev":
"f4e4ee41880338d33efa261846dcf0fc37579ef8", "commit_rev":
"15d1b34174529968cac05a8902c96368687a8fd4"}

[commit-bot: I haz the power, 2017-06-09 20:29:25]

Description was changed from

==========
Fix signed integers in track_run_iterator.cc

Sample sizes and durations in MP4 TRUN box are unsigned 32-bit integers
per ISO BMFF Byte Stream Format spec (see ISO/IEC 14496-12 section
8.8.8.2 describing syntax of TrackRunBox). They are correctly declared
as unsigned ints in box_definitions.h, but incorrectly declared as
signed integers in track_run_iterator.cc
==========

to

==========
Fix signed integers in track_run_iterator.cc

Sample sizes and durations in MP4 TRUN box are unsigned 32-bit integers
per ISO BMFF Byte Stream Format spec (see ISO/IEC 14496-12 section
8.8.8.2 describing syntax of TrackRunBox). They are correctly declared
as unsigned ints in box_definitions.h, but incorrectly declared as
signed integers in track_run_iterator.cc

Review-Url: https://codereview.chromium.org/2932853002
Cr-Commit-Position: refs/heads/master@{#478396}
Committed:
https://chromium.googlesource.com/chromium/src/+/15d1b34174529968cac05a8902c9...
==========

[commit-bot: I haz the power, 2017-06-09 20:29:27]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/15d1b34174529968cac05a8902c9...