Fix indentation and redundant whitespace

Change-Id: I8e0193ad5aa9405f64203bf7814d8a9e434c5dde

Reconcile with ics-mr1-release

Change-Id: I2e44bc34002b9d53adf8f7df72c9980f612e6274

merge in ics-mr1-release history after reset to ics-mr1

am 28cd3686: Merge "CTS-like Test for Face Effects recording support" into ics-mr1

* commit '28cd368665e7e5576102edfa8fa67ec09a6d3c69':
  CTS-like Test for Face Effects recording support

Merge "CTS-like Test for Face Effects recording support" into ics-mr1

am b7b605ff: Reconcile with ics-mr0-release

* commit 'b7b605ff5b4682bc650461c2b987aa031a1071ca':
  DO NOT MERGE: Fix ordering of SurfaceTexture startup in SurfaceTextureSource.

Reconcile with ics-mr0-release

Change-Id: I5f6ea1b12b77924424ff87495db2b2a262f8f3f4

DO NOT MERGE: Fix ordering of SurfaceTexture startup in SurfaceTextureSource.

Minimal cherry-pick from MR1.

An asynchronous SurfaceTexture only sends out a frame available
callback the first time a new buffer comes in; if a onFrameAvailable-
listener is not registered at this point, the callback never happens
even when new frames come in.

SurfaceTextureSource was calling its onSurfaceTextureSourceReady-
listener with a newly created SurfaceTexture before hooking up the
SurfaceTexture's onFrameAvailable-listener. This opened a window of
time for the onSurfaceTextureSourceReady-listener to set up the
provider end of the SurfaceTexture, and for the provider to get
buffers into the SurfaceTexture queue before the
onFrameAvailable-listener was registered.

And as a result, no new frame callback ever fired, and
SurfaceTextureSource eventually times out, or goes into permanent
sleep.

This change simply makes sure the onFrameAvailable-listener is
registered before the onSurfaceTextureSourceReady callback is fired.

Bug: 5614661
Change-Id: I8d6a72444ffc36b5c48952d0b1edd530ecb76478

CTS-like Test for Face Effects recording support

Vendors can run this test for checking if the device supports the
effects recording, which needs them to support a particular EGL
extension. That cannot be part of the actual cts test for now since that
part of the filterfw is not part of the  public api.

Related bug: 5584686
Change-Id: Ica444e54d1e6fb79721b401b67518f182e8616a6

am 6b77115d: (-s ours) am 3e843769: DO NOT MERGE: Fix ordering of SurfaceTexture startup in SurfaceTextureSource.

* commit '6b77115da5de1e6d82a1e96e30ffdb3f1a90a54d':
  DO NOT MERGE: Fix ordering of SurfaceTexture startup in SurfaceTextureSource.

am 3e843769: DO NOT MERGE: Fix ordering of SurfaceTexture startup in SurfaceTextureSource.

* commit '3e843769e46d3d53aa2565bf80af1e800a3bbf1e':
  DO NOT MERGE: Fix ordering of SurfaceTexture startup in SurfaceTextureSource.

DO NOT MERGE: Fix ordering of SurfaceTexture startup in SurfaceTextureSource.

Minimal cherry-pick from MR1.

An asynchronous SurfaceTexture only sends out a frame available
callback the first time a new buffer comes in; if a onFrameAvailable-
listener is not registered at this point, the callback never happens
even when new frames come in.

SurfaceTextureSource was calling its onSurfaceTextureSourceReady-
listener with a newly created SurfaceTexture before hooking up the
SurfaceTexture's onFrameAvailable-listener. This opened a window of
time for the onSurfaceTextureSourceReady-listener to set up the
provider end of the SurfaceTexture, and for the provider to get
buffers into the SurfaceTexture queue before the
onFrameAvailable-listener was registered.

And as a result, no new frame callback ever fired, and
SurfaceTextureSource eventually times out, or goes into permanent
sleep.

This change simply makes sure the onFrameAvailable-listener is
registered before the onSurfaceTextureSourceReady callback is fired.

Bug: 5614661
Change-Id: I8d6a72444ffc36b5c48952d0b1edd530ecb76478

Reconcile with ics-mr1-release

Change-Id: I028ce272d04e55877b502cfeaabe6bd7e59ca05e

am 30f49ce0: Merge "Bug 5622637 missing XA_PLAYEVENT_HEADATPOS" into ics-mr1

* commit '30f49ce0c8f9ee464e87552a6fb4fb3d6b09959c':
  Bug 5622637 missing XA_PLAYEVENT_HEADATPOS

resolved conflicts for merge of 2917f475 to master

Change-Id: I63f4689fabb2aee1682d785ad9c41f06de380396

merge in ics-mr1-release history after reset to ics-mr1

Merge "Bug 5622637 missing XA_PLAYEVENT_HEADATPOS" into ics-mr1

am 7898a5c5: Reflect (internal) API changes made to the IStreamSource interface.

* commit '7898a5c5f283c9487da48ef92f623c149d8a7a72':
  Reflect (internal) API changes made to the IStreamSource interface.

Fix bug 5553055 support video only format change in MP2TS

Allow command data when enqueueing a format change command.
Support format change where the application signals that the
 change is for the video stream only. Consider any other change
 or format change command with invalid data as a full format
 change.

Change-Id: I6c684eab36a51dded7e3d7e72b8effe13dac6cc8

Merge "Bug fixes and cleanup for video codec profiles"

Bug fixes and cleanup for video codec profiles

Bug fixes:
 - removed the line VideoDecoderNbProfLevel[c] = 0;"
   it could overwrite the MPEG2 entry if only one codec implementation,
   or worse if there were multiple codec implementations
 - android_videoCodec_getProfileLevelCombinationNb was returning success
   and zero count if a decoder has zero profile/level combinations,
   but OpenMAX AL 1.0.1 spec says it must fail in that case.

Minor cleanup:
 - omx does not need to be global
 - We only use the first codec implementation for a given
   decoder ID / MIME type, so simplify the code that iterates through
   the array of codec implementations
 - Add some const
 - Add some comments

Also includes #if 0 code to restrict reported video codec profiles,
as a placeholder -- disabled for now.

Change-Id: I81991f164efaf3606beb5789e1a79cf76ad033a0

Reflect (internal) API changes made to the IStreamSource interface.

Change-Id: I5cde6bc0d4bd9c62b4851d7e6375cd9216c7c062
related-to-bug: 5553055

Merge "Remove AudioTrackProxy since AudioTrack is RefBase"

Merge "Bug 5550980 test xaVideoDecoderCapabilities"

Bug 5550980 test xaVideoDecoderCapabilities

Print video codec profiles and levels as strings in addition to numbers.

Change-Id: I88d84121fe15c1cc1ba84067ec6b6bdb3405d704

Bug 5622637 missing XA_PLAYEVENT_HEADATPOS

Fix problem with a missing XA_PLAYEVENT_HEADATPOS after seek backwards,
due to a race condition between getPosition and onSeekComplete.

Change-Id: I408c8675df8a9b3cbd2874435fbeccdca355bc46

Merge "Bugfix for B5582675: FaceLift on 2x2 images"

audio utils: fix resampler frame count calculation

The calculation of the number of input frames needed by
resampler_resample_from_provider() for a given number of output frames
is broken. The consequence is that we request too many frames when upsampling
causing unecessary buffering and delay and too few frames when downsampling causing
more calls to provider->get_next_buffer() than necessary.

Change-Id: I74b6a1708f347f45ca3c12312bf1b47320823742

Bugfix for B5582675: FaceLift on 2x2 images

Any image that was smaller than 4 in width would cause an exception. This
edge case is now handled properly.

Change-Id: I7316a3f99070ab43ccba824d3cbccef4b613fc03

Reconcile with ics-mr1-release

Change-Id: I75bc4cd928873b79be9bf0d8561f765f61e6fbde

merge in ics-mr1-release history after reset to ics-mr1

Reconcile with ics-mr1-release

Change-Id: I65e29d550f42cd3614ad170b79589cec9a5a1d93

Merge "Bug 5238515 AndroidBufferQueue miscellaneous"

am b47a61dd: Fix ordering of SurfaceTexture startup in SurfaceTextureSource.

* commit 'b47a61dddc87d4ccc2cfc1031991172a043b906a':
  Fix ordering of SurfaceTexture startup in SurfaceTextureSource.

Fix ordering of SurfaceTexture startup in SurfaceTextureSource.

An asynchronous SurfaceTexture only sends out a frame available
callback the first time a new buffer comes in; if a onFrameAvailable-
listener is not registered at this point, the callback never happens
even when new frames come in.

SurfaceTextureSource was calling its
onSurfaceTextureSourceReady-listener with a newly created
SurfaceTexture before hooking up the SurfaceTexture's
onFrameAvailable-listener. This opened a window of time for the
onSurfaceTextureSourceReady-listener to set up the provider end of the
SurfaceTexture, and for the provider to get buffers into the
SurfaceTexture queue before the onFrameAvailable-listener was
registered.

And as a result, no new frame callback ever fired, and
SurfaceTextureSource eventually times out, or goes into permanent
sleep.

This change simply makes sure the onFrameAvailable-listener is
registered before the onSurfaceTextureSourceReady callback is fired.

Bug: 5614661
Change-Id: I8d6a72444ffc36b5c48952d0b1edd530ecb76478

Remove AudioTrackProxy since AudioTrack is RefBase

Change-Id: I4a1f755178699e0fdedf58e1f3cbd82e6067e071

Bug 5238515 AndroidBufferQueue miscellaneous

AndroidBufferQueue:
 - errors found by setItems are now hard errors, and cause Enqueue to
   return an error result (e.g. PARAMETER_INVALID or PRECONDITIONS_VIOLATED)
 - disallow EOS with non-zero data
 - disallow Enqueue after EOS
 - Enqueue checks MPEG-2 sync byte of first packet to reduce chance of downstream failures
 - use MPEG-2 terminology "packet" instead of "block"
 - Clear and init don't need to erase buffer content, as it is inaccessible
 - put placeholder in IAndroidBufferQueue_SetCallbackEventsMask
   for additional events beyond SL_ANDROIDBUFFERQUEUEEVENT_PROCESSED
 - comment unused field mBufferState in AdvancedBufferHeader
 - fix a minor typo
 - add dump method, #if 0 out by default

Change-Id: I11921e3784bfdb30e2cebaa1dabb705ea5ab0b92

Merge "Remove private ANativeWindow_fromSurfaceTexture"

AAC decode test additions

Add prefetch status check: believe it or not, this can actually fail if
the first AAC frame is invalid (I found this out the hard way).

Use SL_PLAYEVENT_HEADATEND event to detect end of decode.
No more sleeps!

Fix typos in checking the per-buffer context.

Display message when corrupt AAC frame, and display frame sizes.

Add option to enable raw output, on by default.

Use same naming conventions as other tests: slesTest_decodeAac.

Clean up the comments.

Move metadata key queries (but not values!) to before prefetch started.

Change-Id: I69e89f3c67c5310e45117dffd08c55bed9f5914a

Merge "Simplify code for the AAC decode use case"

Simplify code for the AAC decode use case

Details:

AacBqToPcmCbRenderer constructor
now takes an additional parameter: the IAndroidBufferQueue * interface.

So now there is no need to register a callback after construction,
and AacBqToPcmCbRenderer::registerSourceQueueCallback
can be deleted.

This allows android_audioPlayer_androidBufferQueue_registerCallback_l
to be deleted.

AacBqToPcmCbRenderer::mBqSource is always non-NULL and const, so this
allows removal of mBqSourceLock and some error-checking code.

BufferQueueSource source constructor now takes the IAndroidBufferQueue *
interface instead of the list [user, context, caller].

This allows mAndroidBufferQueueSource to be const, which then allows
more code to be deleted in IAndroidBufferQueue_RegisterCallback.

new AacAdtsExtractor can't fail.

Change-Id: I1f39ccc1e69833fe6df616feb76edbe4ce82fca7

Merge "More logging of media events for debugging"

merge in ics-mr1-release history after reset to ics-mr1

am e75ba3ef: Merge "Bug 5201321 recover from mediaserver crash" into ics-mr1

* commit 'e75ba3eff247622fdf36974d2216dcaff269ea5c':
  Bug 5201321 recover from mediaserver crash

Merge "Bug 5201321 recover from mediaserver crash" into ics-mr1

merge in ics-mr1-release history after reset to ics-mr1

am 760eb949: Fixing mirrored background video in camera effects

* commit '760eb949cf0abe299e6c6710cda5c52364945ae0':
  Fixing mirrored background video in camera effects

Remove private ANativeWindow_fromSurfaceTexture

ANativeWindow_fromSurfaceTexture was a private API that is now deprecated
and is being removed in a separate CL.  This change replaces all the
known uses by the preferred API ANativeWindow_fromSurface.

Note: a Surface can be derived from a SurfaceTexture by:

    SurfaceTexture st = ...;
    Surface s = new Surface(st);
    // use s
    s.release();

Change-Id: Ia55c023fc1487177ec76ae406ed77193083c9eac

Fixing mirrored background video in camera effects

The MediaSource is set to rotate so that it adjusts to the
rotation of the device, and displays upright. The backdropper does not
have to do any rotation except for the mirroring in case of front
camera.
Bug: 5490654
Change-Id: I36124c7ca7cce73b24683277370227abab97d55b

Merge "MediaPlayer kEventPrepared similar to AudioPlayer"

Bug 5201321 recover from mediaserver crash

Add a media player death notifier to recover from media server crashes.
Downgrade some logs.
Destructors should be protected.

Change-Id: I7774733bcb12fc7fbfe237bdbeffca1adde46e78

MediaPlayer kEventPrepared similar to AudioPlayer

Now that AudioPlayer kEventPrepared logic for successful/unsuccessful
prepare completion has been combined, make MediaPlayer work the same
way.  This will make it easier to merge similar code AudioPlayer and
MediaPlayer in an upcoming CL.

Change-Id: Iaa58dcc85b0ede82036c0a5f45d61d0e7853fe0b

Reconcile with ics-mr1-release

Change-Id: I448b449f721790e8a952fedb8696dfffa5daccb3

am e0746488: Merge "Bug 5353078: Reduce foreground holes in Backdropper." into ics-mr1

* commit 'e07464884a4bcb4d1548e7408015364dc037bdc1':
  Bug 5353078: Reduce foreground holes in Backdropper.

merge in ics-mr1-release history after reset to ics-mr1

am 5696c8a7: Merge "Bug 5585323 Authorize audio decode for system" into ics-mr1

* commit '5696c8a7a0e2fde4b051230544a58db853d3551f':
  Bug 5585323 Authorize audio decode for system

am b9217405: Merge "Bug 5578928 Fix race in stop for destroy" into ics-mr1

* commit 'b921740510e15a4e403e2819f1a8f0a9a6c03da9':
  Bug 5578928 Fix race in stop for destroy

Merge "Bug 5353078: Reduce foreground holes in Backdropper." into ics-mr1

am e8c04ee9: Merge "Fixing the mirroring in MediaSource Filter" into ics-mr1

* commit 'e8c04ee9ea63b90667aaf5be6fbb3475ecd93520':
  Fixing the mirroring in MediaSource Filter

Bug 5353078: Reduce foreground holes in Backdropper.

Change-Id: I77b84e4c345bad237af13b82989f2f8119770bd3

merge in ics-mr1-release history after reset to ics-mr1

Merge "Bug 5585323 Authorize audio decode for system" into ics-mr1

Merge "Bug 5578928 Fix race in stop for destroy" into ics-mr1

merge in ics-mr1-release history after reset to ics-mr1

Merge "Fixing the mirroring in MediaSource Filter" into ics-mr1

Fixing the mirroring in MediaSource Filter

MediaSource Filter was outputing mirrored video.
Bug:5585630

Change-Id: If87ee799c0fccff783d3db7cf04080160a5a5ac4

Bug 5585323 Authorize audio decode for system

Authorize the use of specific audio codecs for audio decoding
by the system, which guarantees audio is decoded for playback,
when the appropriate system permission is used.

Change-Id: I0633b04b4d0aebe89804ef48752bf05d72abc56d

am e87e28c3: Merge "Remove MediaPlayer::setVideoSurface legacy." into ics-mr1

* commit 'e87e28c32f35ed40a12b18ef51cc6af0f44ec270':
  Remove MediaPlayer::setVideoSurface legacy.

Merge "Remove MediaPlayer::setVideoSurface legacy." into ics-mr1

Bug 5578928 Fix race in stop for destroy

Change-Id: Ic5c84406bffa372a2fd3e92ad1aca1f6f5185026

Merge "CT and spec errata for MuteSolo::GetNumChannels"

Merge "Comments"

Comments

Change-Id: I1592d1979ce5f5811ff5a47b4b60db93e8670313

Merge "Fix line length"

Fix line length

And add explanation for -p# option of slesTest_recBuffQueue

Change-Id: I32a06caf633c6796e263a1beeec93513cd775096

Merge "xaplay test additions"

CT and spec errata for MuteSolo::GetNumChannels

The question of what to return when channel count is unknown was dicussed
by Khronos working group.  They decided to return zero channel count
and success if unknown.  Both the spec errata and CT have been updated for this.

Change-Id: I820291271f83bad2142046372d9b22e4d05faf07

Merge "DataSource::getSize() returns ERROR_UNSUPPORTED"

xaplay test additions

Allow playing a subset of input file.
Add format change test.
Call gComposerClient->dispose before exit.
Remove dead code related to re-initializing the engine.
Add position, seek, loop tests.
Force callback failures with -c option.
Don't enqueue EOS more than once.
Add STOPPED at end of xaplay.
Add temporary set play state to paused.
Added discontinuity test.
More stream information queries and printfs.
Shorter marker to make it easier to hit when looping.
Add explanation of command line options.

Change-Id: I5b425a52f9afc2c8f9083badb6de0773f711c6d9

Reconcile with ics-mr1-release

Change-Id: I3217d6a7871bd15bfa77f492f01d343237ed4dff

Merge "Fix build errors and warnings on native"

am 762f1065: am c157a9c2: Bug 5544082 Filter out unsupported codecs in OpenSL ES

* commit '762f10659c2ac6fb66afd09be5bddd7ffebbf9df':
  Bug 5544082 Filter out unsupported codecs in OpenSL ES

AudioPlayback_Parameters cleanup

Use AudioPlayback_Parameters more consistently in parameter list of
constructors.  Also, StreamPlayer::StreamPlayer was over-assigning to
mPlaybackParams (already assigned by its superclass's constructor).

Move initialization of AudioPlayback_Parameters from each case
to before the switch, as it is shared by each use case.

Remove old asserts.

Change-Id: I8925d36da031c80308457c7e940418d3a4e47b62

Merge "Finish out the StreamInformation interface"

merge in ics-mr1-release history after reset to ics-mr1

Merge "Reverb comply with OpenSL ES 1.0.1 spec"

am c157a9c2: Bug 5544082 Filter out unsupported codecs in OpenSL ES

* commit 'c157a9c259474d4e1736c4f9ccccad10b5000734':
  Bug 5544082 Filter out unsupported codecs in OpenSL ES

Bug 5544082 Filter out unsupported codecs in OpenSL ES

When decoding compressed audio data, disregard tracks
 encoded with unsupported codecs.

Change-Id: If0535013990923c6859829106ccf9e92e5856f90

Reconcile with ics-mr1-release

Change-Id: I89995f978dbd3c063c1f5f6a815c609f7e962d87

Remove MediaPlayer::setVideoSurface legacy.

Change-Id: I95aa181f39b375087b30ac65fea37589108c9f16

merge in ics-mr1-release history after reset to ics-mr1

am 13ffc7f7: am d0b07222: Merge "Bug 5526807 inform client of errors after prepare" into ics-mr0

* commit '13ffc7f7c69e2fabf1ef70ebb2ea81489be9bd74':
  Bug 5526807 inform client of errors after prepare

am d0b07222: Merge "Bug 5526807 inform client of errors after prepare" into ics-mr0

* commit 'd0b0722276aea9a06a118a64abfc8b3e9498e931':
  Bug 5526807 inform client of errors after prepare

Merge "Bug 5526807 inform client of errors after prepare" into ics-mr0

am 21b2f0ce: am d7762ba2: Bug 5526272 ignore player events during destroy

* commit '21b2f0ce677aa42c34df65995eefb30e20f2c3aa':
  Bug 5526272 ignore player events during destroy

am d7762ba2: Bug 5526272 ignore player events during destroy

* commit 'd7762ba20bc7d13e1af73539d7a2a75183e2960b':
  Bug 5526272 ignore player events during destroy

Bug 5526807 inform client of errors after prepare

Change-Id: Ic5c88f8e1c8e7d6226a7a71a2a89a70f0fa876f8

Bug 5526272 ignore player events during destroy

Change-Id: I45c67660c7cbfabcf66b0ffffbaedf56c17453d7

am 9a371a20: Merge remote branch \'goog/ics-mr0\' into ics-mr1

* commit '9a371a201366ff0c0c132a31029a6c2357400b3e':

merge in ics-mr1-release history after reset to ics-mr1