Merge "Fix race condition when deleting an AudioPlayer"

Merge "Added Random and RoundRobin Scheduler."

Added Random and RoundRobin Scheduler.

Change-Id: I03300714f66543784f3f8bee40d9865479127b77

Fix race condition when deleting an AudioPlayer

When deleting an AudioPlayer that is used as a PCM decoder,
 there can be callback underway, for instance when trying to
 delete a player just as the notification of its preparation
 arrives.

The fix consists in:
- flagging all callback with the CallbackProtector
 mechanism,
- only entering callback when it is valid to do so
- renaming AudioTrackProtector to CallbackProtector
 as this mechanism is not exclusively used for the
 AudioTrack callbacks.

Change-Id: I9336a75981de43f71a983c1300f3a0ff314ac1e0

Merge "Renamed port classes to more intuitive names."

Merge "Bug 4571308 Test video timestamps"

Bug 4571308 Test video timestamps

Change-Id: I4d1fe3b9e090fe014c45432a91ea43360213ee2e

Renamed port classes to more intuitive names.

Ports are used internally only, so this does not change API.

Change-Id: I60a274ddd30bbb1f82d1b12980b6b95a07cb65ce

Original problem fixed.

Revert "Revert "Bugfix for Illegal SurfaceTexture reuse and GLEnvironment exception handling.""

This reverts commit 0a4de9e89040ad82df1e99b8fe2745d34e25826a

Revert "Bugfix for Illegal SurfaceTexture reuse and GLEnvironment exception handling."

This reverts commit 06b4f19b51c9e2a1efe10cf2ac37c407cddca907.

Merge "Bugfix for Illegal SurfaceTexture reuse and GLEnvironment exception handling."

Merge "Fix NPE in audio decoding"

Fix NPE in audio decoding

If the data source wasn't properly initialized, the corresponding
 member field should be accessed.

Change-Id: I811827991078db4b8dc4278014f046361f06e1a2

Bugfix for Illegal SurfaceTexture reuse and GLEnvironment exception handling.

Change-Id: I4bb401766778e392800669f717e1344bc8a9c68d

Use runtime detection to filter FilterPort logs.

Change-Id: I2ecb848dfe07d1f50b0c663c3dd8a665a7bad275

Switch verbose logging in several places to use runtime detection.

Change-Id: I4abd4a38ee1dddb1aa6a5cd933a087a30bd5be49

Bugfix in opening outputs in Filter.

Change-Id: If8e6aaba27743970dfe8ace55ac1d5ecb2bf6fea

Bugfixes.

Change-Id: I45a596f7c4fb28d6883f398668a4cebeffaf4417

Fix resize bug.

Change-Id: Ib4c2d1df8b4c94d6d39a3d8b6c7db77c010f5f15

Merge "Port + Parameter Unification and Type Check Refactoring."

Port + Parameter Unification and Type Check Refactoring.

Change-Id: Ie546431039058eb27f185a023ef8b18b533eaa12

Merge "Bug 4606138"

Merge "OpenSL ES: additional features when decoding to PCM"

OpenSL ES: additional features when decoding to PCM

Support the following features useful when decoding a compressed
 asset to PCM:
- GetDuration in SLPlayItf
- implemented SLMetadataExtractionItf on an AudioPlayer whose
  data sink is a buffer queue (i.e. it's a decoder)
- added support for 6 new metadata keys that can be queried
  through SLMetadataExtractionItf, which match the fields
  of the SLDataFormat_PCM structure.
Updated the decode sample code to use these features: the
 slesTestDecodeToBuffQueue app now displays:
- the number of channels
- the channel mask
- the sample size
- the container size
- the sample rate
- the format endianness

Change-Id: I3ca1e29828fd8c16c89da43a28df429c9cdef7af

Merge "Add set method to Point class"

Add set method to Point class

Change-Id: I8ca425963ed1eeed22bf570fabfdc236927abd88

Merge "Better defaults and fix warnings"

Merge "Update CameraSource filter parameter handling."

Update CameraSource filter parameter handling.

Now allows parameters to be set and read when the filter isn't
actively running.

Change-Id: Id35e0b1b7b89390560a37d89d073693ffdf74c81

Remove mRunners in FilterContext.

Added missing @Override

Change-Id: Ief0b20b63195a1283dc93910c19767e87aea0fe4

Merge "Revert "voice processing""

Revert "voice processing"

This reverts commit 7585d227843769079ee2afb385607b3e23ac1e07

Merge "voice processing"

Merge "Added Blending in Shader program."

Added Blending in Shader program.

SetBlendEnabled
SetBlendFunc

Change-Id: Ic30f118bdb7d9789b656c6f5bc7fa5a5eabe3124

Bug 4606138

Change-Id: If4745ee39eeedf11388800cc30c26e4c4f8ddaa0

Fix MediaSource filter pausing behavior.

Now correctly pauses the underlying MediaPlayer.

Change-Id: Iaa57bf34dbc0b001cd303b9494fd7c2786ace3f5

Better defaults and fix warnings

Use better defaults for slesTestFeedback rxBufCount and channel count.
Fix build warnings.

Change-Id: Ieef4750cbc454029e61524cc8011a6c774088762

Merge "Code clean up."

voice processing

Change-Id: I0b6129f566349fd333f34773589d5d15c36a7c15

Code clean up.

add @Override
and remove unnecessary imports.

Change-Id: I274bd84c98e1dc657e35787ab0727b8247d70fdf

Merge core filter framework libraries into one.

Combines JNI and native shared libraries into a single shared library.

Change-Id: If2cdb2b048c9c45275b1eabd06c9846c20434422

Merge "Throws JAVA exception when native program process fails."

Throws JAVA exception when native program process fails.

Change-Id: I2645bed644fd87ba483e6f0288a3255939b4dc4b

Refactored KeyValueMap, Parameter Binding

Also adds ability to bind directly to program parameters.

Change-Id: Ia5571bcf1f87583eb2ffab0a9b839a510a979e00

Fixes to SurfaceRenderFilter, SurfaceTextureSource.

Multiple SurfaceRenderFilters can now share a single destination
FilterSurfaceView, as long as only one of them is running at a time
(ie, they're in different graphs that aren't run simultaneously)

SurfaceTextureSource now creates its SurfaceTexture on open() instead
of prepare(), to make it easier for an app to switch between multiple
processing graphs using the same ultimate source, and to keep down
memory consumption.

Change-Id: I37931b60b3de15ffa484c258e5018c15e9335371

Allow CallbackFilter parameter updates

Change-Id: I9c65b09ce059a4dadeb317aae25a0a155644bae9

Move runner done callback code to GraphRunner interface.

Instead of just being in AsyncRunner, implement runner done callbacks
in SyncRunner as well.

Change-Id: I9fab177493821b79c080940a69d123f1d6de35fc

Adds a SurfaceTextureSource filter.

This is a filter intended for applications that want to manage the
data source for the SurfaceTexture themselves, instead of using
existing simple interfaces from CameraSource or MediaSource.

The filter provides a SurfaceTexture object that can be connected up
to any data source to provide frames.

Change-Id: Icb7d48411e95c330d0023aa9e528160b20308c6e

Stop the OMX codec when destroying the OpenSL ES decoder

The AudioSfDecoder class uses a MediaSource interface from
 an OMXCodec object. In the class destructor, the codec must
 be stopped before the codec is anihilated.

Change-Id: Ibf0d729e055dcd8f4e05f3fb8332d650f9be0d30

Merge "Add filter for efficient conversion from color in-GPU to grayscale native"

Merge "Stress test for OpenSL ES: create and delete many players"

Stress test for OpenSL ES: create and delete many players

Implementation of a test to deal with 40 AudioPlayers:
- creation
- realize
- configuration
- prefetching
- playback
- destruction
If a player was created, regardless of whether the other
 operations were successful, it will be destroyed.
The number of players, 40, is chosen specifically because it
 is larger than the number of objects Wilhelm is currently
 capable of supporting, and larger than the number of tracks
 supported by the platform mixer.

Change-Id: I42287039d59b77e59c60d074fbc0177d9726b359

Add filter for efficient conversion from color in-GPU to grayscale native

Change-Id: I369d4d5a96d1958c70584e1a44fc9bffc99fb3ea

Merge "Add pause support to MediaSource"

Add pause support to MediaSource

Change-Id: Ic009c3f27a582c4b82aa4a449db409a54536f011

Revert "Use AudioTrack::event_type not int in callback"

This reverts commit 1d7ee1569f98f7f1b629a04bb0895165255c7f7b

Use AudioTrack::event_type not int in callback

Change-Id: Icd60d7eff86e3bdf9907c573b6900c8eca96acf1

Add missing makefile to new filterpack.

Change-Id: I5a8e9020c2a0073d59ac64a6fc76f695f4fc5ea1

Merge "Move Camara- and MediaSource filters to system/media."

Merge "Created abstract base class for frontend environments."

Created abstract base class for frontend environments.

Change-Id: Ib9c9396545e1b30aa621784f651ef6ffc0575adb

Move Camara- and MediaSource filters to system/media.

Change-Id: Ic7abc3ccc9424d6e563fa0b385b92e7993bbbb58

Cleanups of mobile filter framework core code.

- Eliminate basictypes.h
- Include what you use
- Consistent namespaces/header guards
- Copyright headers

Change-Id: I3658a4ce8268a58644e48544a162a910447123d1

Merge "remove unneeded dependency"

remove unneeded dependency

Change-Id: I52f1b4b57e46138fe6eb5de138d7a9acbbe82538

Merge "New audio effect API."

Renamed ObjectFrame to JavaFrame, FilterEnvironment to FilterContext.

Also renamed Filter functions for input/output formatting.

Change-Id: I8303add0084dda718762edb635421dce7ca41313

Minor fixes.

- Filter now has a getInputFormat() method.
- Requests for illegal filter ports now throws exceptions.
- More detailed toString() implementations for Frames.

Change-Id: I03e80ae30243256d8dd45665f172a46049aa2245

Renamed filterfw2 lib to filterfw.

Change-Id: Ib21cd71f1bbe3e7d65e4378fa158cbb4e0302cd6

New audio effect API.

media/EffectApi.h header file has been moved and renamed to
hardware/audio_effect.h

Change-Id: Ie24295fb16f3925f1cc43577605d371fd9a18528

Add filterpacks' native base library.

Change-Id: Ia83f68dee76eb9c5684f760380b48de9e146c3c5

Initial public check-in of Mobile Filter Framework

Change-Id: I952e3b11c1742b861cc31d3f3ecf2713cfa434f8

update for new audio.h header location

Change-Id: I2140f1def08bedd8b2bd5170c6ac5edccb80035c
Signed-off-by: Dima Zavin <dima@android.com>

Update automated OpenSL ES test for playback from file path

Remove test where duration is queried after the player is realized
 but before prefetching has started: invalid test, duration isn't
 known at this time.

Changed wording of error messages to make test failures easier to
 identify

Added test to compare retrieved duration, and expected duration.

Fixed expected file duration: was 75s, should be 71s

Change-Id: I6230c00f0c945c6966e1f08ecfbc67452667cf33

system/media: convert to use new defines from hardware/audio.h

Change-Id: I8f0f0d26e10dbbb4dfbc3b52f1c02737a375c5e0
Signed-off-by: Dima Zavin <dima@android.com>

- Add another parameter in notify() to be able to send timed text sample
through listener during video playback.
For feature request 800939.

Change-Id: I14aa18d2f803163a62c05f9e6bd990518c58cd3f

Merge "Fix build warnings caused by the 'long' change"

Fix build warnings caused by the 'long' change

Change-Id: I34e74d723ae7fd829dd140614928d7880e669f03

Fix stuttering at beginning of playback

When an AudioPlayer is created, its playstate is set to STOPPED
 three times. A stop command is translated into a pause + seek(0).
 When a seek is underway, any new seek command was postponed. So
 when playback would start after a first seek(0), the other seeks
 would be executed, causing the stutterring.
This CL verifies that, when seeking to a certain time, any new
 seek command to the same point in time received while seeking
 is ignored.
It also correct the update of mStateFlags, which should be done
 BEFORE the command is issued to the player, in case the seek
 complete notification happens syncronously with the seek command.

This CL does NOT fix why a newly created player sees its playstate
 updated to STOPPED three times in a test app like slesTest_playUri
 before playback starts.

Change-Id: Ic7627678735f559b911ea3c5a0e1e00d7d7534d2

Fix various bugs found with unit tests

ISeek::SetLoop was returning undefined status.
Khronos Bug 7614 - sizeof(long) != 32 bits on x86_64.
Fix bug in IID hashing due to the above.
Fix bug in GenericMediaPlayer::onLoop; was using mPlayer without checking for NULL.
Fix wrong API level in tests/sandbox/object.c.
Decrease log level from I to V in MediaPlayerNotificationClient::notify
    and GenericPlayer::~GenericPlayer.
Fix memory leak in tests/sandbox/engine.c.
Comment out overly verbose log in tests/sandbox/streamSource/slesTestPlayStream.cpp.

Change-Id: I322fa9e19ee95ea1e18870a20f6a3c03b8f21208

Merge "Refactor the handling of updates to attributes"

Refactor the handling of updates to attributes

Attribute handler functions are more modular and
get rid of those endless if statements.

Change-Id: I3221aeb3d26d2dcc5dc9a3f6d37d088f5ce511a1

Remove compilation warnings for SLmillibel level comparisons

In OpenSL ES, the volumes are expressed in millibels, which are
 stored as SLint16.

The mix and max values are defined as:
 define SL_MILLIBEL_MAX  ((SLmillibel) 0x7FFF)
 define SL_MILLIBEL_MIN  ((SLmillibel) (-SL_MILLIBEL_MAX-1))

Therefore the comparison
 (SL_MILLIBEL_MIN <= level)
is always true.

This CL removes those comparisons rather than use the compilation
 flag to suppress the warning (it is a useful warning!).

Change-Id: I5672f257661ac045ba5282ecb0ecd14d985d8102

Use reference to sp on GenericPlayer

When an object like GenericPlayer is stored in CAudioPlayer or
 CMediaPlayer by a sp, don't use a pointer to GenericPlayer when
 the object is passed as a function parameter, but rather use
 a reference to the sp that maintains it.

Change-Id: I49ff219618ac4b657deb43cfffb65f0e0dd6d58d

Re-generate autogen files

Change-Id: I3ff71790a981ea0f1e604bae344e4a949509405a

Merge "OpenMAX AL: support the SeekItf and XA_PLAYEVENT_HEADATEND"

OpenMAX AL: support the SeekItf and XA_PLAYEVENT_HEADATEND

Implemented support for:
 - seeking to a position
 - looping (only over full duration is supported, no loop points)
 - notification of end of content reached
This is all for the playback of URIs or FDs.

Change-Id: I015c40e66feb7ac280459895f5f7915ac964df31

Merge "Fix build on non-Android platform"

Merge "Line length 100"

Line length 100

Change-Id: Ie4459eed7f94870413bc87252910c1189ecf2e06

Fix build on non-Android platform

Change-Id: I85b25467ac5bae45b12ec5b0abfb4696449c2da8

OpenSL ES / OpenMAX AL code clean up

Do not use cryptic acronyms for Android object type definitions.

Harmonize object state and type naming conventions.

Change-Id: I62bed7aba2e0b9ea0c47264af8e3cf29f74746a8

Move support for SL URI and FD playback under GenericPlayer class

The SfPlayer class was implementing URI and FD decode and playback
 in the application process. This CL removes this class entirely
 and moves its functionality under the GenericPlayer class. This
 means that the playback of URIs and FDs is now implemented through
 an android::MediaPlayer object running in the media service.

The SfPlayer header had many StageFright and system includes, which
 are now moved to the relevant files where they are required,
 instead of automagically coming from sles_allinclusive.h.

Note that this CL breaks support for the following OpenSL ES features
due to missing Android MediaPlayer features:
 SLPlayItf.GetPosition
           HEADATMARKER
           HEADATNEWPOSITION
 SLVolumeItf.EnableStereoPosition
           SetStereoPosition
 SLMuteSoloItf
 SLPlaybackRateItf

 Support for those features will be reintegrated in subsequent CLs.

Change-Id: I8d7f6ea006eb7b876ef1ca1909d74b4517335850

Fix bug in last commit

Always read the documentation:
 - Add hooks for new OpenMAX AL interfaces
 - Since OpenSL ES engine interface has a deinit hook, add one for OpenMAX AL also

Change-Id: I266bf2112e6f8e525e5b3c06bbb6a68a4764e978

Merge "Add basis for future OpenMAX AL 1.0.1 interfaces"

Add basis for future OpenMAX AL 1.0.1 interfaces

Bug fix:
OpenMAX AL media player was using SL instead of XA symbols for the Object
and DynamicInterfaceManagement interfaces. A Object::GetInterface using
the XA symbols would have failed. Fortunately, these are not used in
typical applications.

Minor cleanup:
 - Add README.txt for mphtogen tool
 - Fix clang warning
 - Update comments
 - Remove spurious commas
 - Fix warning at declaration of SL_IID_array
 - SL_IID_ANDROIDBUFFERQUEUE was added in API level 12, not 10

Change-Id: I75abced5bc3c3fdabe40face25463810de741bf2

Add OpenMAX AL 1.0.1 spec

Change-Id: Iaeb78d64d01cd776f8a7096371f20ce366a64b79

Reduce include noise in SL and XA class definitions

In classes.h: we should only have to include the headers
 for the objects that are actually stored as member variables of
 the OpenSL ES and OpenMAX AL objects (so here AudioTrackProtector,
 Effect, and GenericPlayer).
In CMediaPlayer.c: remove useless includes and change the video
 surface function prototypes to use const refs to strong pointers
 instead of pointers to access the GenericMediaPlayer objects.

Note that android_SfPlayer.h is still in sles_allinclusive.h because
 it will be dealt with when the SfPlayer functionality is absorbed
 under LocAVPlayer.

Change-Id: Ic2c9459862588399d9545a247cc15341effc0f66

Clean up main header

There are many includes in sles_allinclusive.h that should be moved
 to the files that rely on those headers. This is a first cleanup
 to move Android audio effect related headers to where they belong.
 + one relocation for a media server related file. More to come
 with AudioPlayer refactoring.

Change-Id: I181dde96db522d9cfb537dffea18ba57f855b99e