am feb6d27b: Merge "Workaround slow AudioTrack destruction" into klp-dev

* commit 'feb6d27bf61cd266cf753215e9cae16b9bc9dbbd':
  Workaround slow AudioTrack destruction

Merge "Workaround slow AudioTrack destruction" into klp-dev

am d1b28d41: audioflniger: fix leaked audiosession references

* commit 'd1b28d41dbda203ffb420ba2e36cbe736b163ff8':
  audioflniger: fix leaked audiosession references

audioflniger: fix leaked audiosession references

acquireAudioSessionId() should not create session references
if the caller is not listed in the known notification clients.
It happens when the MediaPlayer or AudioTrack is created by the
mediaserver on behalf of a client (e.g CameraService). In this case
releaseAudioSessionId() can be called from a different pid and
the session reference will not be removed.
Also we will not be able to remove the reference if the client
process dies.

Bug: 10606426.
Change-Id: Ibb5e27518daeb484a2e66302e4c6221cbaca5b70

Workaround slow AudioTrack destruction

Bug: 10809586
Change-Id: I5f30d4deb1233e8ade8967568e40684ef680c395

am 5baf2af5: more support for audio effect offload

* commit '5baf2af52cd186633b7173196c1e4a4cd3435f22':
  more support for audio effect offload

more support for audio effect offload

Offloading of audio effects is now enabled for offloaded
output threads. If an effect not supporting offload is enabled,
the AudioTrack is invalidated so that it can be recreated in PCM
mode.

Fix some issues in effect proxy related to handling of effect
commands to offloaded and non offloaded effects.

Also fixed a bug on capture index in software Visualizer effect.

Bug: 8174034.

Change-Id: Ib23d3c2d5a652361b0aaec7faee09102f2b18fce

am 9a98b6de: Merge "Partial fix for SoundPool not terminating" into klp-dev

* commit '9a98b6de791aeb130192df10744f5b35f8b6ef1a':
  Partial fix for SoundPool not terminating

Merge "Partial fix for SoundPool not terminating" into klp-dev

Partial fix for SoundPool not terminating

SoundPool was waiting for EVENT_UNDERRUN only to indicate end of clip.  In
J, AudioTrack delivered both EVENT_UNDERRUN followed by EVENT_BUFFER_END.
However, as of K, AudioTrack is only delivering EVENT_BUFFER_END (this
lack of EVENT_UNDERRUN is another bug which still needs to be fixed).
The workaround is to also respond to EVENT_BUFFER_END in SoundPool.

Bug: 10787103
Change-Id: Id68a23bddd6dd9df6c49c55138197260d71ca468

am ab6c93b4: Merge "Use changed MediaCodec.PARAMETER_KEY_VIDEO_BITRATE value" into klp-dev

* commit 'ab6c93b4bea96dc4f44a7acfefee1e11a3abda5b':
  Use changed MediaCodec.PARAMETER_KEY_VIDEO_BITRATE value

Merge "Use changed MediaCodec.PARAMETER_KEY_VIDEO_BITRATE value" into klp-dev

Use changed MediaCodec.PARAMETER_KEY_VIDEO_BITRATE value

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

am 6cc3a994: LoudnessEnhancer audio effect implementation

* commit '6cc3a9948b51193dfdcb0c3527d7f3d1ca38aa3c':
  LoudnessEnhancer audio effect implementation

LoudnessEnhancer audio effect implementation

Implementation based on DRC effect, controlled by a
 target gain.
 The target gain is used to amplify the signal at
 the input of the DRC, and to compute the knee
 of the DRC.

Bug 8413913

Change-Id: I386d64793a9fa3f7218e053d6f0a99f6836c02bd

am 18f86140: Merge "Fix underruns when fast track denied due to SRC" into klp-dev

* commit '18f861404efc054da0a2ea6c582e293940f63bc8':
  Fix underruns when fast track denied due to SRC

Merge "Fix underruns when fast track denied due to SRC" into klp-dev

am fc270954: am e56f3c96: Merge "strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe."

* commit 'fc270954192ef7e15ac2c88daadd8890d22096e3':
  strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe.

am e56f3c96: Merge "strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe."

* commit 'e56f3c96fa6e7550b67e8b049f999aaa2ada1192':
  strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe.

Merge "strtok stores its values in thread local storage. So it can not guarantee works well when multithread environment. AudioFlinger has multithread. so strtok_r is more safe."

Fix underruns when fast track denied due to SRC

OpenSL ES requests a fast track.  If sample rate conversion is needed,
the request is denied by server, and a larger client buffer is used
to handle the higher latency of a normal track.  However the client
notification period was calculated based on buffer being divided into
2 sub-buffers.  That resulted in the notification period being too long.
The server pulls chunks that are smaller than half the total buffer.
So now the client uses 3 sub-buffers when there is SRC.

Also removed the 'defer wake' optimization because it was incorrect.
This optimization attempted to reduce the number of wakeups of client,
when server releaseBuffer knows that another releaseBuffer will be
following.  But there is no way for the first releaseBuffer to predict
how soon the second releaseBuffer will occur.  In some cases it was
a long time, and the client underran.  So now the client is woken up
immediately if the total number of available frames to client is >=
the minimum number the client wants to see (the notification period).

Also fix bug where minimum frame count was not being used in the
calculation of notification period.

Bug: 10342804
Change-Id: I3c246f4e7bc3684a344f2cf08268dc082e338e2a

strtok stores its values in thread local storage.
So it can not guarantee works well when multithread environment.
AudioFlinger has multithread.
so strtok_r is more safe.

Change-Id: I6d77ef9cc49a4478dd856dcdca14e4920ce955c6

am bc910e5c: Merge "Fix hang" into klp-dev

* commit 'bc910e5c8d05f24531bd6b11379cf41cc73415ed':
  Fix hang

Merge "Fix hang" into klp-dev

am a099d1f6: Merge "Camera3: Always set a trigger ID for an active trigger" into klp-dev

* commit 'a099d1f6be3fbbb1af4c12b2ab627397f0826982':
  Camera3: Always set a trigger ID for an active trigger

Merge "Camera3: Always set a trigger ID for an active trigger" into klp-dev

Fix hang

Specify that the surface is controlled by the app, to avoid a hang.
b/10531761

Change-Id: Idccc2c73aa3d368d8e7fbdc071ce36e2382efea4

am 72a2f171: Merge "Camera: setCallBackWindow only if callback surface is used" into klp-dev

* commit '72a2f171ddf5c1eab7d6ccd47ac2b7ac4fc9c087':
  Camera: setCallBackWindow only if callback surface is used

Merge "Camera: setCallBackWindow only if callback surface is used" into klp-dev

am a5652e9c: Merge "Add FLAG_CAN_SEEK_BACKWARD and FLAG_CAN_SEEK_FORWARD see flags" into klp-dev

* commit 'a5652e9cdaf6624e03a7e84c40590a42e44c5217':
  Add FLAG_CAN_SEEK_BACKWARD and FLAG_CAN_SEEK_FORWARD see flags

Merge "Add FLAG_CAN_SEEK_BACKWARD and FLAG_CAN_SEEK_FORWARD see flags" into klp-dev

am 66281c3a: Camera: Don\'t overwrite the error status

* commit '66281c3a5d7eea486ddc6ad30088ba92956fd4fd':
  Camera: Don't overwrite the error status

Camera3: Always set a trigger ID for an active trigger

This is to work around HAL implementations that expect to see an
ID for every trigger.

Use a dummy trigger ID of 1, since 0 has special meaning for older
HALs.

Bug: 10720617
Change-Id: I9cb1b8dbcb113cb9e737e5adb04032fd61a54c12

Camera: setCallBackWindow only if callback surface is used

Calling setCallBackWindow without checking the previewCallbackSurface could make
the stream to be deleted during callback stream operation, which is bad because
the preview is still active and the delete fails.

Bug: 10730496
Change-Id: I8addac25e5aa7901bf200d730c749bee2c088090

Camera: Don't overwrite the error status

Bug: 10749544
Change-Id: I6096effde03e18ee1b8c63cf3b36dce0344054c5

am d176ec1e: Merge "Fix SoundPool.play() looping" into klp-dev

* commit 'd176ec1e371a686c58385692a28da677e398c302':
  Fix SoundPool.play() looping

am 7f91e5e0: Merge "audioflinger: fix ro.audio.silent in offload mode" into klp-dev

* commit '7f91e5e09cf3bd8068894b2e54cbdaac70bf53a0':
  audioflinger: fix ro.audio.silent in offload mode

am d4dadf23: Merge "Camera: Limit supported preview sizes" into klp-dev

* commit 'd4dadf23dd2dd8acd514d60d84475954070340b8':
  Camera: Limit supported preview sizes

am 972a173d: audioflinger: longer offload thread standby delay

* commit '972a173d7d1de1a3b5a617aae3e2abb6e05ae02d':
  audioflinger: longer offload thread standby delay

Merge "Fix SoundPool.play() looping" into klp-dev

Merge "audioflinger: fix ro.audio.silent in offload mode" into klp-dev

audioflinger: fix ro.audio.silent in offload mode

Change-Id: I5a1a79000d53146689b0a198cc5419c36509703f

Merge "Camera: Limit supported preview sizes" into klp-dev

Camera: Limit supported preview sizes

The ANDROID_SCALER_AVAILABLE_PROCESSED_SIZES can provide sizes as large
as jpeg sizes, which could cause potential issues when trying to do
full size still capture and full size preview. This is not supported
by many devices due to hardware limitation. This change limits the
preview (as well video) size to no more than 1080p.

Bug: 10625115
Change-Id: I9467ab843553ec06e8249b4a17c0ecf4c6d6f04e

audioflinger: longer offload thread standby delay

- Increase offloaded output thread standby delay to 1 second
to allow transition between tracks with going to stanby
if reusing the same audio track (gapless)

- Make sure pause/flush/resume sequence is sent to the HAL
in the right order

- Fix format display in track dump

Bug: 8174034.
Change-Id: I43ef6f8fdbf7427e4eff6cc2d0665d7d1463ea8a

am 99bf6267: Merge "Fix MediaDrm cts tests" into klp-dev

* commit '99bf6267f0a4e2c3153be49ced59ae164d55ea6d':
  Fix MediaDrm cts tests

Merge "Fix MediaDrm cts tests" into klp-dev

am 877c673c: Merge "ACodec: fix flush/resume for decoder-output-meta-data mode" into klp-dev

* commit '877c673c3d3794e89e4677d448f38f77bdee7fdb':
  ACodec: fix flush/resume for decoder-output-meta-data mode

Merge "ACodec: fix flush/resume for decoder-output-meta-data mode" into klp-dev

ACodec: fix flush/resume for decoder-output-meta-data mode

When in decoder-output-meta-data mode, ACodec does not hold onto
buffers, but they are either with the native window, or with the
component/client.  However, for flushing we did not release the
discarded buffers back to native window (this makes sense because
they will be resubmitted shortly.)  This logic can be handled by
the normal resubmission.

Change-Id: Ic472b386422251515ef12f426e187f208f14decc
Signed-off-by: Lajos Molnar <lajos@google.com>
Bug: 10621959
Bug: 10192533

am dc33c542: Merge "Fix AudioTrack shared memory leak" into klp-dev

* commit 'dc33c542351449ceee95cc50a9009f495e7c72ba':
  Fix AudioTrack shared memory leak

Merge "Fix AudioTrack shared memory leak" into klp-dev

Fix MediaDrm cts tests

Moving the build of the mock MediaDrm plugin back to
frameworks/av since it can't be installed by cts.

b/10668350

Change-Id: Id662bdb0775389b458f86a299392217f2d959422

Add FLAG_CAN_SEEK_BACKWARD and FLAG_CAN_SEEK_FORWARD see flags

Also update seek flag in NuPlayerDriver, otherwise MediaPlayer will
get wrong flags.

Bug: 10676387

Change-Id: Ice30f27a9a04e37b4718d26228a407fea8d9e4fc

am efc4d489: Merge "audioflinger: add HOTWORD audio source." into klp-dev

* commit 'efc4d4899e03fb8464017dd48539106cec2163de':
  audioflinger: add HOTWORD audio source.

Merge "audioflinger: add HOTWORD audio source." into klp-dev

am a911f51c: Pass subtitle track properties to getTrackInfo

* commit 'a911f51c21430ac92f1d796b2338878fd98382e9':
  Pass subtitle track properties to getTrackInfo

Pass subtitle track properties to getTrackInfo

Bug: 10326117

Change-Id: I15fcc49ad02e26d7cc92e82ee670bafca62a09a7

Fix SoundPool.play() looping

This is done by configuring SoundPool for shared memory and fast track.
Previously SoundPool used a streaming track, and looping in streaming
mode relied on the ability to loop the most recently enqueued data.
That 'feature' was lost in the new implementation of streaming, so we're
now switching from streaming mode to shared memory mode.  Shared memory
mode had always been desired, but was blocked by bug 2801375 which is fixed now.

Bug: 10171337
Change-Id: I2a938e3ffafa2a74d5210b4198b50db20ad5da0e

Fix AudioTrack shared memory leak

Bug: 2801375
Change-Id: I50e15164fe310f69ea019dca5b49171a02bc6992

audioflinger: add HOTWORD audio source.

HOTWORD is a special capture audio source that uses
the same audio tuning as VOICE_RECOGNITION but allows an
active capture to be preempted.

Bug: 10640877.

Change-Id: Iea09a11cbcdbacc8d434e5230e7559dcd1b44ac0

am c728c580: Merge "Make sure the encoder instance is safely release()\'d before its" into klp-dev

* commit 'c728c580fc512b50604f33f1c4581c500062b245':
  Make sure the encoder instance is safely release()'d before its

Merge "Make sure the encoder instance is safely release()'d before its" into klp-dev

am 7f9551f7: MediaMuxer: Hook up setLocation method

* commit '7f9551f75eedb3e4e1fe8feaaba48d8080635fc4':
  MediaMuxer: Hook up setLocation method

am 3c939600: am d6ca2e8b: Merge "Uninitialized pointer can cause crash."

* commit '3c9396002f51da24f9ba8b8dc96ae06bf667ba2c':
  Uninitialized pointer can cause crash.

am d6ca2e8b: Merge "Uninitialized pointer can cause crash."

* commit 'd6ca2e8bc17db104f0421ae1e7e0799488b96d70':
  Uninitialized pointer can cause crash.

Merge "Uninitialized pointer can cause crash."

MediaMuxer: Hook up setLocation method

This method is needed when mediamuxer is used for camera video recording.

Bug: 10594784
Change-Id: I9bd006a07e5e2ac7019849e3f4f7cf7b8356d669

am 6220f993: Merge "camera3: Dump latest request sent with dumpsys" into klp-dev

* commit '6220f993541b0317e6ace3c410dd35a8d76e30bc':
  camera3: Dump latest request sent with dumpsys

Merge "camera3: Dump latest request sent with dumpsys" into klp-dev

am 6ca83fad: Merge "audioflinger: no effects on offloaded tracks" into klp-dev

* commit '6ca83fad0c32f8d9c81c9e16b4f87025d1abbda1':
  audioflinger: no effects on offloaded tracks

Merge "audioflinger: no effects on offloaded tracks" into klp-dev

am d1d6467d: Camera: Correct stream id types

* commit 'd1d6467d3bcbc1305eeba0176a2edf04925c368e':
  Camera: Correct stream id types

audioflinger: no effects on offloaded tracks

Invalidate offloaded tracks when an effect is enabled
so that the track is recreated in PCM mode and the effect
can be applied.
This is temporary until effect offloading is implemented.

Bug: 8174034.

Change-Id: I77b8b54a10db6cb8334be76d863ea7e720eaad09

Camera: Correct stream id types

Bug: 10604704
Change-Id: I223ac5d200eb6439c08955fb7ad586592099fbae

camera3: Dump latest request sent with dumpsys

Bug: 10606535
Change-Id: I0dd5ca3e09f45701659ab9ebd3b51939a2f6e6ba

am 84b7fb0c: Camera: Send VIDEO_SNAPSHOT intent for video snapshot case

* commit '84b7fb0c88ddd05ed7c148c82fe1691040a9404d':
  Camera: Send VIDEO_SNAPSHOT intent for video snapshot case

Camera: Send VIDEO_SNAPSHOT intent for video snapshot case

Bug: 9924539
Change-Id: Ief39eff79c31f2c2aa5df2fe1a0f04c4afc74690

am 3b4529e0: audioflinger: remove async write race conditions

* commit '3b4529e03c5fc7a44c22f9091ad15a269bfca3a8':
  audioflinger: remove async write race conditions

am 60c60df7: Effect Offload Proxy for effects offload

* commit '60c60df7db278d2fa5c90b0fa14f99a61d50272b':
  Effect Offload Proxy for effects offload

am a82975e0: Merge "Effects Factory changes for effects offload" into klp-dev

* commit 'a82975e02ccf78fd9c4dd2f7c1a66c254b52fe3b':
  Effects Factory changes for effects offload

Make sure the encoder instance is safely release()'d before its

reference count drops to 0.

Change-Id: I6601226e2754f17e964125cacf02dc02cfbf6231
related-to-bug: 10150263

audioflinger: remove async write race conditions

Remove possible race conditions between async callback
thread and offload thread when clearing and setting the
draining and write blocked flags.

Bug: 8174034.
Change-Id: I7af10491f39dc0e7d7414862a9d8e763daa2e2b7

Effect Offload Proxy for effects offload

Effect Proxy abstracts the sub effects to the upper layers.
It has the following functionalities:
- creation and release of sub effects
- routing the effect commands and process to the appropriate sub effect

Bug: 8174034.

Change-Id: I22d8136636048e7fe8f8807cbc6e348ffa200a22
Signed-off-by: jpadmana <rpadmanaban.jayashree@gmail.com>

Merge "Effects Factory changes for effects offload" into klp-dev

Effects Factory changes for effects offload

audio_effects.conf - commented changes to illustrate the
addition of Proxy and sub effects to the conf file

Added an effectFactoryApi - EffectGetSubEffects for querying the
sub effect descriptors from the factory. This api is used by the Proxy
to get the sub effects

Added functions and data structures in factory code for
loading the sub effects

gSubEffectList - has the Proxies and their corresponding sub effects

- addSubEffect() - reads a sub effect node and adds to the gSubEffectList
- findSubEffect() - searches through the gSubEffectList to find a SubEffect

Bug: 8174034.

Change-Id: I25b0c62b2ad523a52337128b51469e628209ea3e
Signed-off-by: jpadmana <rpadmanaban.jayashree@gmail.com>

am c23981d9: Merge "Better workaround for slow decoders." into klp-dev

* commit 'c23981d9375d4a55f5701a4090b36deecda92f92':
  Better workaround for slow decoders.

Merge "Better workaround for slow decoders." into klp-dev

am 36a8b33a: Merge "camera: Fix race condition between setParameters() and stopPreview()." into klp-dev

* commit '36a8b33a23dca49c9617e97e22e5d67f7d25c268':
  camera: Fix race condition between setParameters() and stopPreview().

Merge "camera: Fix race condition between setParameters() and stopPreview()." into klp-dev

camera: Fix race condition between setParameters() and stopPreview().

Bug: 10414772
Change-Id: I928cb3935afb101da8a7931f88c27dccdb03b4a9

am 72119ce8: Merge "Start \'am\' with exec()" into klp-dev

* commit '72119ce833d4f0e1c0342f1722d1433f2ba12b87':
  Start 'am' with exec()

Merge "Start 'am' with exec()" into klp-dev

Start 'am' with exec()

We were using system() to run 'am broadcast'.  Switch to
fork()+exec().

Bug 10606037

Change-Id: Ifae1a2b4f1914f015d09f2cac8cd0b1896d87a84

Better workaround for slow decoders.

This is more in the spirit of the original code. Now it checks
whether a codec instantiated by name is a video codec, and enables
the extra looper if so.

b/10528409

Change-Id: Ia253c04c1283d4ecf66f213ef4bf523279ad7cca

am a1991255: Merge "wifi-display: do not use HDCP\'s encryptNative method if its unsupported" into klp-dev

* commit 'a19912554f3779fac0c341ac702227bf4b5fe399':
  wifi-display: do not use HDCP's encryptNative method if its unsupported

Merge "wifi-display: do not use HDCP's encryptNative method if its unsupported" into klp-dev

am 7a4c2711: Merge "Fix CameraService error flag when running HAL1 with API2." into klp-dev

* commit '7a4c2711d7a85ce962b9d18fee41975f9c574c0b':
  Fix CameraService error flag when running HAL1 with API2.

Merge "Fix CameraService error flag when running HAL1 with API2." into klp-dev

Fix CameraService error flag when running HAL1 with API2.

Bug: 10620944

Change-Id: I44c0d09a3d313ae8f724fbe61fa5f36cf5b38b6d