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

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

Add timestamp as a port to GLTextureSource.

Allows filter chains using external texture sources to propagate
timestamps provided by the external source.

Bug: 5149729
Change-Id: Ic365257306636ea1083ad23a811a48a5e872bd5b

Merge "Fixes to BackDropperEffect."

Merge "Bugfix (5144111): Choose FBO pixel read over texture pixel read."

Bug 5126938 dependency for video sinks

Fixing bug 5126938 will involve configuring the video sink in two places:
at the initial data locator when creating the MediaPlayer object, and
then later in the IAndroidConfiguration interface.  This CL is preparation
for that.

Details:
 - Abstract out common code into android_Player_setNativeWindow.
 - Move the code for checking and initializing the video sink from an
   #ifdef ANDROID in the CMediaPlayer_Realize portable code to into
   platform-specific android_Player_realize.
 - Only set one of mSurface or mSurfaceTexture to non-NULL.
 - For methods which can't fail, replace the SLresult return value by void.

Change-Id: Iec8fd91e1a6de45eb64241b35e3f7d7faf63d727

Fixes to BackDropperEffect.

- Enable auto-wb
- Fixes to graph, filter references

Bug: 5039738
Change-Id: I9fb17659fb706920a4637f921a6e230984aa3051

Bugfix (5144111): Choose FBO pixel read over texture pixel read.

Change-Id: Ibdae635445551c302d7edb3f6fd77369e43b7293

Merge "add ImageSlicer and ImageStitcher filters."

Merge "Initialize video decoder capabilities fields even when unknown"

Merge "add customize option in SurfaceTectureTarget"

Initialize video decoder capabilities fields even when unknown

We only support the discovery of codecId, profile and level when
 using the VideoDecoderCapabilities interface. The other fields
 in the XAVideoCodecDescriptor structure need to be initialized
 even if they're not filled.

Change-Id: I8833437b3b23748a50076b65baf5b9a250290b86

Merge "Bug 5126938 dependency and miscellaneous bug fixes"

Merge "Updates to slesTestRecBuffQueue test app"

Merge "Bug 4603165 playback rate change"

Merge "Bug (5106403): Serialize filter input values across threads."

Merge "Bugfix (5088434): Accessing bitmap data on non-RGBA native frames."

Updates to slesTestRecBuffQueue test app

Allow the preset number to be specified on command line,
or to use default.

Change-Id: Ia4e996f8726a6bd8ca781bba5930b63bcba854bc

Merge "Add background replacement Effect."

Merge "Add background replacement filter to mobile filter framework."

Merge "Bug 5135150 Fix ifdef C++"

Merge "Add playback rate to buffer queue test player"

Bug 5126938 dependency and miscellaneous bug fixes

Fixing bug 5126938 will involve adding the IAndroidConfiguration interface
to CMediaPlayer object.  In the process of doing this, I found a few
miscellaneous bugs in the error handling for IAndroidConfiguration.  This
CL fixes those bugs, and prepares for adding the interface CMediaPlayer
by making it easier to support new objects with the interface.

Details:

 - Generic error handling:
   Move some error checks from the object-specific code into the generic
     IAndroidConfiguration.c.
   Fix bug in SetConfiguration and GetConfiguration where we were not
     checking configKey for NULL
   Return SL_RESULT_FEATURE_UNSUPPORTED instead of
      SL_RESULT_PARAMETER_INVALID if IAConfig interface is exposed on
      an object which doesn't really support it (this could if the classes.c
      interfaces table was updated before the cases were updated).
      Compare with IMuteSolo for non-AudioPlayer.

 - Object-specific error handling:
   Fix bug where we weren't returning the actual value size in
     audioPlayer_getConfig when pValueSize was non-NULL and pConfigValue
     was NULL.
   Return result SL_RESULT_BUFFER_INSUFFICIENT instead of
     SL_RESULT_PARAMETER_INVALID when output buffer is specified
     as non-NULL but available space is too small.  Compare with
     IAudioIODeviceCapabilities.
   Return result SL_RESULT_SUCCESS instead of SL_RESULT_PARAMETER_INVALID
     when output buffer is specified as NULL; this is intended to allow
     the application to discover the required space.  Compare with
     IAudioIODeviceCapabilities.

  - Use 'switch' instead of 'if' for checking object types, to improve
    readability when adding more object types

Change-Id: I8b2cdd506a36bfa9b1326a8a9966a6633bbbba6d

Merge "Miscellaneous loop test improvements"

Bug 5135150 Fix ifdef C++

Change-Id: Ic28452ce6b0c717722098699092f1797424a6d9b

Miscellaneous loop test improvements

Enable HEADATEND event to see whether looping causes that event.
Recover from prefetch errors using the new approach of condition signal rather than polling.
Check result returned by GetFillLevel.

Other changes:
 - Make assert work.
 - Pass NULL as 3rd parameter to RegisterCallback when context is not needed.
 - Use CheckErr instead of assert when checking result codes.
 - Add comments.

Change-Id: I972b9eca9836e62141b6a078345f091e60689321

Add playback rate to buffer queue test player

Change-Id: I831290c1d70bbd66031f609a63549801b7c7f57b

Add background replacement Effect.

Bug: 5061093
Change-Id: I8db69381627c89eb9851a3383e3715ce8ee934df

Add background replacement filter to mobile filter framework.

Bug: 5098987
Change-Id: I242d2a13403b3d1ace74b8b41bd23734ce0333e2

Merge "Revert "Add background replacement Effect.""

Merge "Revert "Add background replacement filter to mobile filter framework.""

Revert "Add background replacement Effect."

This reverts commit 3aeb06f571d69db6e7dcdca85f328b022c041f90

Revert "Add background replacement filter to mobile filter framework."

This reverts commit b07371016b15886136e88f1ec62d53daa3c5053f

Merge "Add background replacement Effect."

Merge "Add background replacement filter to mobile filter framework."

Merge "Bug 5903270 play interface, events, markers, etc."

Add background replacement Effect.

Bug: 5061093
Change-Id: Icdcf7597c9ce2ee9e6443d3b9eeaf9c99b57c809

Bugfix (5088434): Accessing bitmap data on non-RGBA native frames.

Change-Id: I8db22b6d78c5350815c043acbc55e578bee1a8a0

Merge "Fix race in reverb test"

Merge "Bug 5110581 SL_PLAYEVENT_HEADATEND while looping"

Add background replacement filter to mobile filter framework.

Bug: 5098987
Change-Id: Ib96da73756b736f2960b39cdc47673b52134e881

Bug 4603165 playback rate change

Non-trivial implementation is only for AudioTrack with rate range 500
to 2000 per mille and property SL_RATEPROP_NOPITCHCORAUDIO; for Android
MediaPlayer the rate range is 1000 to 1000 per mille.

Details:
 - no multi-range
 - fix SMP races in GetRate, SetPropertyConstraints, GetProperties
 - SetRate only updates mRate if successful
 - Rate_SetPropertyConstraints only updates mProperties if successful
 - return SL_RESULT_FEATURE_UNSUPPORTED for non-AudioPlayers
   which have this interface (currently only MidiPlayer)
 - for Android, atomically set playback rate and constraints
   and simplify to use hard-coded constraints rather than querying
 - per the spec, SetPropertyConstraints now returns
   SL_RESULT_PARAMETER_INVALID if the constraints are out of range,
   or SL_RESULT_FEATURE_UNSUPPORTED if the constraints are in range but
   not supported by this implementation

Change-Id: I051ff92b20f3845ffa12bccec778937d42823e5d

Bug 5903270 play interface, events, markers, etc.

As the media framework does not directly support marker and periodic
callbacks, we use a retriggerable one-shot timer at application level
to emulate this.  This has the advantage of being faster, but is also
less accurate than if it were in the framework.

Details:
 - Get position is now synchronous and (mostly) lock-free
 - Fix regression in SL_PLAYEVENT_HEADATMARKER and SL_PLAYEVENTHEADATNEWPOS
 - Fix SMP races in GetPlayState, GetCallbackEventsMask, GetMarkerPosition,
   GetPositionUpdatePeriod
 - AudioSfDecoder was not updating mPositionMsec when getPositionUsec failed
 - Updating attributes is relatively expensive, so only do it when a significant change
    in SetMarkerPosition, ClearMarkerPosition, SetPositionUpdatePeriod
 - Error on unknown message type in onMessageReceived
 - Delete an obsolete FIXME
 - "Clearing" a marker is now equivalent to disabling the marker event,
   and the default marker position is cleared.
 - use Android time units and types (int32_t, ANDROID_UNKNOWN_TIME, etc.) where appropriate
   instead of SLmillisecond and SL_TIME_UNKNOWN

Change-Id: Ib78bafa20d883b8d927364769663837389b6ea1d

add ImageSlicer and ImageStitcher filters.

Change-Id: Id47e121aa783d02ef7f63b6b1c8d724fa4e2d57b

Merge "Fix for 5117329: Stored frames which are externally visible check validity."

Merge "Fix for 5117999: Make isEffectSupported() a static method."

Merge "Fix for 5117285: Stream-based filters no longer close streams."

Fix for 5117285: Stream-based filters no longer close streams.

Stream-based filters should not close streams, as they do not open them.

Change-Id: I01688fecde4d80e6089fb4c2fd017144dfc88408

add customize option in SurfaceTectureTarget

Change-Id: I297168c0f67928b2180f2d0ce95c2f6d3bbbf107

Merge "Fix structgen script to build Java directory hierarchies."

Fix for 5117999: Make isEffectSupported() a static method.

This is required for applications such as GTalk, which need to verify that
certain effects are available before they setup an EffectContext.

Change-Id: I0f22b2ef378ebf0cb916305bc2a9f6b093b0f715

Bug (5106403): Serialize filter input values across threads.

Change-Id: I973cfc681f4e9a53fcdee15acf55d1935d9a9676

Fix for 5117329: Stored frames which are externally visible check validity.

This patch adds callbacks to Frames to let them prepare frames for storage,
and validate them on retrieval.
Also includes some name changes to badly named functions in gl_frame.cpp.

Change-Id: Ie25ef5136a6f583cbb8f70d7c2f1c418b4bceb83

Merge "Added FilterGraphEffect."

Fix structgen script to build Java directory hierarchies.

Bug: 509898
Change-Id: I27a1ee3dd3f3fcfa82c5f2620e5e30193327105d

Fix race in reverb test

Was setting play state to PAUSED before enabling prefetch callbacks

Change-Id: Idc3d19d87799afe7e8314199c1317cf2cea2c449

Merge "Increase MediaSource's startup timeout."