Closed Bug 1237540 Opened 5 years ago Closed 5 years ago

Upgrade ffvp8/ffvp9 to current master (n2.9-dev)

Categories

(Core :: Audio/Video: Playback, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
firefox46 --- fixed

People

(Reporter: jya, Assigned: jya)

References

Details

Attachments

(3 files, 1 obsolete file)

Significant improvements and addition to extra vp9 profiles.

Additionally, ffmpeg has received lots of security patches that we've reported in the past couple of months
Attachment #8705113 - Flags: review?(ajones)
Attachment #8705113 - Attachment is obsolete: true
Attachment #8705113 - Flags: review?(ajones)
Attachment #8705112 - Flags: review?(ajones)
Attachment #8705429 - Flags: review?(ajones)
Priority: -- → P1
Blocks: 1237907
Attachment #8705112 - Flags: review?(ajones) → review+
Attachment #8705429 - Flags: review?(ajones) → review+
Comment on attachment 8705430 [details] [diff] [review]
[ffmpeg] P3. Configure FFmpeg PDM to use new mozavcodec version.

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

::: dom/media/platforms/ffmpeg/FFmpegRuntimeLinker.cpp
@@ +192,2 @@
>      case 56:
>      case 55: module = FFmpegDecoderModule<55>::Create(); break;

Suggestion: You could switch the order of case statements while you're here.
Attachment #8705430 - Flags: review?(ajones) → review+
https://hg.mozilla.org/mozilla-central/rev/8daf14085785
https://hg.mozilla.org/mozilla-central/rev/7850b42a8255
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.