Closed Bug 1533220 Opened 5 years ago Closed 1 year ago

Enable AudioIPC builds on Windows aarch64

Categories

(Core :: Audio/Video: cubeb, enhancement, P2)

enhancement

Tracking

()

VERIFIED FIXED
113 Branch
Tracking Status
firefox113 --- verified

People

(Reporter: kinetik, Assigned: kinetik)

References

Details

Attachments

(1 file)

Windows AudioIPC started landed in bug 1512445. Unfortunately, it is build time disabled on Windows aarch64 due to legacy winapi 0.2.x dependencies. See bug 1512445 comment 8 for more.

Rank: 15
Priority: -- → P2

FWIW: the 2 projects linked in the above comment use winapi 0.3 since a while.

Depends on: 1708634
Assignee: nobody → kinetik
Status: NEW → ASSIGNED
Depends on: 1726279
No longer blocks: 1432303
Severity: normal → --
Depends on: 1432303
Pushed by mgregan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6ea529a8a1ab
Enable AudioIPC on Windows arm64.  r=cubeb-reviewers,padenot
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 113 Branch

Does this need manual QA? Note that we're not running any Windows ARM64 tests in CI anymore.

Flags: needinfo?(kinetik)

I've done some manual testing locally on a Surface Pro X (SQ1), with everything (audio related) working as expected. More testing would be appreciated if resources permit, though.

Flags: needinfo?(kinetik)
Flags: qe-verify+

Do we need to test something specific or some sanity/exploratory testing around basic audio functionality on top sites, maybe including some WebRTC coverage is enough? Thank you!

Flags: needinfo?(kinetik)

Just sanity/basic audio testing would be enough I think, thanks!

Flags: needinfo?(kinetik)

I've run an audio sanity check on RC 113 with Windows 10 ARM. I've used several popular websites providing media content (youtube, facebook, netflix, twich, spotify, tunein, hypem), a WebRTC call (jitsi meet) and several different audio format (MP3, OGG, FLAC, ACC). I didn't encountered any issue, therefore, I will mark this work as verified fixed.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: