Closed Bug 1625876 Opened 4 years ago Closed 4 years ago

RLBox - Port libOgg usage code in the OGGDemuxer to use the RLBox API

Categories

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

x86_64
Unspecified
enhancement

Tracking

()

RESOLVED FIXED
mozilla77
Tracking Status
firefox77 --- fixed

People

(Reporter: shravanrn, Assigned: shravanrn)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Move to RLBox API to invoke functions in libOgg in the OGGDemuxer so we can easily switch between unsandboxed and sandboxed versions of libGraphite. For now this task will still use the unsandboxed libOgg included in tree.

Blocks: 1626174
Attachment #9136705 - Attachment description: Bug 1625876 - Port libOgg usage code in the OGGDemuxer to use the RLBox API r=froydnj → Bug 1625876 - Port libOgg usage code in the OGGDemuxer to use the RLBox API r=froydnj,padenot
Pushed by padenot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a3117fce845d
Port libOgg usage code in the OGGDemuxer to use the RLBox API r=padenot

Ok, it looks like there are two issues

On android there is a warning which triggers Werr based on my change (easy enough to fix)
On linux and other builds, it looks like I inadvertantly removed an include. Not sure why this passed try the first time.

I have updated both phabs.

Flags: needinfo?(shravanrn)
Pushed by padenot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7382f19a6ade
Port libOgg usage code in the OGGDemuxer to use the RLBox API r=padenot
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: