Closed Bug 945085 Opened 6 years ago Closed 6 years ago

Improve comments in PlatformDecoderModule.h

Categories

(Core :: Audio/Video, defect)

x86_64
All
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla28

People

(Reporter: cpearce, Assigned: cpearce)

References

Details

(Whiteboard: [qa-])

Attachments

(1 file)

The comments in PlatformDecoderModule.h could be clearer, it would help in understanding how the PlatformDecoderModule fits into things.
Attached patch PatchSplinter Review
Adds comments to PlatformDecoderModule.h, and describes the call model better. Renames CreateAudioDecoder and CreateVideoDecoder to CreateAACDecoder and CreateH264Decoder respectively, so that they reflect what they do.
Attachment #8340864 - Flags: review?(kinetik)
Attachment #8340864 - Flags: review?(kinetik) → review+
https://hg.mozilla.org/mozilla-central/rev/4531393b5af6
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.