Fix 5249076 Don't let the shared memory buffer drain

Whenever pushing data to the shared memory from the
 AndroidBufferQueue, post a message to do the same operation
 again if there is still room in the shared memory. This
 ensures the consumption of data will keep the
 shared memory buffer full, not the notification of space
 available in shared memory, which may happen while the
 Android Buffer Queue is empty (after a clear() for
 instance).

Change-Id: I397d70ebb927fa6a1d0fb97321a70db0440b96fe

Merge "Bug 5249076 When starting to stream, have at least one buffer in shared mem"

Merge "Rename Android buffer queue GUID for future sink"

Merge "Bug 5244675 handle the switch cases of AAC decode"

Merge "Bug 5244675 miscellaenous AAC decode test"

Bug 5249076 When starting to stream, have at least one buffer in shared mem

StreamPlayer is receiving notifications from the MediaPlayer that
 a buffer in shared memory is available during playback, which
 causes StreamPlayer to consume data from the AndroidBufferQueue.
 If the shared memory is drained while the player is paused, there
 will be no new notification of a buffer in shared memory being
 available to start the cycle of consumption from the Android
 buffer queue.
This change causes StreamPlayer to consume from the buffer queue
 to fill one buffer in shared memory.
A subsequent change will change the logic to make sure the
 consumption from the buffer queue fills as much of the shared
 memory as possible.

Change-Id: I886832629dd5ef449719b77899ebdcc7ffc64330

Check callback result code and warn if non-success

The result code is currently unused, but reserved for future use.
This change will help make sure that developers do the right thing now,
so that they will not be impacted when we begin to act on the result.

Change-Id: I8c712882d41d09506defa77340676079bb34de6c

Rename Android buffer queue GUID for future sink

This change will make it easier to have Android buffer queue as a sink in
the future.  For backwards source code compatibility with beta developers,
retain the old name temporarily.  As soon as they have switched to the
new name, we will remove the old name.

Details:
 - rename XA_IID_ANDROIDBUFFERQUEUE to XA_IID_ANDROIDBUFFERQUEUESOURCE
 - rename SL_IID_ANDROIDBUFFERQUEUE to SL_IID_ANDROIDBUFFERQUEUESOURCE
 - rename MPH_ANDROIDBUFFERQUEUE to MPH_ANDROIDBUFFERQUEUESOURCE
 - did not rename the data locators and interface struct,
   as they can be shared across both source and sink
 - did not rename the member field mAndroidBufferQueue in
   CAudioPlayer / CMediaPlayer as that would be a huge change;
   can do that later if necessary
 - the SINK definition will be added later when needed

Change-Id: I36dc01050efb99d4d7802b2ca2b870f4d978ee21

Merge "Class name cleanup. (1/3)"

Add rewind (discontinuity) to native-media test

Change-Id: I0f32564d2a2937092436dd48fef877ed55b2d4f3

Bug 5244675 miscellaenous AAC decode test

Fix bug in counting encoded frames.
Test get play position and duration at various points.
Display all PCM metadata (in particular the channel mask).
Add example of expected output.

Change-Id: I0d0a629755a37b71a1540e79ae7a6f1ed7095a24

Merge "clean up in echo reference code"

clean up in echo reference code

Change-Id: Ie06ed382eee2f1d4d1289d9a49d8a93dcf103e4e

Merge "Added a FixedRotationFilter."

Merge "Make background replacement roughly independent of input size."

Added a FixedRotationFilter.

bug: 5221073
RotationFilter is used for dealing the 4 orientations of the device.
Fixed ToPackedGrayFilter to correctly resize the image
if aspect ratio needs to be kept.

Change-Id: Idd3c26ae2ec12b64f7d040a301a87c619fe146e3

Merge "rewrite lomo and documentary filters"

Class name cleanup. (1/3)

Addresses bug 5214502.

Change-Id: I82321ad516b93e98047fce361a97da020b7ac09e

Bug 5244675 handle the switch cases of AAC decode

Fix handling of AUDIOPLAYER_FROM_ADTS_ABQ_TO_PCM_BUFFERQUEUE decode use case:
  - Play::GetPosition: explicitly list decode case and return media
    player time; previously returned zero.
  - Seek::SetLoop for AudioPlayer or MediaPlayer: return
    SL_RESULT_FEATURE_UNSUPPORTED on non-URI data source.  This is
    correctly passed up to app.

The remaining have no immediate impact, but should make future maintenance easier:
  - Play::GetDuration: explicitly list decode case and return unknown time;
    no impact as already covered by default.
  - Seek::SetPosition: explicitly list decode case and handle as no-op; no
    impact as already covered by default.
  - Seek::SetPosition for AudioPlayer or MediaPlayer: return
    SL_RESULT_FEATURE_UNSUPPORTED with unsupported data source e.g. buffer
    queue.  No impact as the result is currently ignored, but added a
    FIXME to pass this up to app.
  - AndroidBufferQueue::Clear: explicitly list decode case as a no-op
    other than the partial clear of upper layer queue.
  - AndroidBufferQueue::onRefilled: explicitly list decode case as
    a no-op, but with a FIXME that this may need more work later.

Change-Id: I53c8a0b736f32cd265ee9c94a70127d4f80869b5

rewrite lomo and documentary filters

Change-Id: Ia88123992c4033d9d68c2e47ba22db4f6c911694

Make background replacement roughly independent of input size.

Change-Id: I1257718f4ba2ae72add6501d238ec78d0465861b

Fix recording resolution mismatch when using CamcorderProfile.

Did not use profile-provided resolution everywhere, resulting in
videos with tiny images in big black boxes.

Bug: 4510826
Change-Id: I845c436de091299fbd8a49eb3b8f70707aadc6f5

Merge "Updates to the MediaEncoderFilter."

Merge "Bug 5217144 AndroidBufferQueue::Enqueue error chks"

Merge "MediaSource: switch to MediaPlayer#setSurface"

Merge "Add fillblack option"

Updates to the MediaEncoderFilter.

- Support more of the MediaRecorder parameters
- Allow toggling recording on/off during filter graph execution.
- Pass timestamps on correctly

Bug: 4510826

Change-Id: I63c9c5e9554d8e8570f2051f08e81ef753069fa1

Merge "Example of AAC ADTS decoding with OpenSL ES"

MediaSource: switch to MediaPlayer#setSurface

This change updates MediaSource to use the new MediaPlayer#setSurface method, which
is replacing MediaPlayer#setTexture.

Change-Id: I4e4c5d1c5e481100db9265114bb285b766ae9877

Merge "Fix Grain filter and optimize other filters"

Merge "Fix b/issue?id=4882048"

Fix Grain filter and optimize other filters

bug: 5127871

Change-Id: I627a840906417891270aa2d93ea966c217d98b3c

Bug 5217144 AndroidBufferQueue::Enqueue error chks

Add more error checks to Enqueue:
 - check for a NULL data pointer with non-zero data size
 - check for a NULL item array with non-zero item array size

For the Enqueue item processing:
 - examine all items if there is more than one in the array
 - check for invalid combinations of items
 - check that total length of item array is large enough for each
   item's header and any optional data
 - check for an EOS with a non-empty item data
 - log if an unknown item key is found
 - added the EOS item handling code for AAC ADTS
   (the underlying implementation is still not yet done).

Fix copy/paste typo in initializeAndroidBufferQueueMembers.

AacBqToPcmCbRenderer::validateBufferStartEndOnFrameBoundaries:
 - now returns a more specific result code rather than bool
 - readAt return type ssize_t not size_t
 - readAt return 0 (EOS) is a LOGV not LOGE.

Fix race in Android buffer queue callback: replaced logic in
BufferQueueSource.cpp by safer code from android_StreamPlayer.cpp

Change-Id: I423ccbd6d76a736a486b49b5dfdb7898b52cdd02

Bug 5226085 miscellaneous metadata issues

The metadata keys are now exposed for AAC decode.

The metadata keys are now exposed at Realize. The keys were previously
only available after prefetching was complete. This made it more awkward
for an application to use them.

Fixed race condition in access to metadata values.

Change-Id: I3daed2da61f5ed0add6948f1558963cfe66795fc

Example of AAC ADTS decoding with OpenSL ES

Put AAC ADTS in an AndroidBufferQueue, retrieve PCM data through
 a BufferQueue.

Change-Id: I9718d7a9e030a2c3568de8579f3329b76f63d891

Fix b/issue?id=4882048

Change-Id: Ib26f4c9f1c64c70e779ac5533892a91cef23d1fe

Recording MFF Frames using MediaEncoder

Changed the GL native and jni to incorporate media encoding
This involves being able to call disconnect on the SurfaceMediaSource
and choosing the right egl config

Related to bug id: 5216763

Change-Id: I048bbbd2b308b6163fd380edd7575e52dee23ccf

Merge "Metadata keys for PCM decoding"

Add fillblack option

If quad box is outside of image, it will display black.

Change-Id: I4871e4bf495ec461a0923ea80cafd2566e07caeb

Merge "General code / documentation clean-up."

Merge "Change the OneShotScheduler to schedule only the source node once."

General code / documentation clean-up.

- Addresses API council's comments.
- Adds identity effect.
- Other clean-up.

Change-Id: Ib87ea95b8df44eef39be970971f436e46f0ade2d

Change the OneShotScheduler to schedule only the source node once.

Bug: 5214756
Change-Id: Ia14b57b618d875e71d0fd65909eb7aa69b867f95

Merge "xaplay command-line mediaplayer for MPEG-2 TS ABQ"

Metadata keys for PCM decoding

Declare in a public header the metadata keys an application which
 decodes audio data would use to figure out the output PCM format.

Make the SL ES Android header also include the configuration and
 metadata headers.
Update test apps to reflect this.

Change-Id: I298db4fdb800df2d48047691441aa7489aad4822

xaplay command-line mediaplayer for MPEG-2 TS ABQ

Change-Id: Id5ea3f478f45d4fa4b86772593ae831d9eade952

Bug 5193695 Fix crash after MediaPlayer destroyed

The StreamSource callback thread was continuing to run after the
MediaPlayer object was destroyed.  Fixed by adding a callback protector
and a pre-destroy hook. GenericMediaPlayer::preDestroy now also
calls MediaPlayer::stop just in case.

Change-Id: I5bd771d4d1936f433d2a8c9959593782c96daed9
Miscellaneous:
 - added an explicit destructor on CMediaPlayer::mAVPlayer for consistency
   (probably not a bug since there was a clear on it earlier)
 - updated comments for CallbackProtector
 - made some CallbackProtector fields private since no sub-classes yet

Merge "Bug 5153313 keep compatibility of local files URI"

Merge "Bug 5105712 synchronous callbacks"

Bug 5153313 keep compatibility of local files URI

Also fix a bug in decode from fd to PCM buffer queue, where
we were closing the application's fd.

Change-Id: I220cd1c242f2f5e240b4dd1b9e8d79c88878bdcd

Bug 5105712 synchronous callbacks

After GetPosition is made synchronous, it is safer to make client
callbacks synchronous also.

Change-Id: I2688f709ed672aaa62d046d5f95db1b81504c9e3

Merge "Converted libaudioutils implementation to C."

Merge "NEW_API: Make rest of the effect names public in EffectFactory."

NEW_API: Make rest of the effect names public in EffectFactory.

Bug: 4966161
Change-Id: If194ebb5d1c45e8826a53fe9a83a94a1690cc11d

Merge "Fix minor logging bug in core Filter code."

Merge "Bug 5198051 MediaPlayer prefetch status interface"

Merge "Assertion checks do LOGF not fprintf to stderr"

Fix minor logging bug in core Filter code.

Change-Id: Ic03692e2456d2ca2615b75c7da4672ae3fed60d9

Merge "Bug 5198051 MediaPlayer StreamInformation various"

Merge "Add reset function to Native Program."

Merge "Bug 5198051 Video sink on MediaPlayer is optional"

Merge "Bug 5198051 MediaPlayer PLAYEVENT_HEAD* callbacks"

Assertion checks do LOGF not fprintf to stderr

Also add comments to Android.mk, and add SL_LOGF not yet used.

Change-Id: I1e051c040b667e2e70b6df0c03648c309efef7d6

Bug 5198051 MediaPlayer prefetch status interface

Change-Id: I426c870f803df0a7052f17be955ba797a814b4cc

Converted libaudioutils implementation to C.

Converted libaudioutils implementation and interfaces from C++ to C
and removed dependencies from frameworks/base classes so that it can
be used by any audio HAL implementation.

Change-Id: I3f7ce541be8495d41864661451540971b067359b

Add reset function to Native Program.

Bug: 5202663
Change-Id: I6d49e7cb765f776e193e8bf1fcde26678438e12e

Merge "Several improvements for interacting with Camera app."

Bug 5198051 Video sink on MediaPlayer is optional

Change-Id: I2c78508fa1efac765ff5812c22e3a4c2cfc33b73

Bug 5198051 MediaPlayer PLAYEVENT_HEAD* callbacks

Change-Id: Ibb04fbf727997281bb77fd22c51cd90bd044e505

Merge "Bug 5198051 MediaPlayer play state PAUSED at end"

Bug 5198051 MediaPlayer StreamInformation various

Fix uninitialized field frameRate.

QueryMediaContainerInformation returns XA_RESULT_PARAMETER_INVALID if
info is NULL, otherwise it copies the information via the info pointer,
and returns XA_RESULT_SUCCESS.

QueryStreamName returns XA_RESULT_FEATURE_UNSUPPORTED.

Simplify RegisterStreamChangeCallback.

Change-Id: I261ac0c053be0de7bac5fc80661b979f7f9c84b6

Bug 5198051 MediaPlayer support GetDuration

Change-Id: I09d9292685d82b88cddd243e6e7f5410ffaa2248

Bug 5198051 MediaPlayer play state PAUSED at end

Change-Id: I9160336a9e74b2f2f501cb94d1f24047b2979b76

Several improvements for interacting with Camera app.

- Add support for extra library loading in FilterFactory
- Add support for listing custom libraries in graph files
- Add new SurfaceTargetFilter for output to a Surface managed by the app.

Bug: 4510826
Change-Id: Icfd350d8f83627934549c30cdfb52cd1a21d3e72

Merge "Created library for common audio HAL functions."

Merge "Moving photo editor effects to imageproc"

Merge "8-bit now works"

Merge "Tune the background replacer filter toward shipping quality."

Tune the background replacer filter toward shipping quality.

Works reasonably in talk; issues with dark objects remain.

Bug: 5111139
Change-Id: I32c84d73b8d69c89e1f1d7ae9a5f7738f4025fe3

Merge "Bug 5126938 dependency for video sinks"

Merge "Test playbq: 8-bit and big-endian byte ordering"

8-bit now works

Change-Id: I70ce7e10f0b465250b9f4d7644d21f5c4b4c99fa

Merge "Use XA instead of SL in type cast"

Moving photo editor effects to imageproc

Change-Id: Idb11915f9e70aa40a228532ed998563227b4c5ff

Use XA instead of SL in type cast

Change-Id: Icc8d678484f454860b32d8823d4a274d3d14d991

Merge "Add backdropper to list of effects, and reformat documentation."

Add backdropper to list of effects, and reformat documentation.

Backdropper is still hidden, until we have the full list of effects.

Bug: 4966161
Change-Id: I95560615eb891c951da676beeb2dc3ac610dc07e

Merge "Bug 5159291 lock safety"

Merge "Bug 5161265 Recover from mediaserver process crash"

Test playbq: 8-bit and big-endian byte ordering

Change-Id: Ic34f35226fee07945925cad8cfbc10c756092fee

Created library for common audio HAL functions.

libaudioutils groups functions that can be used by any audio HAL
implementation.
Current functions are:
- a resampler based on speex resampler.
- an echo reference module providing resampling, buffering and
time stamp management of PCM data between an output stream and an
input stream for use as echo reference by an AEC module.

This commit is the first step consisting of moving code previously in
crespo audio HAL. This C++ code will the be converted to C and stripped from
dependencies to frameworks/base classes for use by all audio HALs.

Change-Id: Ifa51e0c6358c23122a379d7f5d9ce13a65ff0cf0

Avoid future circular dependency in framework libraries.

Explictly list which framework libraries we depend on, instead of
accepting the default list (which will include this library in the
future).

Bug: 5098987
Change-Id: I781143f535d1e84122de460125915866420e8d07

Merge "Bug 5147281 prefetch status deadlocks"

Merge "Remove dead code from backdropper filter shaders."

Merge "AAC buffer decode to PCM buffer queue"

Remove dead code from backdropper filter shaders.

The code is not approved of by all shader compilers, and is unused
in any case.

Bug: 5111139
Change-Id: I83af451b870d3c2cb1be5c8d0b780e700453e04c

Bug 5161265 Recover from mediaserver process crash

Use the utility method IMediaDeathNotifier::getMediaPlayerService()
to get a reference to media player service.  It retries after errors,
and keeps a single global per-process reference to the service rather
than one per object.  If we still can't contact the media player service
after retries, handle it like an unsuccessful prepare.

Miscellaneous:
 - Remove unused field Parcel metadatafilter

Change-Id: I451a64bd76b4a7f157774abe967448ec081f1014

Bug 5159291 lock safety

Change-Id: I56f5fd038d63d033954d794168df4cbcdade26f3

Bug 5147281 prefetch status deadlocks

Fixed two deadlocks related to AudioTrack prefetch event callbacks:
 - if PrefetchStatus interface was initialized on an AudioPlayer,
   there was a deadlock at the point where it checked whether to do a callback
 - prefetch status callbacks are now called with the object unlocked,
   so they are permitted to do APIs on the object

Miscellaneous fixes:
 - SMP-safe locks for GetPrefetchStatus, GetFillLevel,
   GetCallbackEventsMask, SetFillUpdatePeriod, GetFillUpdatePeriod
 - SetCallbackEventsMask error handling for invalid bitmask
 - Enqueue sets prefetch fill level to 1000

Change-Id: Ic8b2c440aeb4efa3446a4a73d3422d934ceb0578

AAC buffer decode to PCM buffer queue

Implement AAC ADTS decode with buffers of compressed data being
 passed to the framework through an AndroidBufferQueue, and
 decoded PCM buffers being passed to the application through
 a BufferQueue (just like existing decode from URI/FD).

Change-Id: Id992a44a5ca7e404088b929f2b0afe9ca1f85223