Fail decodeAudioFileData() when duration is unknown.

Fail decodeAudioFileData() when duration is unknown.

This API as currently written can only work with streams of a
known duration, so explicitly fail instead of misinterpreting
a massive 64-bit value as the actual duration.

BUG=668849
TEST=new unittest and layout test.

Committed: https://crrev.com/f7f6a8ef3024fe72f77d74426e69a8f3fa54c3f5
Cr-Commit-Position: refs/heads/master@{#435118}

[DaleCurtis, 2016-11-29 00:28:21]

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

[Raymond Toy, 2016-11-29 16:18:01]

lgtm.

Do you have a test file for this? If so, it would be great to have a layout test
for this.

[DaleCurtis, 2016-11-29 21:09:42]

Description was changed from

==========
Fail decodeAudioFileData() when duration is unknown.

This API as currently written can only work with streams of a
known duration, so explicitly fail instead of misinterpreting
a massive 64-bit value as the actual duration.

BUG=668849
TEST=none
==========

to

==========
Fail decodeAudioFileData() when duration is unknown.

This API as currently written can only work with streams of a
known duration, so explicitly fail instead of misinterpreting
a massive 64-bit value as the actual duration.

BUG=668849
TEST=new unittest and layout test.
==========

[DaleCurtis, 2016-11-29 21:09:51]

Yup. Done. PTAL.

[Raymond Toy, 2016-11-29 21:16:35]

lgtm.

Thanks for doing this!

[DaleCurtis, 2016-11-29 21:35:26]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2016-11-29 21:36:11]

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, 2016-11-29 23:39:17]

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

[commit-bot: I haz the power, 2016-11-29 23:39:17]

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

[DaleCurtis, 2016-11-29 23:43:20]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2016-11-29 23:44:06]

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, 2016-11-30 01:52:49]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1480463000338440,
"parent_rev": "1d6f321feb987ff4405479cd3b235d29e84eff1d", "commit_rev":
"1c4ffb1ae76f23f207f4b3288d0807b01bb6df8f"}

[commit-bot: I haz the power, 2016-11-30 01:53:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-30 01:58:53]

Description was changed from

==========
Fail decodeAudioFileData() when duration is unknown.

This API as currently written can only work with streams of a
known duration, so explicitly fail instead of misinterpreting
a massive 64-bit value as the actual duration.

BUG=668849
TEST=new unittest and layout test.
==========

to

==========
Fail decodeAudioFileData() when duration is unknown.

This API as currently written can only work with streams of a
known duration, so explicitly fail instead of misinterpreting
a massive 64-bit value as the actual duration.

BUG=668849
TEST=new unittest and layout test.

Committed: https://crrev.com/f7f6a8ef3024fe72f77d74426e69a8f3fa54c3f5
Cr-Commit-Position: refs/heads/master@{#435118}
==========

[commit-bot: I haz the power, 2016-11-30 01:58:54]

Patchset 2 (id:??) landed as
https://crrev.com/f7f6a8ef3024fe72f77d74426e69a8f3fa54c3f5
Cr-Commit-Position: refs/heads/master@{#435118}