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

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

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

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

Restore stagefright command line tool.

Change-Id: Ia17f48522f601d6a057ef3e1e8cd01f80a5c9951

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

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

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

Support for "request-sync" parameter in ACodec

Also returns errors to caller instead of asserting...

Change-Id: Id3018655a2b3da4289167fba16af907350a511ae

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 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 "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

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

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

Merge "Handle undefined NAL type for h264 streaming"

Merge "Cleanup comments for the new control block implementation"

Merge "Adds a drm resource busy error, removes some deprecated errors"

Merge "IOMX: add updateGraphicBufferInMeta method for metadata mode"

Merge "fix build"

fix build

Change-Id: I859bd75eb796b0cb8da30ad947776d3e38f3873f

Fix build. last attempt.

Change-Id: Ib955134f90b78cfc0d31272021ad5907a41df708

IOMX: add updateGraphicBufferInMeta method for metadata mode

This is used to set the handle in the metadata buffer that is
valid in the mediaserver process, as well as to keep a reference
for the graphic buffers in BufferMeta.

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

separte producer and consumer interfaces

Bug: 9265647
Change-Id: Iefabc11e4bd2e2e8ffd31160476c450affe6629c

Merge "AudioRecord notification frames"

Merge "AudioRecord::openRecord_l use flags"

Merge "Reject unprepared MediaPlayers"

Merge "AudioRecord::openRecord_l now take flags"

Merge "RecordThread::createRecordTrack_l use flags for fast tracks"

Merge "RecordTrack::createRecordTrack_l flags are in/out"

Merge "IAudioFlinger::openRecord track_flags_t flags is in/out"

Merge "AudioRecord::set and constructor now take flags"

Merge "Make AudioRecord and AudioTrack comments more similar"

Merge "Camera3: Provide consumer usage flags to HAL for each stream"

Camera3: Provide consumer usage flags to HAL for each stream

At stream configure time, pass on the stream's consumer usage flags
to the HAL, to speed up final hardware configuration.

Bug: 9592202
Change-Id: Ie467be053be36a09e482f5f05cad65df42d66476

Merge "AudioRecord callback thread waits for priority boost"

Reject unprepared MediaPlayers

Don't allow a MediaPlayer that has been reset() or release()d to be
used as the argument to setNextMediaPlayer.

Change-Id: I47da1460ec3742f5b2bd7b79e7998b290032d5a1

Merge "Remove default parameters for AudioTrack"

Merge "AudioRecord has no default sample rate"

Merge "Recording of non-linear formats is not supported"

Adds a drm resource busy error, removes some deprecated errors

Change-Id: Id719fb6df9c9d955ac24803082d08f2a4a5bc766
related-to-bug: 9695816

AudioRecord notification frames

Change-Id: I76ec536d1504eb9a558178b62bf225aace4b40d1

AudioRecord::openRecord_l use flags

Use the flags to determine input parameters for IAudioFlinger::openRecord.

Change-Id: I98d2726503af75c8830ce80ceaf3b94a755b342f

AudioRecord::openRecord_l now take flags

The new parameter 'flags' of type audio_input_flags_t will be used for
requesting a fast track, but is currently ignored.

Change-Id: If68dfda8b2d4eaaca42927d721b4630c47f71f3b

RecordThread::createRecordTrack_l use flags for fast tracks

Look at client's request for a fast track in the flags parameter,
and check whether the request can be satisfied.

Change-Id: I65ddaeb1b85670fe9066076f638bfed0cda01c0d

RecordTrack::createRecordTrack_l flags are in/out

The flags are currently unused, but will be used for requesting a fast
track.  Making flags in/out will allow reporting back up to client that
the request is denied.

Change-Id: Ifbee57da3632ce130551065a426577fb97b1a68d

IAudioFlinger::openRecord track_flags_t flags is in/out

This will allow AudioFlinger to tell client it is denying a request.

Change-Id: Iff2be3ad6636371bbda9c9899a283c94620c1f06

AudioRecord::set and constructor now take flags

The new optional parameter 'flags' of type audio_input_flags_t will be
used for requesting a fast track.

Change-Id: Ia7e070cb57c833e608352da354fb30dc26df6918

AudioRecord callback thread waits for priority boost

Change-Id: Iae38fa4ac20a45751566169213a08a15deb0a2f6

Recording of non-linear formats is not supported

Such formats are already rejected in AudioRecord::set()

Change-Id: I5ba1fd9e4cd659e5226c75aa4f63e52f655e0521

Cleanup comments for the new control block implementation

There was some obsolete and incomplete text left over
after the new control block code was submitted.
This cleans up all those comments to be accurate again.

Change-Id: Ic52f5869cb723cde25d709514d6deea6aa6f20aa

AudioRecord has no default sample rate

Change-Id: I72feefdd6f3a623fd3669b80d4b264518fdc0929

Remove default parameters for AudioTrack

The defaults aren't needed; all users must know these values.

Change-Id: I0c1c97cb80bfea8e69a7cfa53ec85a91a8d48bb5

Make AudioRecord and AudioTrack comments more similar

Change-Id: I122a7cf7bfc162090cb27f37c325db7a23985bc7

Merge "Use screen dimensions as default video size"

Use screen dimensions as default video size

Instead of 720p, use the screen dimensions as the default video
size.  For example, a phone with a 768x1280 display would be
recorded as a 768x1280 video when in portrait, or a 1280x768
video when in landscape.

If the device is not able to record at that resolution, we
fall back to 720p.

Also, fixed a problem introduced by a buffer timestamp change.

Change-Id: I7230ddb323c669ab07327f26df8d103c192c21d3

Merge "fix fallout from binderizing BufferQueues consumer side"

Merge "Report underruns for fast tracks also"

fix fallout from binderizing BufferQueues consumer side

Change-Id: I626bac6df4fc3d8478046193f06ecc7ea60dd3a8

am 6d4d8790: am b687ff9f: Merge "Fix mismatched delete/new couple"

* commit '6d4d879037a495f9a4276a6996d1845594ce9090':
  Fix mismatched delete/new couple

am b687ff9f: Merge "Fix mismatched delete/new couple"

* commit 'b687ff9f2e02790e8a8e3964cb73e87546cd72dd':
  Fix mismatched delete/new couple

Merge "Fix mismatched delete/new couple"

Merge "Camera: fix the metadate parcel write issue"

Unregister any handlers still registered on now "dead" ALoopers

upon the death of an ALooper.

Change-Id: I64c0835b8db04486204f3d0fa7173ee53708a116
related-to-bug: 10106648

Camera: fix the metadate parcel write issue

Also rename the frameId to requestId to make it less confusing.

Change-Id: I14a20b6fcabb86294afb31fc0c56397e185c1373

Report underruns for fast tracks also

This fixes a regression that was introduced earlier
by commit 9f80dd223d83d9bb9077fb6baee056cee4eaf7e5
called "New control block for AudioTrack and AudioRecord".
That commit broke underrun reporting for fast tracks.

Also remove Track::mUnderrunCount, which counted the number of underrun
events, and was only used by dumpsys media.audio_flinger.

Now dumpsys media.audio_flinger reports the number of underrun frames,

Isolated underrun-related control block accesses via the proxy, so that
the server is not directly poking around in the control block.

The new proxy APIs are AudioTrackServerProxy::getUnderrunFrames() and
AudioTrackServerProxy::tallyUnderrunFrames().  getUnderrunFrames() returns
a rolling counter for streaming tracks, or zero for static buffer tracks
which never underrun, but do a kind of 'pause' at end of buffer.
tallyUnderrunFrames() increments the counter by a specified number of frames.

Change-Id: Ib31fd73eb17cbb23888ce3af8ff29f471f5bd5a2

am b57fb786: am d0ef1ccd: Merge "rtsp handle response line ended with \'\n\'"

* commit 'b57fb786a32d4ea78cd8bbf24a65593353d87a88':
  rtsp handle response line ended with '\n'

am d0ef1ccd: Merge "rtsp handle response line ended with \'\n\'"

* commit 'd0ef1ccd987e650e025b840e72f91770d03a5110':
  rtsp handle response line ended with '\n'

Merge "rtsp handle response line ended with '\n'"

rtsp handle response line ended with '\n'

Change-Id: I5bfafd3fa2c95083e833da2846556282eada2b02
Signed-off-by: Yajun Zeng <beanz@marvell.com>

Merge "Add resampler comments and fix a typo"

Add resampler comments and fix a typo

Change-Id: Ie071673875f663de4212eed4a4dff89d51a5a915

Remove default channel mask in AudioRecord constructor and set()

Change-Id: I22ad4ba8777842bf6705e79c6ad796fdb9a4104c

Camera: Rename new API to camera2, rearrange camera service

 - Support API rename from photography to camera2
 - Reorganize camera service files
   - API support files to api1/, api2/, api_pro/
   - HAL device support files into device{1,2,3}/
   - Common files into common/
   - Camera service remains at top-level

Change-Id: Ie474c12536f543832fba0a2dc936ac4fd39fe6a9

Move control block mName to createTrack() output

This is part of a series of CLs to clean up the shared memory
control block, by removing any fields that don't have to be there.

Change-Id: I6e51003a1293b6800258c31b22cff2eba42162e7

Rename control block flags to mFlags

Change-Id: I7b6d31e24531954ab1ecdf3ed56c19433700bd89

Merge "AudioRecord and HAL input stream must be 16-bit PCM only"

Rename control block server to mServer and add comments

Change-Id: Ieabd91acee92d0e84e66fbd358df5282b856306e

Use correct type for OutputDescriptor::format

Change-Id: Ide608ef452d57da29b708180d90470361c123d1d

Merge "Fix theoretical race using TrackBase::sampleRate()"

Fix mismatched delete/new couple

In RecordThread::readInputParameters(), mRsmpInBuffer is allocated
with new[] and should be freed with delete[] instead of delete.

This error has been found by valgrind.

Change-Id: I2f947abb658a21a7d1e61eebb99270cb37d2d147
Signed-off-by: Andrei V. FOMITCHEV <andreix.fomitchev@intel.com>
Signed-off-by: David Wagner <david.wagner@intel.com>
Signed-off-by: Jack Ren <jack.ren@intel.com>
Signed-off-by: Bruce Beare <bruce.j.beare@intel.com>

Remove CC_LIKELY and CC_UNLIKELY where not needed

Only keep them in performance-sensitive code

Change-Id: Ib257ddd7bc39ce9896997ffae008c524ac743d01

Rename RecordThread::mInputBytes to the more generic mBufferSize

This prepares for using it in PlaybackThreads later

Change-Id: Id90a92aa6372e4b69914b0008cef07296ca5d6a3

Move members from ThreadBase to PlaybackThread

Move mNormalFrameCount and frameCountHAL(), since they're not used by
record threads.  Also comment which fields are updated by readParameters().

Change-Id: I5fc0a8a89cc637976f22d49271a5a3e136dab4e1

Merge "Fix typo in AwesomePlayer"

Merge "Use standard name and type for channel mask"

Merge "camera2 api: Override default RGBx formats to IMPLEMENTATION_DEFINED"

Merge "Simplify getParameters()"

Merge "Treat mChannelCount as uint32_t consistently"

Merge "Fail more gracefully on version mismatch"

Treat mChannelCount as uint32_t consistently

mChannelCount was 8-, 16-, or 32-bits

Change-Id: I2cc2fedf3e33144e5c8bbd9894763282d9217f63

camera2 api: Override default RGBx formats to IMPLEMENTATION_DEFINED

Bug: 9487482
Change-Id: I09d3b4e41454d350c4bc7b1f8e893c3dad655e73

Fail more gracefully on version mismatch

b/9900647

Change-Id: I9ea508a2685ff8adc780edd5ecec30dd1a9b0997

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

Merge "Declare methods in binder opcode order"

Declare methods in binder opcode order

Change-Id: I5f624b7a51ffe1a17a67c056cf984f74e4c56eac

Merge "Remove obsolete mBufferEnd"