Merge "Notify when prefetching fails" into gingerbread

Notify when prefetching fails

There is no event in the SLPrefetchStatus event callback in
 OpenSL ES 1.0 to indicate a prefetch error. So we signal it by
 sending simulataneously two events:
  - SL_PREFETCHEVENT_FILLLEVELCHANGE with a level of 0
  - SL_PREFETCHEVENT_STATUSCHANGE with a status of
      SL_PREFETCHSTATUS_UNDERFLOW

Change-Id: I399ea3474e6cc7796c940401306ca9a076fac4e2

Workaround for build error on sim-eng

This is not a good long-term fix.

target SharedLib: libOpenSLES (out/host/linux-x86/pr/sim/obj/SHARED_LIBRARIES/libOpenSLES_intermediates/LINKED/libOpenSLES.so)
out/host/linux-x86/pr/sim/obj/SHARED_LIBRARIES/libOpenSLES_intermediates/IAndroidEffect.o:(.data.rel.ro._ZTIN7android12SortedVectorINS_16key_value_pair_tImPNS_11AudioEffectEEEEE[typeinfo for android::SortedVector<android::key_value_pair_t<unsigned long, android::AudioEffect*> >]+0x10): undefined reference to `typeinfo for android::SortedVectorImpl'
collect2: ld returned 1 exit status
make: *** [out/host/linux-x86/pr/sim/obj/SHARED_LIBRARIES/libOpenSLES_intermediates/LINKED/libOpenSLES.so] Error 1

Change-Id: Iea6678d31d0f747651285e76f2c2e616937bfbd7

Add support for 8-bit and 48 kHz PCM

Change-Id: I9e4ab432dc966d0eb531457b0ca2661d0626255a

Fix build warnings

Change-Id: I449da85b97901a632591dd16a52cb924a2722883

Test cleanup

Fix bug in QueryEffect test - was not re-initializing the name length.
Add QueryEffect test for over-writing the name buffer.
Don't create output with VOLUME or NULL interfaces.
Use <stdlib.h> symbolic names for exit codes instead of 0 and 1.
Remove Android-style logging from examples; use only printf.
Remove extra blank lines in test output.
Remove the MAX_NUMBER_OUTPUT_DEVICES macro.
Use <unistd.h> to declare usleep.
Remove obsolete #include statements.
Fix build warnings.

Change-Id: I8d198a317246b3894f104edf7c9f317a51593d07

More tests and bug fixes

New tests:
 - urimime.c tests audio player with URI data source and MIME data format
 - dim.c tests DynamicInterfaceManagement
 - engine.c new test cases:
  - create more than one engine simultaneously
  - exit application without destroying engine first

Bug fixes:
 - Can't request SLBufferQueueItf or its alias SLAndroidSimpleBufferQueueItf
   if the data source is not a buffer queue.
 - BufferQueue interface is explicit on AudioPlayer.
 - OutputMixExt is unavailable (internal) on AudioPlayer.
 - There was a nested mutex lock when resuming a non-suspended interface.
 - This implementation supports at most one engine, so check for that
 - Class configuration table had wrong entry for DynamicInterfaceManagement
   on 4 of 10 classes.

Restore most of the infrastructure for removing interfaces (code review):
 - DynamicInterfaceManagement is included, but body of RemoveInterface is compiled
   out, and replaced by FEATURE_UNSUPPORTED until all our deinit hooks are clean.
 - Remove hook must be called with mutex locked, rather than unlocked as before.
   The remove hook is called during IDynamicInterfaceManagement::RemoveInterface
   and Object::Destroy, so the entry conditions must be the same.
 - Add NULL remove hooks to the interface hook table.

Change-Id: I328dd254ed24a32b220996a6d9592cc8c0f4e943
Miscellaneous:
 - Always memset object to garbage in Object::Destroy to catch broken applications.
 - Add comments about how GetInterface manipulates the v-table pointer (code review).
 - Re-enable DynamicInterfaceManagement for testing the example code.

Interface testing and bug fixes

Add some new tests, and fix the bugs they found.

New and improved tests:
Added output mix API test with focus on interfaces.
Add mute solo and seek test cases to the automated buffer queue test.
Buffer queue configuration test is now listenable:
  Buffer queue configuration test now generates a sequence of sine
  waves in order so that the listener can tell if each format is
  supported correctly.

Bug fixes:
Engine::Create... was not checking if the interface was available
  for a particular class.  Fixing that bug showed some other bugs
  in the class/interface configuration tables.
The effects interfaces is available on an output mix.
The Volume interface is not available on an output mix.
The Object interface can do GetInterface before realized.
OutputMix interface is available on an OutputMix;
  needed by the demos/examples in back of book
OutputMix::ReRoute fails if number of output device IDs is not 1.
Improve interface checks on a buffer queue source.
Don't allow mute solo interface on a mono buffer queue.
The earlier code to check for seek interface on a buffer queue
  compared by address, which didn't work if application
  used a private copy of the GUID. Now compares using the
  MPH which will work in all cases.
Buffer queue, effect send, and mute solo are explicit interfaces on audio player.
slCreateEngine was not publishing the engine object.

Miscellaneous cleanup:
Removed the hard-coded fake device ID constants in
  OutputMix::ReRoute that were there just for testing.
In  Engine::Create..., log an error for all unsupported interfaces, not just the first one.
Make hash generator build again:
  It turns out interfaces.c is needed, even though it duplicates
  OpenSLESUT.c.  Restore from 1a6bb4f8e738c9387dc9629db294ea5de618a53c.
Remove some redundant comments.

Change-Id: Iddabe73e298b69a44f3b43ed224c918eb95961a1

Object an interface initialization/destroy cleanup

Prior to this, object and interface fields were being initialized
and deinitialized in random places. This resulted in some real and
potential leaks, especially in the case of partial object construction.
There were even some places where fields were not being initialized
at all.

Details:

Move destructor for AndroidEffectCapabilities from CEngine::Destroy
  to IAndroidEffectCapabilities deinit hook.
Move destructor for equalizer preset names from CEngine::Destroy
  to IEngine deinit hook.
In Object::Destroy, call deinitializer for each initialized interface,
  regardless whether it was ever exposed.
Remove double initialization of mStepSize in IPlaybackRate constructor.
Remove obsolete logs.
Add expose hooks that are called when interface is requested in
  Engine::Create...  or by DynamicInterface::AddInterface.
Always initialize all interfaces, even if not exposed
Fix typos in same-line comments on INTERFACE_ constants.
Add new interface state INITIALIZED: uninitialized -> initialized -> exposed.
DynamicInterface::Remove sets interface state to initialized (but
  not exposed), instead of uninitialized.
Add explicit constructor calls to initialize fields in C structs
  (e.g. smart pointers), using placement new. See also build warning
  which was turned off with -Wno-invalid-offsetof.
Add comments about per-instance fields not associated with an interface.
Add default initializations for all per-instance fields that are
  not associated with an interface, in case the object is only partially
  constructed and then needs to be destroyed due to an error.
Add placement new comments.
Update comments on private per-instance fields not associated with an interface.
Move effect clears from the containing object destructor
  (CAudioPlayer_Destroy and COutputMix_Destroy) to the interface
  destructor. In this process this caught a bug that environmental
  reverb and preset reverb were not being destroyed properly for an
  audio player.
Add comments on interface hooks.
Add entries for interface destructors.
Add placement new (explicit constructor) and explicit destructor calls
  for C++ fields in C struct.
Fix template instantiation error.
Move explicit calls to IBufferQueue_Destroy (now renamed to
  IBufferQueue_deinit) from CAudioPlayer_Destroy and CAudioRecorder_Destroy
  to be implicit via the BufferQueue interface deinit hook.
ThreadSync interface destructor now logs a warning if
  ThreadSync::EnterCriticalSection was active at Engine::Destroy
Add some comments.
Clean up gMonitorFp close code.
Fix typo in log message.
Fix DynamicInterface to handle distinction between UNINITIALIZED and
  INITIALIZED states for an interface, and to call the optional expose hook.
Do not use memset per interface, because the size of the last interface is unknown.
Set the v-table pointer while lock is held.
Add curlies to if statements.
Log errors if interfaces active during Object::Destroy, instead of asserting.
Log a warning if interface requests an interface but it is not available,
  and is optional.
Add log with the class name and address of each new object.
Mark only interfaces with init hooks as INITIALIZED; others are UNINITIALIZED.

Change-Id: Ibb9c1b5f21e6195af6b20b2f7a5c7157905b735b

Add new test, fix bugs found by test & code review

Bugs:
 - EnvironmentalReverb::GetEnvironmentalReverbProperties left mutex
   unlocked at end.
 - buffer queue configuration test revealed not checking enough
   combinations
 - would have failed conformance test in a couple places
 - replace peek locks by shared locks for SMP
 - undocumented symbols were being exported by library
 - fix handling of EffectSend for MIDI player

New test:
 - Add reverb test

Miscellaneous:
 - Move some FIXME to to-do list where they are easier to organize
 - Fix typos in name of profile constant
 - EffectSend::IsEnabled sets enabled = false for unsupported aux effect.
 - Add curlies to if statements.
 - Add more error logging.
 - Fix indentation in switch statement.
 - Add comments to makefile.

Details:
 - Reduce visibility of libOpenSLESUT symbols.
 - Use LOCAL_STATIC_LIBRARIES := libOpenSLESUT where needed in
   applications.
 - Move FIXME for OEM configuration, interface, SDL, Sndfile,
   buffer queue, destroy to to-do list
 - For SDL, disallow buffer queue with source not 44.1 kHz because it
   is hard-wired to 44.1 kHz and there is no sample rate converter.
 - return successful result for enabled effect send on unsupported
   MIDI player
   for future conformance testing

Change-Id: I39cb543169996f78c16e3ac363ea6157a9242743

Merge "Add buffer queue configuration test" into gingerbread

Add buffer queue configuration test

Add comments to makefile for each test

Change-Id: I8741d93e7aadb02c4b75e3a0a7a81d593dfa7f97

Lock tweak between interface and object

When locking the interface for the prefetch status, lock the
 object instead of the prefetch interface, as it updates
 the android object state.

Fix the playState test app.

Change-Id: Ib4e0e99a4cf0720e2b7ebd920fe92f69377f19d9

Code cleanup: function name unification.

Change-Id: I37790738d81bb8d6b5496467a93c17c6f1385b8f

Fix issues with SLPlaybackRateItf interface

Set rate was not applied when the AudioTrack is created after
 prepare completed.
Modified playback rate functions in AudioPlayer to pass a
 CAudioPlayer pointer, rather than a pointer to IPlaybackRate
 so the rate update function can be queried from the CAudioPlayer.
Added test that exercises the SLPlaybackRateItf interface as
 well as content looping.
Simplified evaluation of delay between decode and render to
 use a fixed (small) delay that balances time to run the
 event looper, and prevents clicks.

Change-Id: I598e9a96e93ef7a7c3603c8467f9d6b36dacd26d

Merge "Add MuteSolo checks for audio player and fix bugs" into gingerbread

Add MuteSolo checks for audio player and fix bugs

The specification says "This [SLMuteSoloItf] interface cannot be
exposed on a player whose audio format is mono." This has a lot of
implications ...

Deny an explicit interface request for SLMuteSoloItf at audio player
creation time if the player is known pre-realize to have mono channel
count (e.g. PCM buffer queue).

Deny a GetInterface on MuteSolo if channel count is known to be mono,
but allow it if the channel count is still unknown.

When DynamicSource (or its replacement) is implemented, it will be
possible for the application to have done a GetInterface successfully,
and then to change the source such that the MuteSolo interface is
no longer valid.  In that case, return SL_RESULT_FEATURE_UNSUPPORTED
for most MuteSolo operations, if called when channel count is unknown
or is mono.  We _do_ allow MuteSolo::GetChannelCount to succeed if
channel count is known to be mono.

Enhance the URI-based mute solo test to check some of the above,
and the interactive buffer queue test to get and test the channel count.

Add curly braces to if statements.

Other bug fixes:
 - Do not rely on peek locks; they don't work on SMP.
 - Channel count and sample rate were not being set correctly
   for buffer queue data source audio players.
 - CreateAudioPlayer and AudioRecorder were both referring to
   application pointers more than once, with the possibility of
   reading inconsistent data.

Change-Id: Ie0109cbddc0aff8a56c0b53d989fb2be823e627b

Merge "Fix bug 3004701 do not prefetch during Realize." into gingerbread

Fix bug 3004701 do not prefetch during Realize.

Removed unused Android object states in enum, not used in
 play state machine.
Made SfPlayer responsible for the creation of its
 AudioTrack once it knows the sample rate and number of
 channels of the content to play once it's done
 preparing.
Added notification from SfPlayer to signal when it's
 done preparing. This informs of the audio characteristics
 and kicks off the content prefetching.
In the AudioPlayer realize:
 - do not start the prefetching, do it when the player
  is set to PAUSED
 - don not create the AudioTrack, do it when SfPlayer
  is done preparing.

Change-Id: Ic345040e71cef0e5953ea47366930d1dfbc5e761

Enable and test more interface ID queries

Enable Engine::IEngine_QueryNumSupportedInterfaces and
  Engine::QuerySupportedInterfaces.
Add object ID test.
Return SL_RESULT_FEATURE_UNSUPPORTED from Engine::Create... if the
  corresponding class is not configured..
Add slesutObjectIDToString to utility toolkit.

Change-Id: I28b8beff71d65e7eccd558e586652bb606c89848

Add engine interface test

Change-Id: I5c1fd1e8ab6dc050f416f28708e3ef218910cd8d

Merge "Destroy for an unrealized engine was hanging" into gingerbread

Remove old stream type code that's not used anymore.

Change-Id: I66e0d93ffa8fbf8fd64642d5554026513d561153

Destroy for an unrealized engine was hanging

Change-Id: Ib11b15fe40726d2f9aad0c54fa66382824c95cfa

Initialize mEffects if audio player partly created

See https://android-git.corp.google.com/g/70088
File opensles/libopensles/android_AudioPlayer.cpp
Line 1111:     if (NULL != pAudioPlayer->mAndroidEffect.mEffects) {
mAndroidEffect.mEffects is not initialized to NULL.

Change-Id: Ic6b952748c7a8024c797d7bec4a26cb138fa8840

Fix AudioTrack sessionId bug

The sessionId associated with an AudioPlayer is initialized during
 its creation. The sessionId was not passed to the AudioTrack
 constructor.

Change-Id: I9ccafd707313367f8da78013575cd6048637f240

Remove unused Android object type AUDIOTRACK_PUSH

Change-Id: I1ab5494ae47239eed0b0a18dc817e3b0ead37871

Merge "Support content looping for URI and FD data sources." into gingerbread

Support content looping for URI and FD data sources.

Implement SetLoop() on AudioPlayer with URI and FD
 data sources. Only looping over the whole duration
 of the audio content is supported here.

Change-Id: Ic033f61063e57c3a9f8c51bad15852a42b21ab46

Restore the 2 unmerged equalizer tests

Change-Id: I667cfdddd34fff7017fedab9ee84e954c1c9f690

Log which effect initialization function is used.

Change-Id: I7c5c7440e50d65faf3dadf360e7a3a23dc5d7781

Implement buffer cache fill level and status notification.

OpenSL ES requires the notification of buffering status
 changes and caching fill level updates. Fill level
 updates happen only when the level has changed by
 a certain amount.

Change-Id: I8b5e82ac1dfff35d7a27eda0dd1f395311d554f3

Temporary fix for broken build

Change-Id: I1b02cbe6ce22766ccbab4e8bf040418eba69d6fc

Replace one configuration hack by another

Previous hack allowed an application to call GetInterface for any
  interface prior to Realize.  The new hack allows an application
  to do so only for interfaces which have a "pre-realize OK"
  relationship to their associated class.  Currently this is
  SL_IID_ANDROIDCONFIGURATION.
Reduce logging in record test.
Allow default tracing level to be set in makefile.
Line length 100.

Change-Id: Id17c55ee7284dca108e0eab379a8f987963fd8c4

Cleanup

Remove obsolete source file interfaces.c.
Fix typo in conditional compile.
Clarify comment.

Change-Id: I46e93c61fc0d192118a0f17b5b735ec2118298df

Fix problems found during testing

Fix crash in android_audioPlayer_destroy if audio player is only partially
  constructed, e.g. when using a buffer queue source and the buffer count
  is invalid etc.
Add temporary workaround to buffer queue automated test to be quieter if
  hardware volume control is not working.
Make the record test less noisy on console.

Change-Id: Ief255c3f5f3c94c990f589b2373686b2ac526dd3

Fix bug found by BufferQueue automated test

Fix bug for destroying partially constructed audio player.
Remove obsolete logging.

Change-Id: Ib186153bf50ed1311c021d53711ec90d000bdee2

As part of API review, disable incomplete profiles

Don't compile dead code relating to unsupported features, but keep profiles for CT.
Move whole file conditionals from the source code to Android.mk,
  by just not compiling the module for unsupported features.
There must an interface initialization hook present to expose an
  interface implicitly or explicitly at object creation time, or
  to dynamically add an interface after the object is created.  As
  a safety check, we also double-check for the initialization hook
  when the application does a GetInterface.

Miscellaneous bug fixes:
 - Fix warning "variable may be used uninitialized in this function".
 - Fix build errors on SDL.
 - Was using the wrong conditional for SDL.
 - Fix typo mInterfaceContinued -> mInterfaceStates2 in AudioRecorder.
 - Fix typo in IEngineCapabilities_init.

Other cleanup:
 - Move ugly SDL-specific stuff out of the portable code.
 - Combine duplicated code in CAudioPlayerDestroy and
   CAudioRecorder_Destroy for freeing buffer queue memory.
 - Add curlies to if and do statements.
 - Line length 100.
 - Decrease engine voices from 32 to 30 (based on max of 32 objects, 1 engine, 1 output mix).
 - Disable IRecord_init log.
 - Add symbolic link for Android configuration header file.
 - Enable trace on Android.
 - sl_int64_t is now unconditionally defined rather than only if
   ANDROID defined, because it is used by OpenSLES_Android.h which
   might be included by SDL also.
 - Removed symlink for Android headers

Change-Id: I1a6389257bca09e3819923129a1193473e16669b

Android simple buffer queue locator and interface

Bug fixes:
Fix bug in error handling and recovery for checkDataSource - potential memory leak.
Clean up MPH table; there were some missing and uncommented entries.

Testing:
Add comments about using the END macro to test the MPH table.
Convert feedback test from buffer queue to Android simplified
  buffer queue for rx side only, tx is still 1.0.1 buffer queue.
Add option to sample/intbufq test to use build option to decide
  whether to use 1.0.1 buffer queue or Andorid simple buffer queue.
Convert record test for new APIs.

Miscellaneous:
Update hashgen table and add an install rule.
Add comments.
Add more logging.
Line length 100.
Fix build warning.
Double indent.
Fix SL_DATALOCATOR value for FD.

Change-Id: I35cce8934157fed88251e846b1beb15523b2b1a6

Add explicit dependency to utility library.

Change-Id: I3a85b071b355ac9e0c330a71e6457cb01b665af0

Quick fix for build break, real fix to follow

Change-Id: Ib8542a689a0287fe19cd74b14eceb6b4a8b6afeb

Update logging

Fix bug in ifdef __cplusplus for OpenSLES_Android.h.
Clean up, simplify, and change level of a few log messages.
Use SL_LOGx macros instead of LOGx in SfPlayer.cpp.
Add comments regarding unsupported runtime log level checks.
Line length 100.

Change-Id: I52441ed6fbe9550248fa1b6944fc8e199f1f2724

Fix bug 3022251

Remove prefetch status error codes: not officially in OpenSL ES spec.

AndroidEffectCapabilities: no const for effect name

Android file descriptor data source: don't use a value that will
 likely be used by the specification in the future.

Change-Id: I9b4e0d7eb265dc98c76021cddfe5a1f50b5eb6ef

Merge "Fix bug 3011821 OpenSL ES Volume chain configuration" into gingerbread

Fix bug 3011821 OpenSL ES Volume chain configuration

When setting effect send levels, combine player volume level
 with send level since AudioTrack doesn't have a distinct volume
 and direct level. This fix ensures the send level changes
 according to the player level, without being affected by direct
 level changes.
When setting player volume, update the send level.

Change-Id: I372434267a7107c617a0fdd8c39e92e31e4c339b

Merge "Enable semi-standard logging (does not use LOG macros)" into gingerbread

Enable semi-standard logging (does not use LOG macros)

This is a workaround for conflict in Stagefright's definition of LOG.

Change-Id: I985e2c08d6e86a1664654a2e8a0b8aea145b32ba

Fix AudioRecorder initialization error.

CAudioRecorder fields mNumChannels and mSampleRateMilliHz are
 initialized in IEngine.c CreateAudioRecorder(). It is an error
 to reset them in the Create of AudioRecorder.

Change-Id: I61568b225ad1303a3f8b0d846aac8c0b848386f1

Fix bug 3009239

Query AudioRecord position for SLRecordItf->GetPosition()

Change-Id: I1a520b281d44105d34c92fd555e71013d5170c76

Change object state test for SLAndroidConfigurationItf.

Change-Id: Iabd024720f361bcfa50f3a5daa433afa6caa4930

Remove the SLAndroidStreamTypeItf interface.

It is now replaced by SLAndroidConfigurationItf which implements
 the same functionality (and more).

Modified test to demo setting AudioPlayer stream type to use
 SLAndroidConfigurationItf

Modified SLAndroidConfigurationItf to enable passing a
 NULL pointer to value so the value size for the key ca
 be queried.

Fixed type in AudioPlayer and AudioRecorder Android side
 which reported an error if the SetConfig() function was
 called when the object was unrealized (the intent is
 to check that the object is NOT realized because we
 are configuring the resources the object is going to use).

Change-Id: I20f3dba8c63066db94d537109854b4c4139f65b3

Merge "Turn on the opensl samples." into gingerbread

Improve reliability of Object::Destroy / Create

Separated object construction from publishing, for better safety
with respect to the sync and debugger thread.  Unpublished objects
exist and have an ID slot reserved, but the ID is not assigned.
Destroy of an unpublished object does the right thing.

Add pre-destroy checks:

 - COutputMix all platforms: add check for players attached to
   output mix.  Object:: Destroy on an output mix that has at least
   one attached player is a no-op, but will log an error.

 - COutputMix for SDL: now asks the mixer callback to unlink output
   mix from engine, and then blocks until SDL mixer callback
   acknowledges the request; this avoids a race condition where the
   SDL thread calls the mixer during destroy.

 - C3DGroup: add check for players in group.
   Object::Destroy on a 3D group that contains at least one player
   is a no-op, but will log an error.

Object::Destroy on an audio player that is part of a 3D group now
removes the player from the 3D group during destroy.

Remove redundant field:

 - CAudioPlayer.mOutputMix duplicated a field in DataSink.DataLocator_OutputMix;
   removed mOutputMix and replaced by function CAudioPlayer_GetOutputMix
   which gets the output mix from the data sink.

Remove dead code:

 - IOutputMixExt_Destroy was obsolete.

Miscellaneous changes not related to destroy:
 - Fixed an ancient bug in the class table for Listener object -
   was using wrong offset for interfaces.
 - Add curly braces around if, else, for bodies
 - Add parentheses around complex conditionals
 - Add some logging
 - Fix indentation.
 - Line length 100

Change-Id: I0035fe356e2a28c1932b5d265f881377aa77dc58

Turn on the opensl samples.

Change-Id: I0ca645fe506e8690d960bf113472a400cd422ea3

Allow Android stream type to be set by SLAndroidConfigurationItf.

Allow Android stream type to be set through the
 SLAndroidConfigurationItf interface, instead of the
 SLAndroidStreamTypeItf interface, which is going away before
 release.
 The stream type is now stored in CAudioPlayer, and used before
 the AudioTrack of the player is instanciated.

AudioRecorder preset and AudioPlayer stream type also implement
 the following behavior: when querying a key with a NULL pointer
 as destination for the value, the value size is updated with
 the actual size used to store the value for the given key.

Change-Id: I652a5b463a220698c53fc1d2b485143b794a1d75

Add example for SLRecordItf callback registration and use.

Change-Id: Ibcd27f5a49a42ddbcd313b30e7b07b1ce4f13705

Merge "Signal recording buffer overrun with SL_RECORDEVENT_HEADSTALLED." into gingerbread

Signal recording buffer overrun with SL_RECORDEVENT_HEADSTALLED.

Signal recording buffer overrun with SL_RECORDEVENT_HEADSTALLED,
 not SL_RECORDEVENT_BUFFER_FULL because this latter is not
 appropriate when recording to a buffer queue, and implies that
 recording stops when it is hit.

Fix locks in IRecord.c: should be exclusive when changing
 record attributes.

Change-Id: I763483a54c92128e3fba9d9c97a091f727f4f7fc

Make hashgen build again, and update the hashtable

Change-Id: Id0e5a25208adb4cef7f1d04fd3cceaff2a7a4554

Fix bug 3003630

Support record events HEADATMARKER, HEADATNEWPOS, BUFFER_FULL.

In SLPlayItf, remove useless boolean field for marker set and,
 like in SLRecordItf, make ClearMarkerPosition() reset the
 marker to 0.

Change-Id: Iac624d218b79fa7540c371d403de27bc5c291176

Store sessionId in AudioPlayer.

Store sessionId in AudioPlayer so it can be passed directly to
 resources that need it, and the same session is kept during
 the lifetime of the AudioPlayer object.

Change-Id: I97c46c88cfb9cc7a8147909f379716522c695b9d

Fix bug where OutputMix is not fully initialized by CreateOutputMix()

In the implementation of SLEngineItf, CreateOutputMix() didn't call
 android_outputMix_create() which initializes the mEffects pointers.
Added checks on SfPlayer in case an application start calling
 interface functions when Realize() on an AudioPlayer isn't done.

Change-Id: Id3fb9f02b5c29a1d87255e7935542ebcf4b8dd46

Fixed C/C++ interworking problem in IAndroidEffect

The IAndroidEffect structure contains a KeyedVector object
which constructor was not called when the structure was allocated.

The fix consists in changing the structure member to a pointer
to a KeyedVector and create the object from C++ code when the
object containing the IAndroidInterface is created (AudioPlayer
and OutputMix).

Change-Id: I328b4b61537ddb178b33427c6bad06678a29647a

Merge "Add prefetch status error definitions." into gingerbread

Rename default recording preset to generic.

Change-Id: I77144ebcc593001ca60b94f0176fc2287e7400fd

Add prefetch status error definitions.

Add prefetch status error definitions as defined in post SL ES 1.0.1.
Add fixmes for Android extension discovery.

Change-Id: I36161013f121b0fb0748311df9e8167e45f88808

Implement SLAndroidConfigurationItf on AudioRecorder.

Enable the use of the configuration interface before the
 AudioRecorder object is realized to set the recording
 preset. Recording presets map to the Android
 AudioRecord input source.

Change-Id: I804095a0b39e4676862f6bc8e7fbaa0ba63b9336

Reduce .so size and hide internal global symbols

Change-Id: I95b63aa1f4da94ad068619ff0944e4ff83fc6fd6

Fix some race conditions and deadlocks.

Use strong reference counts to fix race conditions in these object dependencies:
 - audio player on 3D Grouping
 - MIDI player (or more specifically, the data sink locator) depends on LED or Vibra device
 - audio player (or more specifically, the data sink locator) depends on output mix
Fix build errors on non-Android platforms.
Fix incorrect result code for EnvironmentalReverb::SetDensity.
Continue work on making Destroy more reliable:
 - Destroy calls a pre-destroy hook, e.g. for Android audio players.
 - Add new internal object state DESTROYING which is set when the pre-destroy hook returns.
 - Abort (called by Destroy) now blocks until all async operations complete or are aborted.
Add curly braces around simple if and for statements.
Add extra parentheses around complex if expression in if statements.
Add more error logging in checkDataLocator.
Add Doxygen comments.
Remove spurious log entry when SL_DATALOCATOR_ANDROIDFD used.
Disable the unused Resume hooks, which are currently unused.
Remove some dead code in audio player create for Android.
Line length 100.
Prepare for fixing race condition in OutputMixExt by
 moving mActiveMask and mTracks from IOutputMix to IOutputMixExt.
Update comments.
Continued work on Destroy reliability:
 Do not poke around in mixer's data structures directly during
  Object::Destroy for an audio player; instead ask the mixer nicely
  and wait for it to do the job.
 Move master copy of computed gains from Track to AudioPlayer, and then
  let the mixer copy the gains locally when it needs them.
Add resume and predestroy hooks everywhere.
Predestroy hook returns a boolean.

Add SLAndroidConfigurationItf, enabled on AudioRecorder.

This adds a generic configuration interface, first enabled on the
 AudioRecorder object. This is intended to be used to configure
 the audio recording source. It is meant to be usable before the
 object is in REALIZED state.

Change-Id: I8f6c1cf8f5be8a8dfb332c4e89d91f45047f48e3

Hide media player looper from AudioPlayer.

The ALooper that takes care of the rendering loop in SfPlayer is
 only used inside SfPlayer. This CL makes SfPlayer manage the
 lifecycle of its rendering looper.

Change-Id: Ia4d5e02ce0a44bcf79021f428b490612a9fead9c

Implement the SLAndroidEffectSendItf.

This enables sends from an AudioPlayer to an OutputMix
that's using the generic effect interface (SLAndroidEffectItf).
Since there is a single aux bus, the effect ID is only used
to verify the effect referred to in the send is actually
attached to the OutputMix. Also there is a single send
level value and enable flag for the interface, even though
the interface definition supports discerning between multiple
effects (so it scales on platform with multiple aux buses).

Dry (direct) level is now stored in the AudioPlayer,
as it is shared between SLEffectSendItf and
SLAndroidEffectSendItf.

Change-Id: I3dd34c4c045d8ac2961a325e10b181a789bf4913

Definition of SLAndroidEffectSendItf

Define the SLAndroidEffectSendItf interface which mirors the
SLEffectSendItf interface, but uses an SLInterfaceID parameter
to identify the effect the AudioPlayer is sending to.

Change-Id: Iec0ddf315a6974fafedecd566bd01c7231183312

Fix aux reverb issues:

- when creating an effect, only pass the effect type to the
 AudioEffect framework so it selects whether to use an insert
 or aux effect based on the session.
- SL ES aux reverb effects don't have an explicit SetEnabled()
 function, so enable the effect upon creation, or based on the
preset for the preset reverb effect.

Add a test for SLEffectSend.

Change-Id: I683331cac19cf1fb9628d636569ab3d342c02820

Fixed the make file which it can build all the sub-dir.
Remove the samples temporary until it's fixed.

Change-Id: I6dd9b15a16c3dcc22ecee1fd1acecaebbd59262f

Fix automated tests testPlayAbsPath and testPlayfilePath

Do not rewind when reaching the end of the stream.

Change-Id: I98fa7cebc2b2d8fa8d10d371cfb78f24fb1ab277

Support SLAndroidEffectItf on the OutputMix object.

Change-Id: I098dc047ce91848781c9ba18beec5a33abd50e68

Finish implementing the SLEnvironmentalReverbItf interface.

Change-Id: I02bbc6a08607c328253257cbdb6a45b675b9d3ff

Cleanup SLAndroidEffectCapabilitiesItf example.

Change-Id: I3b73d089fde7f8ada5387f6e030869356a918b4a

Fix bug 2947094

"multiplay" is a test that creates an AudioPlayer for each file to
 play. When a player reaches the paused state, SetPosition (ISeekItf)
 is called with a value of 0 (beginning of file). The bug was that
 on Android, the player would stop playing after playing the file
 once or twice.
The bug was caused by the logic in ISeek.c which checks that the
 new seek position is different than the previous one before issuing
 a seek command to the player. The fix consists in not checking
 the old requested seek position so the command is always issued.
 It works on the desktop because the SndFile version was resetting
 the mPos field to SL_TIME_UNKNOWN each time SetPosition was called.

 While investigating this bug, it was also found that the player
 should reset its decode position to 0 each time it reaches the end of
 the stream.

Change-Id: I22f2d6480c62ab6e56c28c7ccae2c117888a7639

Fix deadlock when stopping the player while destroying.

Do not acquire an exclusive lock on the player when querying play
 callback and event mask, or updating the playstate.

Not related to fix:
Do not notify the end of the stream has been reached once the player
 is stopped.

Change-Id: I9c7e1d96c28d1472ff18174c97544faa8deb0e88

Store decoded buffer in player instance.

Do not store decoded buffer reference in message sent from decoder
to renderer, but rather in player instance. This allows access
to buffer between decoding and playback so the media source can
be correctly stopped.

Change-Id: I7f583366d9895db039ca7d0d9e22f36d0d429f55

Implement SLAndroidEffectItf and SLAndroidEffectCapabilities.

Change-Id: Ia2cf6e3f0178189b0ed010690bc9906b61a746a7

Fix OpenSL ES effect test durations based on audio file duration.
Fix make file to build the EQ test.

Change-Id: I509d1f0cea775c4f955c8c23fc27ba313995dd5b

Improve test app portability for other platforms

Change-Id: I8793b7e6f6ebd0f83ff8e5bd534234cd1f4166a2

More error logging. Correct error code.

Normalize error messages to use similar wording, remove "Error:".

Change-Id: I8746d28c0c0ada43e691f7859d0aaeed25a82dbb

Merge "Verify that no extant objects at engine shutdown" into gingerbread

Verify that no extant objects at engine shutdown

Log the total count and ID numbers of any stragglers.
Fix the order of steps in engine shutdown.
Add more comments.

Change-Id: I12b531bcd43c288f9988768193274bad6984748a

Use AudioSystem constant for global effect session ID, instead of 0.

Change-Id: Ib3d236e81da7f0764983fba37ffaa76058aa935c

Support auxiliary effects on OutputMix, and direct and send levels
 on AudioPlayer.

added global function for mB to attenuation conversion.

Change-Id: Ib7f92397786e4b2e21d6ad51585ac1b542ffab6c

Use standard filenames for tests.

Change-Id: I0bfc8bd98f00ce1dd9d7e36d402d4cd422cb8ea6

Fix problems found by the mute solo test.

Bug fixes for USE_OUTPUTMIXEXT (desktop):
 - SL_PLAYEVENT_HEADATNEWPOS was broken
 - GetPosition is now accurate, with or without seeking
 - Fix race condition between AudioPlayer and Track
 - Duration is updated if initial estimate is too low (e.g. if unknown is reported as zero)
 - GetPosition while pending Seek now returns that value so the Seek appears synchronous
 - Remove useless code to retry Enqueues; now drop decoded data if Enqueue fails
For all source files touched, add curlies to the "if" statements.

Change-Id: I1f70d8e683bc92831848af20857347d45c6407a9

Add support for auxiliary effects.

- Implement the OutputMix object.

- Modify the interface of the Android effect initialization since
 an effect can be attached to a player (insert) or to an OuputMix
 (aux)
- fix bug in EQ where SL_RESULT_CONTROL_LOST was returned on a
 properly initialized EQ effect.

- Add a test application which creates an EQ on an OutputMix
- Remove unused constant in test apps.

Change-Id: Id9e4633d0e44e8a784f1154ce1a6df1357b72966

Buffer queue automated test update

Make it run on desktop as well as Android, so we can use desktop driver
  for comparison, and get test coverage using gcov.
Use slesutResultToString for error reporting.
Run each test case for all valid numbers of buffers.

Change-Id: If7753a0e77f4a906a41eae90135e1bc84531f2fc

Fix compile-problem on desktop, and SDL bug.

Fix bug caught by using gtest on desktop: SDL was not being shutdown properly when engine destroyed.
Fix compile problems on desktop for BassBoost, Equalizer, and Virtualizer.
Add new OpenSLESUT function: slesutResultToString.
Rename SDL_start to SDL_open to match SDL naming conventions.
Add SDL_close.
Add symlink for applications to use OpenSLESUT.h.

Change-Id: I6ef9895adc508b3b9570c79302c7a3046ca8246f

Implement the SLVirtualizerItf interface.

Add example for the SLVirtualizerItf.

Unify how an effect interface checks whether a valid effect is
  attached to the interface.

In implementation of SLEqualizerItf, ensure the effect is accessed
  only if its valid (just like for BassBoost and Virtualizer).

Change-Id: I03032de2925488bc78502f23a83972e666435a8b

Fix bugs found by clang (part of clang fix-it)

Serious bugs:
 - SLEqualizerItf::GetPreset was returning wrong preset index.
 - SLEngineCapabilities::QueryVibraCapabilities was returning wrong status code.
Another variable was initialized unnecessarily, but was not a serious bug.

Change-Id: Ifebbae1d457fa800e5adfb09decede4222b60aa1

Fix bugs in BufferQueue automated test and related

Fix test bugs:
Clean up the player, mixer, and the engine (must be done in that order).
Re-enable 2 more tests; all 9 tests pass now.

Fix engine bugs:
IObject now has a de-init hook, which cleans up condition variable and mutex.
Run the de-initializers in reverse order so that IObject de-init hook is run last.
Replace the dedicated mEngine.mShutdownCond by the shared IObject condition variable.
Forgot to iniitalize mShtudownAck.
Unlocking the object mutex during CEngine_Destroy was bogus.

Miscellaneous logging and debugging improvements:
Track the source code location of the most recent mutex unlock.
Add more trace log options for detailed leave reasons.
Improve performance of trace log when disabled.
Turn on assert checking.
Check return value of pthread_cond_destroy and pthread_mutex_destroy.
Add comment.
Line length 100.

Change-Id: I23b7b495d439894b2fd31295a38cb73ef7b6be2e

Refactor effect parameter setting and getting to prepare support
 for additional effects.
Add support for Virtualizer parameters.

Change-Id: I39c69e883457af6444e49ba64e390213898cd55c

Set decode and render thread priority to ANDROID_PRIORITY_AUDIO.

Change-Id: I91843747fd1f51c0eca2d344a22a77d8df853015