Issue a parse error on unexpected negative timestamps.

Issue a parse error on unexpected negative timestamps.

We should not pass such buffers outside of FFmpegDemuxer.

BUG=686482
TEST=new unittest.

Review-Url: https://codereview.chromium.org/2699563002
Cr-Commit-Position: refs/heads/master@{#451172}
Committed: https://chromium.googlesource.com/chromium/src/+/25d15671d9766d5f3cb87b7f5ff810cca5b130e9

[DaleCurtis, 2017-02-14 23:43:02]

dalecurtis@chromium.org changed reviewers:
+ liberato@chromium.org

[liberato (no reviews please), 2017-02-15 05:26:46]

lgtm

[liberato (no reviews please), 2017-02-15 05:26:48]

lgtm

[DaleCurtis, 2017-02-15 07:46:22]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2017-02-15 07:46:31]

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-02-15 08:23:35]

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

[commit-bot: I haz the power, 2017-02-15 08:23:36]

Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[DaleCurtis, 2017-02-15 23:43:24]

The CQ bit was checked by dalecurtis@chromium.org

[DaleCurtis, 2017-02-15 23:43:24]

The patchset sent to the CQ was uploaded after l-g-t-m from
liberato@chromium.org
Link to the patchset: https://codereview.chromium.org/2699563002/#ps20001
(title: "Only fail after rebasing timestamps.")

[commit-bot: I haz the power, 2017-02-15 23:44:41]

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-02-16 00:16:05]

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

[commit-bot: I haz the power, 2017-02-16 00:16:07]

Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[DaleCurtis, 2017-02-16 02:38:47]

The CQ bit was checked by dalecurtis@chromium.org

[DaleCurtis, 2017-02-16 02:38:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
liberato@chromium.org
Link to the patchset: https://codereview.chromium.org/2699563002/#ps40001
(title: "Rebase.")

[commit-bot: I haz the power, 2017-02-16 02:39:49]

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-02-16 04:03:54]

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

[commit-bot: I haz the power, 2017-02-16 04:03:55]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[DaleCurtis, 2017-02-16 04:16:39]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2017-02-16 04:17:14]

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-02-16 06:19:07]

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

[commit-bot: I haz the power, 2017-02-16 06:19:08]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[DaleCurtis, 2017-02-17 00:23:40]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2017-02-17 00:24:59]

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-02-17 01:59:14]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487291020741210,
"parent_rev": "1dd4ccf354c3286647e35ed6e02c36de695901fc", "commit_rev":
"25d15671d9766d5f3cb87b7f5ff810cca5b130e9"}

[commit-bot: I haz the power, 2017-02-17 01:59:56]

Description was changed from

==========
Issue a parse error on unexpected negative timestamps.

We should not pass such buffers outside of FFmpegDemuxer.

BUG=686482
TEST=new unittest.
==========

to

==========
Issue a parse error on unexpected negative timestamps.

We should not pass such buffers outside of FFmpegDemuxer.

BUG=686482
TEST=new unittest.

Review-Url: https://codereview.chromium.org/2699563002
Cr-Commit-Position: refs/heads/master@{#451172}
Committed:
https://chromium.googlesource.com/chromium/src/+/25d15671d9766d5f3cb87b7f5ff8...
==========

[commit-bot: I haz the power, 2017-02-17 01:59:57]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/25d15671d9766d5f3cb87b7f5ff8...