1397440 - log AOMDecoder unhandled frame types

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, enhancement)

Description

[Ralph Giles (:rillian)]

We check the aom_image_t types for unsupported ones, but we should log when we see one.

Comment 1

[Navid Mehregani [:Navid]]

Attachment: Bug 1397440 - Added a very useful comment

This is a practice commit to clarify what arguments we're accepting.

Review commit: https://reviewboard.mozilla.org/r/177028/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/177028/

Comment 2

[Ralph Giles (:rillian)]

Comment on attachment 8905241 [details]
Bug 1397440 - Added a very useful comment

https://reviewboard.mozilla.org/r/177028/#review182030

::: js/examples/jorendb.js:4
(Diff revision 1)
>  /* -*- indent-tabs-mode: nil; js-indent-level: 4 -*-
>   * vim: set ts=8 sw=4 et tw=78:
>   *
> + * Testing new line

Please remove this unrelated change.

Comment 3

[Navid Mehregani [:Navid]]

Comment on attachment 8905241 [details]
Bug 1397440 - Added a very useful comment

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/177028/diff/1-2/

Comment 4

[Ralph Giles (:rillian)]

Comment on attachment 8905241 [details]
Bug 1397440 - Added a very useful comment

https://reviewboard.mozilla.org/r/177028/#review182032

Looks good, thanks!

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9b0cf253ad1f
Added a very useful comment r=rillian

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/9b0cf253ad1f