Closed Bug 1369950 Opened 3 years ago Closed 3 years ago

update aom

Categories

(Core :: Audio/Video: Playback, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: rillian, Assigned: rillian)

References

Details

Attachments

(3 files)

Update to the current master of aom. This fixes a build issue on win64.
Assignee: nobody → giles
Comment on attachment 8874069 [details]
Bug 1369950 - Update av1 mimetype when vendoring.

https://reviewboard.mozilla.org/r/145524/#review149512

LGTM

::: python/mozbuild/mozbuild/vendor_aom.py:103
(Diff revision 1)
> +        filename = mozpath.join(self.topsrcdir,
> +                'dom/media/platforms/agnostic/AOMDecoder.cpp')
> +        with open(filename) as f:
> +            source = f.read()
> +
> +        # Substitue the new revision in lines like

'Substitue' -> 'Substitute'
Attachment #8874069 - Flags: review?(gsquelart) → review+
Comment on attachment 8874070 [details]
Bug 1369950 - Remove aom_codec_stream_info_t sz field references.

https://reviewboard.mozilla.org/r/145526/#review149514
Attachment #8874070 - Flags: review?(gsquelart) → review+
Comment on attachment 8874071 [details]
Bug 1369950 - Update aom.

https://reviewboard.mozilla.org/r/145528/#review149516
Attachment #8874071 - Flags: review?(gsquelart) → review+
Priority: -- → P3
Comment on attachment 8874069 [details]
Bug 1369950 - Update av1 mimetype when vendoring.

https://reviewboard.mozilla.org/r/145524/#review149512

> 'Substitue' -> 'Substitute'

Thanks!
Comment on attachment 8874069 [details]
Bug 1369950 - Update av1 mimetype when vendoring.

https://reviewboard.mozilla.org/r/145524/#review149846

::: python/mozbuild/mozbuild/vendor_aom.py:106
(Diff revision 2)
> +            source = f.read()
> +
> +        # Substitute the new revision in lines like
> +        #   version.AppendLiteral("abcdef0124");
> +        prefix = 'version.AppendLiteral'
> +        new_source = re.sub(prefix + '\("[a-f0-9]+"\);',

Maybe make this match the exact number of characters in the commit hash, rather than just `+`?
Attachment #8874069 - Flags: review?(nfroyd) → review+
Pushed by rgiles@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e709485195e6
Update av1 mimetype when vendoring. r=froydnj,gerald
https://hg.mozilla.org/integration/autoland/rev/67152605b367
Remove aom_codec_stream_info_t sz field references. r=gerald
https://hg.mozilla.org/integration/autoland/rev/12d353fc5067
Update aom. r=gerald
You need to log in before you can comment on or make changes to this bug.