1056534 - Add logging to stubbed out member functions to speed up debugging

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Comment 1

[Matthew Gregan [:kinetik]]

Attachment: Add logging to stubbed out member functions to speed up debugging.

This is ugly in places, but we'll remove it again once we feel we've got decent implementation coverage.

There's a couple of parts to this:
- Move BufferDecoder back to webaudio/ and revert any chances made for SubBufferDecoder
- Rename SubBufferDecoder to SourceBufferDecoder
- Base SourceBufferDecoder on AbstractMediaDecoder directly, and insert logging into any stubbed member functions
- Add some ugly logging into the many stub member functions on MediaSourceResource and SourceBufferResource

It's possible some of the newly added log messages will be too verbose and will need to be removed soon.  The point of the logging is to catch the infrequently called functions that cause difficult to track down bugs.

Comment 2

[cajbir (:cajbir)]

Comment on attachment 8476507 [details] [diff] [review]
Add logging to stubbed out member functions to speed up debugging.

Review of attachment 8476507 [details] [diff] [review]:
-----------------------------------------------------------------

This is going to cause rebase pain so we should land this as soon as we can.

Comment 3

[Matthew Gregan [:kinetik]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b26ca03840c1

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/b26ca03840c1