am 54233057: am 1cb4e166: Merge "Add MEDIA_STARTED/PAUSED/STOPPED events to media players" into klp-dev

* commit '54233057d1c8466d3cc86e2f5ecc39e7816c4f7a':
  Add MEDIA_STARTED/PAUSED/STOPPED events to media players

am 1cb4e166: Merge "Add MEDIA_STARTED/PAUSED/STOPPED events to media players" into klp-dev

* commit '1cb4e1661e4144cb188c2c79b89804fcb902fd19':
  Add MEDIA_STARTED/PAUSED/STOPPED events to media players

Merge "Add MEDIA_STARTED/PAUSED/STOPPED events to media players" into klp-dev

Add MEDIA_STARTED/PAUSED/STOPPED events to media players

This is needed for the MediaTimeProvider java interface, so it does not
continually poll for current media time.

Note: NuPlayer and AwesomePlayer do not correctly handle stop (pause
instead), so for those we will signal PAUSED.

Signed-off-by: Lajos Molnar <lajos@google.com>
Change-Id: I3c61e1bda475f131323f475c18a42e3ec66c9ae1
Bug: 10326117

am ce57e797: am 75c16dde: Merge "You can now opt-in to having the video decoder push blank buffers to the" into klp-dev

* commit 'ce57e797526db119f5baaaeab75cd8de4b162c9a':
  You can now opt-in to having the video decoder push blank buffers to the

am 75c16dde: Merge "You can now opt-in to having the video decoder push blank buffers to the" into klp-dev

* commit '75c16dde986e16ac5fd4d34b06b0cc6a90679866':
  You can now opt-in to having the video decoder push blank buffers to the

Merge "You can now opt-in to having the video decoder push blank buffers to the" into klp-dev

Merge "Simplify code that checks for record standby"

Merge "checkForNewParameters_l returns a value which is currently ignored"

Merge "Use Autolock where possible"

Merge "Add ThreadBase::processConfigEvents_l()"

Merge "Pull in scope of effectChains"

Merge "Add record thread locking comments and FIXMEs"

Merge "Convert if/else-if in threadLoop to switch"

am 692ccb68: am c85a986a: Fix build.

* commit '692ccb6868ae5555bab294381f0e618bc3583baf':
  Fix build.

am c85a986a: Fix build.

* commit 'c85a986af84962515c10499faab3dc8f9a0948d4':
  Fix build.

Fix build.

Change-Id: If87636a654c8cfb2449e246df213fb24b08b4ad3

Simplify code that checks for record standby

Take advantage of the new processConfigEvents_l()
and it's post condition that mConfigEvents is empty.

Change-Id: Icd40fb175f3e374203c9c8d2e9df2c3929bed89b

checkForNewParameters_l returns a value which is currently ignored

Change-Id: I19b9cc9df094d67560f6e832d94fa284dca18687

Use Autolock where possible

Change-Id: Id0e1836831c45beda4e5cb11e983005ea699fc2a

Add ThreadBase::processConfigEvents_l()

Change-Id: I6bcdc212a984c52ab98b0b0744943b4ecc1e1542

Pull in scope of effectChains

Change-Id: Id5336f8663099088a0958fb8bd59fc5df9f92829

Add record thread locking comments and FIXMEs

Change-Id: Ia5bdc9b8b013c2e40af17c82833051290bf4df70

Convert if/else-if in threadLoop to switch

Change-Id: I7428abb332ead6c2a1df5903f6c4e166be1f7abb

am 4dc2abeb: am c13a958f: Merge "fix build. optional tests broke" into klp-dev

* commit '4dc2abebf199df8d30b9ff76b4c5045c59a01de2':
  fix build. optional tests broke

am c13a958f: Merge "fix build. optional tests broke" into klp-dev

* commit 'c13a958fba83dae63a8cf23b76b36a8fc8580aef':
  fix build. optional tests broke

Merge "Fix switch indentation to follow our audio style"

Merge "fix build. optional tests broke" into klp-dev

fix build. optional tests broke

Change-Id: Ifb38fb2a7bd9c3d6305726f8e6d661be05cdcf7e

Fix switch indentation to follow our audio style

Change-Id: I137763917962b276838715319b163c763cb54c71

am 3ecddeb9: am 99a2297e: Merge "Another error check" into klp-dev

* commit '3ecddeb951ee77bb2c2a33cee4dfbcc55e184c4c':
  Another error check

am 99a2297e: Merge "Another error check" into klp-dev

* commit '99a2297e4f9746cd5efc3aa3731d17052f32e058':
  Another error check

am 9bb7135c: am 69b5e38c: Merge "Use destroyDisplay()" into klp-dev

* commit '9bb7135c702f57f59f47253839fee81b983aa22e':
  Use destroyDisplay()

am 69b5e38c: Merge "Use destroyDisplay()" into klp-dev

* commit '69b5e38ca1af5696da1c00627d2e64f7db00d2e1':
  Use destroyDisplay()

Merge "Another error check" into klp-dev

Merge "Use destroyDisplay()" into klp-dev

Another error check

Change-Id: I01ed5c0c7dcb520732d2856859bced31df5fbbb5

am 5c9fb641: (-s ours) Reconcile with klp-release - do not merge

* commit '5c9fb641d93fa6758978adfb02cc0f2a51d02bdc':
  Fix regression for AudioRecord streaming callback mode

Reconcile with klp-release - do not merge

Change-Id: I4bdda8ab19e05b9e74dba9094cb6f36af09d58d2

Use destroyDisplay()

Use the new SurfaceComposerClient::destroyDisplay call to shut the
virtual display down.

Change-Id: I0225a44594abf427d386456622b23b01fdd1690f

You can now opt-in to having the video decoder push blank buffers to the

surface on the transition from executing->idle by specifying the key
"push-blank-buffers-on-shutdown" with a value of 1 in the call to configure.

Change-Id: I1155fccf89f18e717728c985e590651597595515

am 984b3f4d: am 7f76a81b: Merge "Check adaptive playback support via CodecCapabilities" into klp-dev

* commit '984b3f4df6ac50a7d655391b19afefa8d5ce30ba':
  Check adaptive playback support via CodecCapabilities

merge in klp-release history after reset to klp-dev

am 7f76a81b: Merge "Check adaptive playback support via CodecCapabilities" into klp-dev

* commit '7f76a81b18ade3158ae27015637074ce08e837b2':
  Check adaptive playback support via CodecCapabilities

Merge "Check adaptive playback support via CodecCapabilities" into klp-dev

Check adaptive playback support via CodecCapabilities

Added flags field to CodecCapabilities, so that applications
can query whether codecs support various features.  For now
added one video-decoder feature:

kFlagSupportsAdaptivePlayback

Media playback applications can query it to see if a codec
supports seamless resolution changes, such as by supporting
dynamic output buffers.

Signed-off-by: Lajos Molnar <lajos@google.com>
Change-Id: I09da46e8ab6b8645fa2749e33128e49eda2e865b
Related-to-bug: 7093648

Merge "Explicit comparison to 0 for integral types"

Merge "Move local variable declarations to point of first use"

Merge "Use curly braces in 'if' to make it easier to add logs"

am 65a6b3b6: am cd06d357: Merge "Optionally repeat the previously submitted frame to the encoder" into klp-dev

* commit '65a6b3b6a19e0b4920ba504a3ee40e4abe3a740d':
  Optionally repeat the previously submitted frame to the encoder

am cd06d357: Merge "Optionally repeat the previously submitted frame to the encoder" into klp-dev

* commit 'cd06d357ed4fcde991aa55dc97317d97cb287332':
  Optionally repeat the previously submitted frame to the encoder

Merge "Optionally repeat the previously submitted frame to the encoder" into klp-dev

Optionally repeat the previously submitted frame to the encoder

if no new frame has been delivered by surface flinger within the timeout
interval.

Change-Id: I282f1b726dfe5646b178d7858d6f5d4f5a264fde

am 8ec8a534: am 1e0b0808: Merge "Restore stagefright command line tool." into klp-dev

* commit '8ec8a5346831efa699d31274363da9cf516981db':
  Restore stagefright command line tool.

am 1e0b0808: Merge "Restore stagefright command line tool." into klp-dev

* commit '1e0b080825a169c6ac655f11a798ff177575608c':
  Restore stagefright command line tool.

Merge "Whitespace"

Merge "Comments"

Merge "Restore stagefright command line tool." into klp-dev

am 00dbc7ba: am 46052913: Better error message

* commit '00dbc7ba6ba089c40ff1cbea370828b57a464b52':
  Better error message

am 46052913: Better error message

* commit '46052913f307b1561f1661bb776fa29c0775758c':
  Better error message

Better error message

Attempt to create the output file before handing it to MediaMuxer,
which doesn't report file-open failures in a useful way.

Change-Id: Ie24ff577dd50e185b4eb72575684d23a46f38d3d

Explicit comparison to 0 for integral types

Change-Id: I2bfe864d1bda719581e297a38d96abfc0e3748b9

Move local variable declarations to point of first use

Change-Id: Ia0fc4b099a3cf8bb5b67b287e7687e9dc9691e0e

Use curly braces in 'if' to make it easier to add logs

Change-Id: I58b33fefdd8bf703647414157a99a3223be3531c

Whitespace

Change-Id: Id3ccc183a03421330d0498faaa62a45915cdc3d6

Comments

Change-Id: I56f82904cb3855bad6f33ad1ff4ddf66a61c5720

Restore stagefright command line tool.

Change-Id: Ia17f48522f601d6a057ef3e1e8cd01f80a5c9951

Merge "Check input HAL channel mask instead of channel count"

Merge "Clean up error handling in createTrack and openRecord"

merge in klp-release history after reset to klp-dev

am 82278b2c: am fe9611bd: Merge "Fix potential leak of audio input handle." into klp-dev

* commit '82278b2ceeebbcf345ed81413eeffa82fa82e05b':
  Fix potential leak of audio input handle.

am fe9611bd: Merge "Fix potential leak of audio input handle." into klp-dev

* commit 'fe9611bd65a8d968d6512f6a83078ac064ec8e6e':
  Fix potential leak of audio input handle.

Merge "Fix potential leak of audio input handle." into klp-dev

Clean up error handling in createTrack and openRecord

Outside callers now use initCheck() to determine whether a TrackBase
has been created successfully, instead of relying on internal knowledge.

Previously, callers needed to know that a TrackBase was only valid if it's
getCbk() != 0.  For a Track (playback), they needed to know to also check
the track's name (track index).  Now, outsiders can just call initCheck().

Other changes:
 - Return a 0 reference if track creation fails
 - Remove a dead line of code in AudioFlinger::openRecord

Change-Id: If374924a3f6fd27906f625aa83dd0a1e3f506e00

Status pointer passed by caller is always non-NULL

in createTrack, openRecord, createEffect, createTrack_l,
createRecordTrack_l, and createEffect_l.

Change-Id: I2e459e4de9c78145f4d496e6abf289479a2f0941

Merge "Remove obsolete TrackBase::getBuffer()"

Merge "Pull up readyToRun into ThreadBase"

am 4d9bc1a7: am b8689b31: Support for "request-sync" parameter in ACodec

* commit '4d9bc1a742b386db4985ad1f973091a50207fdc2':
  Support for "request-sync" parameter in ACodec

am b8689b31: Support for "request-sync" parameter in ACodec

* commit 'b8689b31813f55cbc1bf8e9ca0d46a9ee113e38a':
  Support for "request-sync" parameter in ACodec

Merge "Simplify the code that allocates mMixBuffer by removing mAllocMixBuffer"

Support for "request-sync" parameter in ACodec

Also returns errors to caller instead of asserting...

Change-Id: Id3018655a2b3da4289167fba16af907350a511ae

merge in klp-release history after reset to klp-dev

Fix potential leak of audio input handle.

The audio input handle is ultimately owned by the audio recorder
object but it could be dropped on the floor if an error occurred
before that object was fully initialized.

Rearranged some of the argument validation and merged getInput_l
with openRecord_l to simplify the code and prevent such a leak
from occurring.

Bug: 10265163
Change-Id: I124dce344b1d11c2dd66ca5e2c9aec0c52c230e2

am 60147776: (-s ours) am 7473e356: (-s ours) am 943917c4: (-s ours) Merge "Fix SoundPool.play() looping regression in 4.3 DO NOT MERGE"

* commit '601477769cef254d3a95338e853a37ccf9aac63f':
  Fix SoundPool.play() looping regression in 4.3 DO NOT MERGE

am 7473e356: (-s ours) am 943917c4: (-s ours) Merge "Fix SoundPool.play() looping regression in 4.3 DO NOT MERGE"

* commit '7473e356f5b7de6b67aef3b34faa6ddb1c054060':
  Fix SoundPool.play() looping regression in 4.3 DO NOT MERGE

am 943917c4: (-s ours) Merge "Fix SoundPool.play() looping regression in 4.3 DO NOT MERGE"

* commit '943917c41f447fdceab6ce75e00c05551072000b':
  Fix SoundPool.play() looping regression in 4.3 DO NOT MERGE

Merge "Fix SoundPool.play() looping regression in 4.3 DO NOT MERGE"

Fix SoundPool.play() looping regression in 4.3 DO NOT MERGE

The change I7370d6e59a7ef26dfb284a8b058d5ab2e0a42ccf caused a regression
in SoundPool looping when using SoundPool's streaming implementation.
This reverts a portion of that change.

Bug: https://code.google.com/p/android/issues/detail?id=58113
Bug: 10171337
Change-Id: I8af0dc8683a7c7f225c80f0eb4d39770667b52e5

Merge "Use 0 for sp<>, NULL for raw pointers"

Merge "Rename reqChannels to reqChannelMask"

Merge "Line length 100"

am 1b832c3d: Merge "Fix regression for AudioRecord streaming callback mode" into klp-dev

* commit '1b832c3d8f2b1b24f9bc7025afd2b4a8b4e6c6f9':
  Fix regression for AudioRecord streaming callback mode

am 9bdc9c4e: am bcd86896: am d0f5664a: Merge "Handle undefined NAL type for h264 streaming"

* commit '9bdc9c4ee0b44ca407cdca4499df9b3134bc5884':
  Handle undefined NAL type for h264 streaming

Fix regression for AudioRecord streaming callback mode

The notification period (frequency of client wakeups) was broken in
streaming callback mode.  Fast tracks were OK, but non-fast tracks with
normal latency weren't getting woken up at all.

Bug: 10222943
Change-Id: Ife9a2f57fa73c6eb921f1c5ba62de0bfcc20557b

Merge "Fix regression for AudioRecord streaming callback mode" into klp-dev

am bcd86896: am d0f5664a: Merge "Handle undefined NAL type for h264 streaming"

* commit 'bcd86896e486e303d285e13477e0623b2a920e78':
  Handle undefined NAL type for h264 streaming

Fix regression for AudioRecord streaming callback mode

The notification period (frequency of client wakeups) was broken in
streaming callback mode.  Fast tracks were OK, but non-fast tracks with
normal latency weren't getting woken up at all.

Bug: 10222943
Change-Id: Ife9a2f57fa73c6eb921f1c5ba62de0bfcc20557b

Merge "Pull up mBufferSize from RecordThread to ThreadBase"

Merge "Remove obsolete field RecordTrack::mAudioRecordServerProxy"

am d0f5664a: Merge "Handle undefined NAL type for h264 streaming"

* commit 'd0f5664abb5a8d94ae13f63a5f3491b47383ee2f':
  Handle undefined NAL type for h264 streaming