am 5eab65aa: am ff58698e: am f36ae7ca: Merge "Added support for 3GPP meta data atoms"

* commit '5eab65aa7bd40e907d11dca8aad69714ffef19f1':
  Added support for 3GPP meta data atoms

am ff58698e: am f36ae7ca: Merge "Added support for 3GPP meta data atoms"

* commit 'ff58698ec7e0f32bf9f3e1758173fe170e17270c':
  Added support for 3GPP meta data atoms

am f36ae7ca: Merge "Added support for 3GPP meta data atoms"

* commit 'f36ae7ca47dff5a5266f633bb36bf92308eec3f8':
  Added support for 3GPP meta data atoms

Merge "Added support for 3GPP meta data atoms"

am a8542f05: am e9f3a314: Merge "Refactor CameraService to handle errors properly." into klp-dev

* commit 'a8542f05253e31a64428baf93a31183988a065bb':
  Refactor CameraService to handle errors properly.

am e9f3a314: Merge "Refactor CameraService to handle errors properly." into klp-dev

* commit 'e9f3a3141c2eaafa3a5a3417d1e5682a909c5b31':
  Refactor CameraService to handle errors properly.

Merge "Refactor CameraService to handle errors properly." into klp-dev

Refactor CameraService to handle errors properly.

Bug: 10361136

-Connect calls now return status_t error flags.

Change-Id: Idca453b111e5df31327f6c99ebe853bb2e332b95

Don't lock and then immediately unlock effect chains

Change-Id: Id9a4b1e9c49cecfd3a008a7814b5c14dad9a5831

Fix races in reading activeTrack->mState without lock

Change-Id: I8cfbd9884fa5959a846ddb52cdc797266b896769

Simplify flow by having all mActiveTrack.clear() continue

Change-Id: Icd160b0aa79a747bd19fabaa7feee813afb155e0

Don't sleep with effect chains locked

Change-Id: I6eae037304d3dc8a8f3ef55c5b210fd4f8856e99

Copy mActiveTrack for stability and easier access

Change-Id: If0d88610ec35128054a22f42b2d79f07c4b724eb

Simplify code by pulling up mActiveTrack == 0 test

Change-Id: I225bb032b2fde97d891191f3599d9850c2d27bb2

Merge "Remove dead 'if'"

Merge "Fix race in checking exitPending() in RecordThread::threadLoop()"

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

Remove dead 'if'

Change-Id: I23c082f0115ae598cad293be0ac66a130afb692a

Fix race in checking exitPending() in RecordThread::threadLoop()

Change-Id: I97a1c5ab17a6c334d275be794392dcf6c9a88d0e

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"

Added support for 3GPP meta data atoms

3GPP metadata is not supported, only iTunes Metadata
Format is supported.

Change-Id: I78b41c88788d5a43800c936e0ac0f98424515c62

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