Closed Bug 933579 Opened 6 years ago Closed 6 years ago

Export IsVideoContentType() to VideoUtils

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)

In my patches for bug 908503 I need to use IsVideoContentType() outside of WMFReader, so we may as well move it to VideoUtils where it can be used elsewhere. We can also move VideoUtils into namespace mozilla, currently not all of it is inside namespace mozilla.
Attached patch PatchSplinter Review
Attachment #825697 - Flags: review?(kinetik)
Comment on attachment 825697 [details] [diff] [review]
Patch

+bool
+IsVideoContentType(const nsCString& aContentType);

No newline between bool and function name for the decl.
Attachment #825697 - Flags: review?(kinetik) → review+
https://hg.mozilla.org/mozilla-central/rev/a833c2e9b95e
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.