Closed
Bug 1251502
Opened 8 years ago
Closed 8 years ago
Update in-tree libcubeb
Categories
(Core :: Audio/Video: cubeb, defect, P1)
Core
Audio/Video: cubeb
Tracking
()
RESOLVED
FIXED
mozilla48
People
(Reporter: kinetik, Assigned: padenot)
References
Details
Attachments
(7 files, 11 obsolete files)
1.07 KB,
patch
|
jesup
:
review+
|
Details | Diff | Splinter Review |
873 bytes,
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
1.66 KB,
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
1.31 KB,
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
1.75 KB,
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
143.70 KB,
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
51.65 KB,
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
I've removed the stdint.h generation from libcubeb upstream since it was only needed for old MSVC (pre-2012) and updated the Gecko build to suit. https://treeherder.mozilla.org/#/jobs?repo=try&revision=00adfdb4b66a This also pulls in Paul's resampler rework. It might be good to get that landed sooner rather than later to expose it to more testing (if Paul's okay with that).
Attachment #8723912 -
Flags: review?(padenot)
Reporter | ||
Comment 1•8 years ago
|
||
Note for when I get back to this on Monday: Breakage on Win32: update.sh needs to include cubeb_resampler_internal.h Breakage on Android: OpenSL backend needs updating for resampler API changes
Reporter | ||
Comment 2•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=26910a68553b
Reporter | ||
Comment 3•8 years ago
|
||
The use of unique_ptr that I suggested breaks on Android... https://groups.google.com/forum/#!topic/mozilla.dev.platform/ph-63xS3zwA ...because using 5+ year old C++ features is too much to ask. With polyfill: https://treeherder.mozilla.org/#/jobs?repo=try&revision=cb761062f8cb
Reporter | ||
Updated•8 years ago
|
Attachment #8723912 -
Attachment is obsolete: true
Attachment #8723912 -
Flags: review?(padenot)
Reporter | ||
Comment 4•8 years ago
|
||
Forgot to define CUBEB_GECKO_BUILD in the src/moz.build file: https://treeherder.mozilla.org/#/jobs?repo=try&revision=67be7ba3afe4
Reporter | ||
Comment 5•8 years ago
|
||
These warnings in the resampler need investigating: 01:32:56 INFO - /home/worker/workspace/build/src/media/libcubeb/src/cubeb_resampler.cpp:237:11: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] 01:32:57 INFO - if (got != output_frames_before_processing) { 01:32:57 INFO - /home/worker/workspace/build/src/media/libcubeb/src/cubeb_resampler.cpp:174:8: warning: variable 'got' set but not used [-Wunused-but-set-variable] 01:32:57 INFO - long got = 0; 1:32:57 INFO - /home/worker/workspace/build/src/media/libcubeb/src/cubeb_resampler.cpp:150:11: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] 01:32:57 INFO - if (got != output_frames_before_processing) { 01:32:57 INFO - ^ 01:32:57 INFO - /home/worker/workspace/build/src/media/libcubeb/src/cubeb_resampler.cpp:135:7: warning: variable 'out_unprocessed' set but not used [-Wunused-but-set-variable] 01:32:57 INFO - T * out_unprocessed = nullptr;
Flags: needinfo?(padenot)
Reporter | ||
Comment 6•8 years ago
|
||
Also: 17:59:18 INFO - Thread 89 (crashed) 17:59:18 INFO - 0 xul.dll!noop_resampler::fill(void *,long *,void *,long) [cubeb_resampler.cpp:67be7ba3afe4 : 65 + 0x6] noop_resampler::fill can't handle a null input_frames_count.
Reporter | ||
Comment 7•8 years ago
|
||
Last try run has a bunch of other crashes and test failures that need investigating too. Not planning to invest any more time on this right now.
Assignee: kinetik → nobody
Status: ASSIGNED → NEW
Assignee | ||
Comment 8•8 years ago
|
||
Thanks, I'll finish this.
Assignee: nobody → padenot
Flags: needinfo?(padenot)
Assignee | ||
Comment 9•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=155a2c267d96
Assignee | ||
Comment 10•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=608dc7107c36
Assignee | ||
Comment 11•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=6ad53803d30e
Assignee | ||
Comment 12•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=fad01b459ee0
Assignee | ||
Comment 13•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=97ec1977ff4f
Assignee | ||
Comment 14•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=12f19e05d62f
Comment 15•8 years ago
|
||
Includes all the full duplex work, including the resampler.
Rank: 10
Priority: -- → P1
Assignee | ||
Comment 16•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e7032e710174
Comment 19•8 years ago
|
||
It is uploaded here to make it available to padenot. It contains a fix to select the devid instead of device_id during the device selection in webrtc engine.
Comment 20•8 years ago
|
||
Changes to media/libcubeb/include/moz.build
Comment 21•8 years ago
|
||
Comment 22•8 years ago
|
||
Comment 23•8 years ago
|
||
Comment 25•8 years ago
|
||
Attachment #8734713 -
Attachment is obsolete: true
Comment 26•8 years ago
|
||
Attachment #8734714 -
Attachment is obsolete: true
Comment 27•8 years ago
|
||
Attachment #8734717 -
Attachment is obsolete: true
Assignee | ||
Comment 28•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=6fa0d8210f7c
Assignee | ||
Comment 29•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c8d29b5da711
Assignee | ||
Comment 30•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c898fe57aa33
Assignee | ||
Comment 31•8 years ago
|
||
Attachment #8734801 -
Flags: review?(rjesup)
Assignee | ||
Comment 32•8 years ago
|
||
Attachment #8734802 -
Flags: review?(kinetik)
Assignee | ||
Comment 33•8 years ago
|
||
Attachment #8734803 -
Flags: review?(kinetik)
Assignee | ||
Comment 34•8 years ago
|
||
Attachment #8734804 -
Flags: review?(kinetik)
Assignee | ||
Comment 35•8 years ago
|
||
Attachment #8734805 -
Flags: review?(kinetik)
Assignee | ||
Updated•8 years ago
|
Attachment #8734785 -
Attachment is obsolete: true
Assignee | ||
Comment 36•8 years ago
|
||
Attachment #8734807 -
Flags: review?(kinetik)
Assignee | ||
Updated•8 years ago
|
Attachment #8734805 -
Attachment is obsolete: true
Attachment #8734805 -
Flags: review?(kinetik)
Assignee | ||
Comment 37•8 years ago
|
||
This imports the wasapi backend, the resampler code, and various other commits. The OSX code is split to not completely destroy the blame info.
Attachment #8734808 -
Flags: review?(kinetik)
Assignee | ||
Comment 38•8 years ago
|
||
This is the OSX part.
Attachment #8734809 -
Flags: review?(kinetik)
Assignee | ||
Updated•8 years ago
|
Attachment #8734781 -
Attachment is obsolete: true
Assignee | ||
Updated•8 years ago
|
Attachment #8734769 -
Attachment is obsolete: true
Assignee | ||
Updated•8 years ago
|
Attachment #8734712 -
Attachment is obsolete: true
Assignee | ||
Updated•8 years ago
|
Attachment #8734313 -
Attachment is obsolete: true
Updated•8 years ago
|
Attachment #8734801 -
Flags: review?(rjesup) → review+
Comment 39•8 years ago
|
||
Looks to me like the remaining issues (assuming they're basically just a cubeb import in the main wasapi patch) are all rubber-stamps/build-nits.
Reporter | ||
Updated•8 years ago
|
Attachment #8734802 -
Flags: review?(kinetik) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8734803 -
Flags: review?(kinetik) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8734804 -
Flags: review?(kinetik) → review+
Reporter | ||
Comment 40•8 years ago
|
||
Comment on attachment 8734807 [details] [diff] [review] Update cubeb's udpate.sh script to account for new files. r= Review of attachment 8734807 [details] [diff] [review]: ----------------------------------------------------------------- ::: media/libcubeb/update.sh @@ +33,5 @@ > +cp $1/test/test_resampler.cpp tests/test_resampler.cpp > +cp $1/test/test_duplex.cpp tests/test_duplex.cpp > +cp $1/test/test_record.cpp tests/test_record.cpp > +cp $1/test/test_utils.cpp tests/test_utils.cpp > +cp $1/test/test_devices.c tests/test_devices.cpp Rename it upstream, otherwise it's a pain whenever a reverse-sync is necessary.
Attachment #8734807 -
Flags: review?(kinetik) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8734808 -
Flags: review?(kinetik) → review+
Reporter | ||
Updated•8 years ago
|
Attachment #8734809 -
Flags: review?(kinetik) → review+
Comment 41•8 years ago
|
||
(In reply to Matthew Gregan [:kinetik] from comment #40) > Comment on attachment 8734807 [details] [diff] [review] > Update cubeb's udpate.sh script to account for new files. r= > > Review of attachment 8734807 [details] [diff] [review]: > ----------------------------------------------------------------- > > ::: media/libcubeb/update.sh > @@ +33,5 @@ > > +cp $1/test/test_resampler.cpp tests/test_resampler.cpp > > +cp $1/test/test_duplex.cpp tests/test_duplex.cpp > > +cp $1/test/test_record.cpp tests/test_record.cpp > > +cp $1/test/test_utils.cpp tests/test_utils.cpp > > +cp $1/test/test_devices.c tests/test_devices.cpp > > Rename it upstream, otherwise it's a pain whenever a reverse-sync is > necessary. test_devices is not included in gecko unit test. We could stop coping it.
Comment 42•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/1b22933d97f2 https://hg.mozilla.org/integration/mozilla-inbound/rev/3deb965ce2e4 https://hg.mozilla.org/integration/mozilla-inbound/rev/0e4e9eba11ee https://hg.mozilla.org/integration/mozilla-inbound/rev/030e5bd797cd https://hg.mozilla.org/integration/mozilla-inbound/rev/ca8f6e82829b https://hg.mozilla.org/integration/mozilla-inbound/rev/54534d5c9382 https://hg.mozilla.org/integration/mozilla-inbound/rev/5e1216b00e35
Comment 45•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1b22933d97f2 https://hg.mozilla.org/mozilla-central/rev/3deb965ce2e4 https://hg.mozilla.org/mozilla-central/rev/0e4e9eba11ee https://hg.mozilla.org/mozilla-central/rev/030e5bd797cd https://hg.mozilla.org/mozilla-central/rev/ca8f6e82829b https://hg.mozilla.org/mozilla-central/rev/54534d5c9382 https://hg.mozilla.org/mozilla-central/rev/5e1216b00e35 https://hg.mozilla.org/mozilla-central/rev/b0e1f8344f3e https://hg.mozilla.org/mozilla-central/rev/db4111c66699
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•