Enable av1 decoding in Nightly on mac and linux

RESOLVED FIXED in Firefox 55

Status

()

Core
Audio/Video: Playback
RESOLVED FIXED
2 months ago
2 months ago

People

(Reporter: rillian, Assigned: rillian)

Tracking

(Depends on: 1 bug, Blocks: 1 bug)

unspecified
mozilla55
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox55 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

2 months ago
Unblock testing of av1 decoding by enabling it for nightly only on the two of our four platforms where it builds.
Comment hidden (mozreview-request)
(Assignee)

Updated

2 months ago
Assignee: nobody → giles

Comment 2

2 months ago
mozreview-review
Comment on attachment 8875372 [details]
Bug 1370978 - Enable av1 decoding in Nightly on macOS and Linux.

https://reviewboard.mozilla.org/r/146798/#review150852
Attachment #8875372 - Flags: review?(nfroyd) → review+

Comment 3

2 months ago
mozreview-review
Comment on attachment 8875372 [details]
Bug 1370978 - Enable av1 decoding in Nightly on macOS and Linux.

https://reviewboard.mozilla.org/r/146798/#review151518

r+ with micro-nit (that you may choose to ignore if you don't have the time for that, as it's not that important)

::: commit-message-87f16:3
(Diff revision 1)
> +Enable av1 decoding with the aom reference library on nightly
> +build where it's working so we can get wider testing.

To be nit-picky, it seems you don't exactly "Enable av1 ... where it's working", but instead "Enable av1 ... except where it's not working yet".
Attachment #8875372 - Flags: review?(gsquelart) → review+
Comment hidden (mozreview-request)

Comment 5

2 months ago
Pushed by rgiles@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a344042d76e9
Enable av1 decoding in Nightly on macOS and Linux. r=froydnj,gerald

Comment 6

2 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a344042d76e9
Status: NEW → RESOLVED
Last Resolved: 2 months ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Depends on: 1371609
FYI, once it was enabled, coverity found ~50 defects in this component.
I can grant access if needed (and I created a component just for aom code
https://scan.coverity.com/projects/firefox?tab=overview
You need to log in before you can comment on or make changes to this bug.