1471164 - Update cubeb from upstream to 2968cba

Status: RESOLVED FIXED

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

Description

[Alex Chronopoulos [:achronop]]

Pick commits:
2968cba wasapi: Reduce timeout threshold (#447)
27be938 audiounit: new method on failed stream init to avoid deadlock (BMO 1470113). (#445)
bf2c281 cubeb_winmm.c: Don't define __MSVCRT_VERSION__.

Comment 1

[Alex Chronopoulos [:achronop]]

Attachment: Bug 1471164 - Update cubeb from upstream to 2968cba.

Review commit: https://reviewboard.mozilla.org/r/253064/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/253064/

Comment 2

[Paul Adenot (:padenot)]

Comment on attachment 8987786 [details]
Bug 1471164 - Update cubeb from upstream to 2968cba.

https://reviewboard.mozilla.org/r/253064/#review259996

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6b9607c2eb18
Update cubeb from upstream to 2968cba. r=padenot

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/6b9607c2eb18

Comment 5

[Tom Ritter [:tjr] (OOTO until April)]

The __MSVCRT_VERSION__ fix is something I'd like to backport to esr60... what would be the best way to do that? cherry-pick that individually?

Comment 6

[Paul Adenot (:padenot)]

We usually do the following:
- cherry-pick the patch from trunk, put it in `media/libcubeb`, add it for check in.
- add a line to `media/libcubeb/update.sh`, adding the right patch command call to apply the patch file (like so [0])
- make a regular commit stating the changes. This should include the patch file, the file that has been patch, and the modification to `update.sh`.

This is maybe a slightly unusual process, but the rationale here is to be able to know the base revision of libcubeb that is in the tree, and what exactly have been back-ported (or sometimes added on top, for example for a gecko-specific patch), and all that easily. Let me know if I can be of any help.

[0]: https://hg.mozilla.org/releases/mozilla-esr60/file/tip/media/libcubeb/update.sh#l78

Comment 7

[Jacek Caban]

I attached such esr60 version to bug 1471016.

Comment 8

[Mike Taylor [:miketaylr]]

Alex, can you request beta uplift for this patch (re: https://bugzilla.mozilla.org/show_bug.cgi?id=1470113#c5)

Comment 9

[Alex Chronopoulos [:achronop]]

Attachment: Bug 1471164 - Beta uplift for cubeb crash in stream destroy.

Approval Request Comment
[Feature/Bug causing the regression]: Bug 1466066
[User impact if declined]: crash when request to play audio and the request fail
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]: N/A 
[List of other uplifts needed for the feature/fix]:
It is not needed but I have created this uplift patch on the top of beta uplift for Bug 1469152. I have done that because it would conflict otherwise. If you are not planning to uplift the other bug let me know to update it.
[Is the change risky?]: No
[Why is the change risky/not risky?]: It has been working well on Nightly and it's rather small
[String changes made/needed]:N/A

Comment 10

[Liz Henry (:lizzard) (privacy/fingerprinting team)]

Comment on attachment 8991357 [details] [diff] [review]
Bug 1471164 - Beta uplift for cubeb crash in stream destroy.

Fix for audio crash, it's been on nightly for a while, let's take this for beta 8.

Comment 11

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/77d51fa00f58