1465299 - Update AudioIPC defaults in CubebUtils.cpp to match actual defaults in all.js

Status: RESOLVED FIXED

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

Description

[Matthew Gregan [:kinetik]]

Bug 1446233 landed the prefs with media.audioipc.pool_size == 2 in all.js but AUDIOIPC_POOL_SIZE_DEFAULT == 1 in CubebUtils.cpp.

Bug 1456115 adjusted media.audioipc.stack_size from 64kB to 256kB in all.js but didn't update AUDIOIPC_STACK_SIZE_DEFAULT in CubebUtils.cpp.

It's slightly less confusing if the defaults match all.js, so let's synchronize the values between the two locations.

Comment 1

[Matthew Gregan [:kinetik]]

Attachment: Bug 1465299 - Update AudioIPC pref defaults in CubebUtils.cpp to match actual defaults in all.js.

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

Comment 2

[Paul Adenot (:padenot)]

Comment on attachment 8981743 [details]
Bug 1465299 - Update AudioIPC pref defaults in CubebUtils.cpp to match actual defaults in all.js.

https://reviewboard.mozilla.org/r/247832/#review253992

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b0c5d0ce10d3
Update AudioIPC pref defaults in CubebUtils.cpp to match actual defaults in all.js.  r=padenot

Comment 4

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/b0c5d0ce10d3