The audio backend of Firefox on Mac OS X.
Перейти к файлу
Paul Adenot 214400933b
Fix device reinit when no input/output device are present (#173)
* Revert 8c1c57a, keeping the changes in 8b7d464

* Really attempt to create a device_info when reiniting to ensure a valid device is availble.

* Let the OS error out and propagate the error instead of asserting

* Revert "Let the OS error out and propagate the error instead of asserting"

This reverts commit a5fe4208cc.

* Revert "Really attempt to create a device_info when reiniting to ensure a valid device is availble."

This reverts commit 38896988d7.

* Revert "Revert 8c1c57a, keeping the changes in 8b7d464"

This reverts commit 0a82ab619f.

* Fix device reinit when no input/output device are present

Co-authored-by: Chun-Min Chang <chun.m.chang@gmail.com>
2022-04-12 18:35:15 -07:00
.circleci Update xcode version on circle ci (#146) 2021-12-29 09:50:30 -08:00
.githooks Revise prepush when deleting branch (#76) 2020-04-03 10:06:55 -07:00
.github/workflows Add GitHub Action workflows (#128) 2021-03-09 17:43:32 -08:00
coreaudio-sys-utils Simplify the event logs 2022-03-14 09:35:44 -07:00
src Fix device reinit when no input/output device are present (#173) 2022-04-12 18:35:15 -07:00
.editorconfig Add editorconfig 2018-09-10 11:23:29 -07:00
.gitignore create a empty cargo library crate 2018-09-10 11:23:03 -07:00
.travis.yml Bail if `cargo clippy` or `cargo fmt` fails (#65) 2020-03-25 15:38:33 -07:00
Cargo.toml Bump cubeb to 0.10 2022-03-22 11:31:30 +01:00
LICENSE Add Readme and License 2018-09-10 11:18:00 -07:00
README.md Replace FIXIT by FIXME 2022-03-14 09:35:44 -07:00
build-audiounit-rust-in-cubeb.sh Update links (#134) 2021-07-29 09:23:35 -07:00
install_git_hook.sh Add a hook to check that commits are formatted and linted correctly (#66) 2020-03-25 17:45:33 -07:00
install_rustfmt_clippy.sh Run `cargo fmt` and make scripts executable (#62) 2020-03-24 07:58:20 -07:00
run_device_tests.sh Don't switch input when using non-default device 2022-03-14 09:35:44 -07:00
run_sanitizers.sh Add GitHub Action workflows (#128) 2021-03-09 17:43:32 -08:00
run_tests.sh Remove docs for `test_add_then_remove_listeners` 2020-10-15 11:15:27 -07:00
todo.md Return error for default device if it's `kAudioObjectUnknown` (#167) 2022-03-29 09:55:07 -07:00

README.md

cubeb-coreaudio-rs

CircleCI Build & Test

Rust implementation of Cubeb on the MacOS platform.

Current Goals

  • Keep refactoring the implementation until it looks rusty! (it's translated from C at first.)

Status

This is now the Firefox's default audio backend on Mac OS.

Install

Install cubeb-coreaudio within cubeb

Run the following command:

curl https://raw.githubusercontent.com/mozilla/cubeb-coreaudio-rs/trailblazer/build-audiounit-rust-in-cubeb.sh | sh

Other

Just clone this repo

Test

Please run sh run_tests.sh.

Some tests cannot be run in parallel. They may operate the same device at the same time, or indirectly fire some system events that are listened by some tests.

The tests that may affect others are marked #[ignore]. They will be run by cargo test ... -- --ignored ... after finishing normal tests. Most of the tests are executed in run_tests.sh. Only those tests commented with FIXME are left.

Git Hooks

You can install git hooks by running install_git_hook.sh. Then pre-push script will be run and do the cargo fmt and cargo clippy check before the commits are pushed to remote.

Run Sanitizers

Run AddressSanitizer (ASan), LeakSanitizer (LSan), MemorySanitizer (MSan), ThreadSanitizer (TSan) by sh run_sanitizers.sh.

The above command will run all the test suits in run_tests.sh by all the available sanitizers. However, it takes a long time for finshing the tests.

Device Tests

Run run_device_tests.sh.

If you'd like to run all the device tests with sanitizers, use RUSTFLAGS="-Z sanitizer=<SAN>" sh run_device_tests.sh with valid <SAN> such as address or thread.

Device Switching

The system default device will be changed during our tests. All the available devices will take turns being the system default device. However, after finishing the tests, the default device will be set to the original one. The sounds in the tests should be able to continue whatever the system default device is.

Device Plugging/Unplugging

We implement APIs simulating plugging or unplugging a device by adding or removing an aggregate device programmatically. It's used to verify our callbacks for minitoring the system devices work.

Manual Test

  • Output devices switching
    • $ cargo test test_switch_output_device -- --ignored --nocapture
    • Enter s to switch output devices
    • Enter q to finish test
  • Device collection change
    • cargo test test_device_collection_change -- --ignored --nocapture
    • Plug/Unplug devices to see events log.
  • Manual Stream Tester
    • cargo test test_stream_tester -- --ignored --nocapture
      • c to create a stream
      • d to destroy a stream
      • s to start the created stream
      • t to stop the created stream
      • r to register a device changed callback to the created stream
      • v to set volume to the created stream
      • q to quit the test
    • It's useful to simulate the stream bahavior to reproduce the bug we found, with some modified code.

TODO

See todo list

Issues

  • Atomic:
    • We need atomic type around f32 but there is no this type in the stardard Rust
    • Using atomic-rs to do this.
  • kAudioDevicePropertyBufferFrameSize cannot be set when another stream using the same device with smaller buffer size is active. See here for details.

Test issues

  • Fail to run tests that depend on AggregateDevice::create_blank_device with the tests that work with the device event listeners
    • The AggregateDevice::create_blank_device will add an aggregate device to the system and fire the device-change events indirectly.
  • TestDeviceSwitcher cannot work when there is an alive full-duplex stream
    • An aggregate device will be created for a duplex stream when its input and output devices are different.
    • TestDeviceSwitcher will cached the available devices, upon it's created, as the candidates for default device
    • Hence the created aggregate device may be cached in TestDeviceSwitcher
    • If the aggregate device is destroyed (when the destroying the duplex stream created it) but the TestDeviceSwitcher is still working, it will set a destroyed device as the default device
    • See details in device_change.rs

Branches