933579 - Export IsVideoContentType() to VideoUtils

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

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.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch

Comment 2

[Matthew Gregan [:kinetik]]

Comment on attachment 825697 [details] [diff] [review]
Patch

+bool
+IsVideoContentType(const nsCString& aContentType);

No newline between bool and function name for the decl.

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

Landed with review comment addressed.
https://hg.mozilla.org/integration/mozilla-inbound/rev/a833c2e9b95e

Comment 4

[Carsten Book [:Tomcat]]

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