Bug: 135278534
Test: Run Perfetto with "screen recording" enabled and execute
a query for the "first_frame" slice
Exempt-From-Owner-Approval: cherry-pick
(cherry-picked from commit c840c0bfee)
Merged-In: I20abf7300e8c46b6e6dc52096db699e95c93297d
Change-Id: I20abf7300e8c46b6e6dc52096db699e95c93297d
Move ExtendedAudioBufferProvider.h to libaudioclient. This is
a more appropriate location because EABP extends AudioBufferProvider.
Move SingleStateQueue.h to libnbaio_mono. This is a more appropriate
location because SSQ is a non-blocking queue.
This allows to remove the dependency of libnbaio on libmedia
which is a good thing because libnbaio provides more low-level
abstractions than libmedia.
Also, replace a dependency of libnbaio on libbinder with
a dependency on libaudiohal header library.
Test: make
Change-Id: Ie48b523790cd8230695ec2e4710e50981b616289
isConfigured() in soft vorbis decoder omx plugin was using
mInputBufferCount to detect if the decoder was configured.
But mInputBufferCount was reset to 0 after flush.
This was prone to a race condition, when flush was called immediately
after start(), before codec could signal correct number of channels and
sampling rate.
isConfigured() is now fixed to check decoder state rather than
mInputBufferCount.
Bug: 137736256
Bug: 138213973
Test: atest android.media.cts.DecoderTest#testDecodeWithEOSOnLastBuffer
Merged-In: Ia1e6b88eca45b46fe20f36627f79cd767b73cc1a
Change-Id: Ia1e6b88eca45b46fe20f36627f79cd767b73cc1a
Header files of libnbaio include files from audio_utils
yet the module did not export include path for those. This was
requiring modules depending on libnbaio[_mono] to add those
include paths into their build files.
Test: make
Change-Id: I8982e04db0f3646ce7945fb55529727766f2dfa1
The file name and path of the media profiles can be changed by
media.settings.xml property. So befor test, we should check
media.settings.xml property.
Bug: 138413934
Test: vts-tradefed run vts -m VtsValidateMediaProfiles
Change-Id: Ib7070cca94b9d637b9a1a175d09baf7b78ec0fd9
Found when use google sw decoder,below CTS case will fail:
CtsMediaTestCases android.media.cts.DecodeAccuracyTest 8,12,24,28,40,44.
Checked the failed case,test video height is not 4 bytes align,
eg 426x182,854x362,1920x818.
Some platform GPU may need height 4 bytes align, so the dst_v,dst_u calculation
should consider height alignment.
Here change lock to lockYCbCr to get the actual y,cb,cr.
Change-Id: Ibbccbad3936ab36ee2c25773367b0fbb67d9226e
Signed-off-by: Jessie Hao <juan.hao@nxp.com>
In previous attempt it was mistakenly applied to cc_library_header
instead of the actual library. Do it correctly this time.
The intention of this change is to hide unintentionally
exported symbols from the library.
Bug: http://b/69603741
Test: make
Change-Id: I47967abea0b39aec05e80823c9eb3fe53f9fa591
This change moves almost everything from libaadio.so to
separate libaaudio_internal.so library. This is done to
avoid exporting symbols which apps are not allowed to use
but have them available for platform libraries by exporting
them from libaaudio_internal.so
Bug: http://b/69603741
Test: make
Merged-In: If93118ed2c266faf2964abf21e17b0b13df493d2
Change-Id: If93118ed2c266faf2964abf21e17b0b13df493d2
As far as I can tell, this is the only place where soong is used
to build renderscript.
Bug: 137365032
Test: make checkbuild
Test: cd frameworks/compile/slang/tests
./slang_tests.py
Test: atest CtsRenderscriptTestCases
Test: CtsRsCppTestCases
Change-Id: I10a23ceee696df7fe363cc5d825e45100565cbdf
Merged-In: I10a23ceee696df7fe363cc5d825e45100565cbdf
unlinkToDeath is no longer required, (this change in behavior is to
avoid leaks) so holding onto ActivityManager here (still calling
unlinkToDeath to avoid a log, but may for instance remove all
unlinkToDeath calls in the future).
Bug: 134576445
Test: boot
Merged-In: I273f77aac2b80ba9be70197cc3842f83a11bbd1c
Change-Id: I273f77aac2b80ba9be70197cc3842f83a11bbd1c
(I haven't switched this to std::string because the code's already been
removed internally, so there's no point.)
Bug: http://b/129068177
Test: treehugger
Change-Id: I5675d65b2f74f06e68a9de570741a88ed1899eca
Merged-In: I723570315285d7c2caadffd71cd2d15efde11819
See build/soong/README.md for more information about soong
Bug: 122331945
Test: treehugger
Test: cd frameworks/av/drm/mediacas/plugins; mma
Change-Id: I7a3ab863c4e589bd0c1954d3735602fed7cf79b4
In ACELP_4t64_fx, when iterating over ind array, check index against
array size to prevent OOB write, log an error if such an access
was about to happen.
Bug: 132647222
Test: atest EncoderTest#testAMRWBEncoders
Change-Id: I33f476d94baec2feffc7bcccd0ad0481b8452518
(cherry picked from commit 82cb46d0d5)
Merged-in: I33f476d94baec2feffc7bcccd0ad0481b8452518
(cherry picked from commit 9a44849c88)
AudioInputDescriptor, AudioOutputDescriptor, and AudioSession used
to reference AudioMix instances using a raw pointer. This isn't
safe as AudioMix was owned by AudioPolicyMix, which is not
referenced by descriptors.
Change AudioMix* pointers in Audio{Input|Output}Descriptor and
AudioSession to wp<AudioPolicyMix> which reflects their
relationship correctly.
To ensure that code does not operate on AudioMix instances
independently from AudioPolicyMix, and to avoid introducing
a lot of getter / setter methods into AudioPolicyMix, make
the latter to inherit AudioMix. This makes sense because
AudioPolicyMix is essentially a ref-counted version of AudioMix.
Bug: 124899895
Test: build and sanity check on marlin,
build marlin with USE_CONFIGURABLE_AUDIO_POLICY := 1
Merged-In: Ic508caedefe721ed7e7ba6ee3e9175ba9e8dc23a
Change-Id: Ic508caedefe721ed7e7ba6ee3e9175ba9e8dc23a
(cherry picked from commit a306e2a357)
audioserver should not link directly the code generated from
IAudioRecord.aidl because it also gets it as a result of
depending on libmedia.so and other libraries.
Linking IAudioRecord.o directly causes violation of ODR
because it contains global data.
Bug: 117507477
Test: make
Change-Id: If165768e9097d1c95980883f1fe6e1aacae97506
Merged-In: If165768e9097d1c95980883f1fe6e1aacae97506
Root cause:
1. Play a Vorbis file, and seek several times
2. Resubmit csd-0, mState is cleared and mInputBufferCount is
set to 1 in SoftVorbis
3. Flush again, mInputBufferCount will not reset to 0 due to
mState is null
4. Resubmit csd-0 and csd-1, csd-0 will be used to do
_vorbis_unpack_books due to mInputBufferCount is 1, and
error occurs
Solution: Reset mInputBufferCount in onPortFlushCompleted
Bug: 133472577
Test: Play a Vorbis file, seek several times, and check if the
device can play it normally
Change-Id: I4feff27ffb8a69b17b8009ca91350339691c5100
In order for the build system to track updates to the header files
during incremental builds, always specify the src files using the same
path as the package for C++ compilations.
Bug: 112114177
Test: m
Change-Id: I5fb25124a26f190c462e2e60fc75a88d48643c10
Merged-In: I5fb25124a26f190c462e2e60fc75a88d48643c10