am 192cbbad: Fix audio track pause.

* commit '192cbbad773979a6fe3b5a0c223356de4fc3309c':
  Fix audio track pause.

Fix audio track pause.

AudioTrack::pause() should signal the control block condition
to release threads waiting for available buffers in obtainBuffer().
Otherwise the behavior relies on the timout on the condition
or the fact that audioflinger will mix a new audio buffer while executing
the pause.

Bug 6653769.

Change-Id: I5f8f73c471fe306070f30b814f32fd4b4dc1d575

am 82233b4b: Merge "Better handle invalid duration in Xing/VBRI tags" into jb-dev

* commit '82233b4b5bee06f11618a558ae465abbfdedfd6e':
  Better handle invalid duration in Xing/VBRI tags

Merge "Better handle invalid duration in Xing/VBRI tags" into jb-dev

am e6adde40: Merge "Shorten dumpsys media.audio_flinger output" into jb-dev

* commit 'e6adde403480333b613452ae0df442df9da4a5b2':
  Shorten dumpsys media.audio_flinger output

am 49dd5cf3: Merge "Log track name on obtain/releaseBuffer warnings" into jb-dev

* commit '49dd5cf3469cd755321f8ec10013ad4fcfaf723d':
  Log track name on obtain/releaseBuffer warnings

Merge "Shorten dumpsys media.audio_flinger output" into jb-dev

Merge "Log track name on obtain/releaseBuffer warnings" into jb-dev

am e20435de: Merge "Add NOTICE and MODULE_LICENSE_APACH2 to libs build under /frameworks/av/" into jb-dev

* commit 'e20435de10f63af812da9d4f801b61a8817733a0':
  Add NOTICE and MODULE_LICENSE_APACH2 to libs build under /frameworks/av/

Merge "Add NOTICE and MODULE_LICENSE_APACH2 to libs build under /frameworks/av/" into jb-dev

am cac467b3: Merge "Fix one of the potential ANR issues from AwesomePlayer" into jb-dev

* commit 'cac467b3af9235229522a134d2c57e4fa605ba2f':
  Fix one of the potential ANR issues from AwesomePlayer

Merge "Fix one of the potential ANR issues from AwesomePlayer" into jb-dev

Better handle invalid duration in Xing/VBRI tags

b/6654637

Change-Id: Ie92eddd84ef5914b77dbadaacd80b4de6cd66e1c

Add NOTICE and MODULE_LICENSE_APACH2 to libs build under /frameworks/av/

Change-Id: I0a3af3e2abdedebd5934f3d941d01c32cfc75e26
related-to-bug: 6647465

Shorten dumpsys media.audio_flinger output

Don't include in FastMixer if it's not present.
This removes confusing clutter especially for devices
with a separate deep buffer output thread, or for the
duplicating thread and A2DP output thread.

Change-Id: I84933f5555593256f11ba0895ec915c09cf16963

Log track name on obtain/releaseBuffer warnings

This should help diagnose problems by allowing us to correlate
the logs with the dumpsys media.audio_flinger output.

Change-Id: I8c7c592b4f87d13b0f29c66ce7a2f301a0f063c9

Fix one of the potential ANR issues from AwesomePlayer

Change-Id: I8ac7bbb5fe219f65bdbdd223538fbac0575e2957
related-to-bug: 6565826

am b4ea1ab8: Merge "stagefright: fix AudioRecord callback buffer size" into jb-dev

* commit 'b4ea1ab821d652cb080910d4081f5f1318597a90':
  stagefright: fix AudioRecord callback buffer size

Merge "stagefright: fix AudioRecord callback buffer size" into jb-dev

am 958f463c: Add NOTICE and MODULE_LICENSE_APACHE2 for libstagefright_soft_h264dec and libstagefright_soft_vpxdec

* commit '958f463c4cddba6437c86d5bfef68167f767ee66':
  Add NOTICE and MODULE_LICENSE_APACHE2 for libstagefright_soft_h264dec and libstagefright_soft_vpxdec

Add NOTICE and MODULE_LICENSE_APACHE2 for libstagefright_soft_h264dec and libstagefright_soft_vpxdec

Change-Id: Id4f1ef8dfc21731846a9862b8d12940a92fe99bb
related-to-bug: 6647465

stagefright: fix AudioRecord callback buffer size

Make sure that the maximum number of frames passed to
AudioSource by the AudioRecord callback always fits within
the maximum buffer size defined by kMaxBufferSize.

Also make sure that the total AudioRecord buffer size is more
than the minimum required.

Change-Id: I26a1f998e0cf75ac88b02e67ec9d8db3c0cca193

am bd961f4c: Merge "Add NOTICE and MODULE_LICENSE_APATCH2 to libvideoeditor" into jb-dev

* commit 'bd961f4c4d51c4fcfc4722d02383eb4263ef0ac5':
  Add NOTICE and MODULE_LICENSE_APATCH2 to libvideoeditor

Merge "Add NOTICE and MODULE_LICENSE_APATCH2 to libvideoeditor" into jb-dev

Add NOTICE and MODULE_LICENSE_APATCH2 to libvideoeditor

Change-Id: I19308c5fb0f39f8e696092186f4386e3dad06894
related-to-bug: 6647465

am 10cf121f: Merge "Update make file" into jb-dev

* commit '10cf121f7285e23c37264dab3bad7cffefd754b1':
  Update make file

am b7706712: Merge "Fix gapless for deep buffers" into jb-dev

* commit 'b7706712938c55706c87724af7f6d2f2156b8d1b':
  Fix gapless for deep buffers

am 58c8db6b: Merge "Fix gapless playback" into jb-dev

* commit '58c8db6b7667db7a3bdb9b4bdd973091a5b0202f':
  Fix gapless playback

am 8a61dc02: Merge "Revert "Added some informational log messages to track down ANR related issues"" into jb-dev

* commit '8a61dc02dbbc325dee51f2e9c9f0bbc238759ec5':
  Revert "Added some informational log messages to track down ANR related issues"

am c15d6657: Add audio watchdog thread

* commit 'c15d6657a17d7cef91f800f40d11760e2e7340af':
  Add audio watchdog thread

Merge "Update make file" into jb-dev

Update make file

Change-Id: I7dbd128245f584df2689457df0dd12ce888cf8ef

Merge "Fix gapless for deep buffers" into jb-dev

Fix gapless for deep buffers

Forward current flags to next output.
b/6645676

Change-Id: I584f7a67472d99cbd6bc38f3fe84763331d115d1

Merge "Fix gapless playback" into jb-dev

Fix gapless playback

On some devices the actual framecount per buffer of an AudioTrack
will be different than what was requested, which prevented the
track from being reused. Now we create a new AudioTrack with the
requested parameters, and then compare it to the track we already
have. If they match, we throw away the new track and reuse the
existing one.
b/6644559

Change-Id: Id3e8c4460436f52e59b98ecaeb01c94f02877c1d

stagefright aacenc: Move system headers before local headers

This avoids issues with the encoder's own definition of __inline,
which conflicts with headers brought in by string.h in glibc.

Change-Id: I9e3fbf8d892529940ea67a46e67a9abe71d9c5fa

Merge "Revert "Added some informational log messages to track down ANR related issues"" into jb-dev

Add audio watchdog thread

Change-Id: I4ed62087bd6554179abb8258d2da606050e762c0

Revert "Added some informational log messages to track down ANR related issues"

This reverts commit c558fa89016ba9583049b84dc57e66a913cabde8.

related-to-bug: 6565826

Reconcile with jb-release

Change-Id: Id132825de97a186047a6189a92abde9f5c087a5a

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

stagefright aacenc: Add parentheses in expression with mixed && and ||

This doesn't maintain the exact behaviour, but seems closer to
what the original author probably intended.

This avoids a warning.

Change-Id: I191292faa0a25f715f2b7d46ec770827e2ac0b7b

am b7acdfb8: Merge "Fix green or corrupted video frames in the exported movies" into jb-dev

* commit 'b7acdfb8068bf408ed859dfdd441b4a8722eb12a':
  Fix green or corrupted video frames in the exported movies

Merge "Fix green or corrupted video frames in the exported movies" into jb-dev

Reduce underruns in screen off, esp. with EQ

Add MonoPipe APIs to specify setpoint.
Use screen state to configure pipe setpoint.
Fix a long-standing bug where pipe sleep time was excessive,
  which interacted poorly with governor and low clock frequencies.
  Now it deducts the elapsed time since last write(),
  which was significant when there was EQ and low clock frequency.

Bug: 6618373
Change-Id: I6f3b0072c2244aeb033ef0795ad164491a164ff5

am 28ed2f93: Reduce underruns in screen off, esp. with EQ

* commit '28ed2f93324988767b5658eba7c1fa781a275183':
  Reduce underruns in screen off, esp. with EQ

Reduce underruns in screen off, esp. with EQ

Add MonoPipe APIs to specify setpoint.
Use screen state to configure pipe setpoint.
Fix a long-standing bug where pipe sleep time was excessive,
  which interacted poorly with governor and low clock frequencies.
  Now it deducts the elapsed time since last write(),
  which was significant when there was EQ and low clock frequency.

Bug: 6618373
Change-Id: I6f3b0072c2244aeb033ef0795ad164491a164ff5

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

Fix green or corrupted video frames in the exported movies

At least two use cases are affected: trimming and transition clip generation.

If the starting time of the transition or the trimming position is not the same as
that of any existing video frame, the video frame right before the transition
starting position or the trimming position must be encoded as the the first video
frame for the transtion or the trimmed video, and it is encoded as a key frame.
This frame was not saved without the patch.

In the transcoding phrase, since we could not find such a frame, the existing
code did not return an error, and incorrectly used some other frame to serve
as the reference key frame for the remaining video frames of the transition or
the trimmed video. As a result, we may see corrupted or green video frames in
the exported movies.

The fix is to always save the frame right before the trimming position or
the transition starting time, so that a key frame will always be encoded, and
the right reference frame is used for remaining video frames. In addition,
checks are added to prevent from future mistakes to use incorrect reference frames.

Patch was originally contributed by teng.hong@nxp.com

Change-Id: I182784d4c1786224445224634edab3338bf2278f
related-to-bug: 5713461

am a4f7e0e9: audioflinger: fix duplicating thread standby

* commit 'a4f7e0e9a0e92a063f1b3a08988cf46e2cf1fa94':
  audioflinger: fix duplicating thread standby

audioflinger: fix duplicating thread standby

The code that waits for a track presentation to be complete
before disabling it caused a regression for duplicating threads.
Because of the way output tracks activity is managed, the number
of frames output by the duplicating thread would never
reach the target set for a track to be considered presented.
The track would not be removed from active list and the thread would
not go to standby and keep its wakelock held.

Bug 6606922.

Change-Id: I4b46b420ac4cbf79a86b6791ae6589d407b01c92

am e35a55fb: Merge "Take latency and current time into account for visualization" into jb-dev

* commit 'e35a55fbdff5a695ee35ff43f1466a62fb75662b':
  Take latency and current time into account for visualization

Merge "Take latency and current time into account for visualization" into jb-dev

am 9a0d82df: audioflinger: modify fast mixer overrun management

* commit '9a0d82df8fd681257bc5b29fb5e6c65fcc7cef5a':
  audioflinger: modify fast mixer overrun management

Take latency and current time into account for visualization

Buffer more data, and return the data that is currently being
output from the audio output, to ensure that visualizations are
smooth and responsive even when the audio output has a large
latency and/or large buffers.
b/6522728

Change-Id: I401637f01be7600b3c594a55c869036c13b206c0

Reconcile with jb-release

Change-Id: I08692b2be6cd851d6dcfc1a56113837cb2cc219e

audioflinger: modify fast mixer overrun management

Do not force sleep when an overrun occurs. This was for non
blocking audio HAL and is unused.
Keep statistics on overruns.

Bug 6605215.

Change-Id: I295455d4562201aa3814c88bdd6e52c9226712af

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

am 67c0a58e: audioflinger: various fixes on direct output

* commit '67c0a58e05f4c19d4a6f01fe6f06267d57b49305':
  audioflinger: various fixes on direct output

audioflinger: various fixes on direct output

Various fixes in direct output playback thread implementation:
- threadLoop_write() was broken for playback threads that do not
use a pipe sink.
- output buffer size calculation was hard coded for stereo.
- removed software volume that was implemented for PCM stereo
format only: the audio HAL has to implement volume if needed
for direct outputs.

Change-Id: If211b4489be9af395435707b8cf0388cce1347b2

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

am ab9071b8: audioflinger: fix active sleep time calculation.

* commit 'ab9071b8d1b375418eb797c9a790da71de644344':
  audioflinger: fix active sleep time calculation.

audioflinger: fix active sleep time calculation.

When an audioTrack underruns, the audioflinger mixer thread
sleeps for a certain time to give the app a chance to recover.
This time is based on the reported audio HAL latency.
Some audio HALs implementing deep buffering have a variable
latency and this creates a problem if the sleep time is cached
when the output stream is opened and not updated afterwards.

This change derives the active sleep time from the mix buffer
framecount instead of the latency. This is more conservative
but works for variable latency.

Bug 6588525.

Change-Id: Ia892fc290fe06f836565c3ae15f7a2ce026c88c6

am 39993085: State queue dump

* commit '399930859a75d806ce0ef124ac22025ae4ef0549':
  State queue dump

State queue dump

Bug: 6591648
Change-Id: Iac75e5ea64e86640b3d890c46a636641b9733c6d

am 1295bb4d: Fast track dumpsys

* commit '1295bb4dcff7b29c75cd23746816df12a871d72c':
  Fast track dumpsys

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

am bf0d21fb: Count underruns for normal tracks also

* commit 'bf0d21fb1310e8677caa53b90e8c3aecebc7fc13':
  Count underruns for normal tracks also

Fast track dumpsys

Bug: 6591648
Change-Id: I696f51c682e7233ba690d97da26012084989b412

am 81787e56: Merge "Was calculating measured warmup time wrong" into jb-dev

* commit '81787e56f8f21407eade5accc50c6da39b8af9b2':
  Was calculating measured warmup time wrong

Count underruns for normal tracks also

Bug: 6591648
Change-Id: Iff9cabe392bb2ce97062603adb9c9dc7aa4170d5

Merge "Was calculating measured warmup time wrong" into jb-dev

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

am 7dc5b165: Merge "Fix fast track leak if out of normal track names" into jb-dev

* commit '7dc5b165d4f98a2059eaf1c7fec20b3b5773cd4c':
  Fix fast track leak if out of normal track names

am 0c22272f: Merge "Pipe throttle based on requested frame count" into jb-dev

* commit '0c22272f1c97fd4b7e106030a1fd5e6b50680b61':
  Pipe throttle based on requested frame count

Merge "Fix fast track leak if out of normal track names" into jb-dev

Merge "Pipe throttle based on requested frame count" into jb-dev

Was calculating measured warmup time wrong

Bug: 6592923
Change-Id: I980c6fc69d78c569147ea44ce33a7f8a41739c91

am dadc0ddb: Part of fix for double spins & faster startup

* commit 'dadc0ddbfe309086801b479ddc91c55eeb2a9cbb':
  Part of fix for double spins & faster startup

Part of fix for double spins & faster startup

This change alters the way that the media player
interacts with the Widevine adaptive streaming buffer
logic.  It eliminates the reliance on cached buffer
duration to determine pause/play states and instead
only generates buffering events when the widevine
library is not producing data (i.e. when it is
buffering).  This eliminates unnecessary pause and
rebuffer cycles, reducing startup time and the
frequency and duration of spinners.

Multi-repo commit, depends on related changes in vendor/widevine

Change-Id: I387db2decd83841775a8351bddf5a6381fe4104f
related-to-bug:6503294
related-to-bug:6463780

am 27d33134: Merge "Do not keep audio wake lock when apps underrun." into jb-dev

* commit '27d33134130379cc91d936cdc742700311901a9d':
  Do not keep audio wake lock when apps underrun.

Merge "Do not keep audio wake lock when apps underrun." into jb-dev

am cafc53c0: Fix another buffer overflow issue

* commit 'cafc53c0ac77c2aa7564cea26120da6bc0c589f2':
  Fix another buffer overflow issue

Fix another buffer overflow issue

Using hard-coded length value easily leads to buffer overflow.
Refactor the code a bit to make it more readable, and more
extensible, and less subject to the buffer overflow coding error.

Patch originally contributed by teng.hong@nxp.com

Change-Id: Id262915302ccea8ae3b0121bf39890ab463aeeb7
related-to-bug: 6328360

Fix fast track leak if out of normal track names

Bug: 6580402
Change-Id: I3ac7f012062c35833147f47ba822eb4bf532a824

Do not keep audio wake lock when apps underrun.

Do not automatically restart an AudioTrack after an underrun
if the callback is executed but no data is written by the app.

Bug 6541286.

Change-Id: I11e7ab8dc968d7ff087058fec68f44490d3a7731

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

am 83410a85: Avoid prematurely terminating parsing when some track only has a single access unit

* commit '83410a85993ad6f5f0c122036ff0bda42bf1d4f7':
  Avoid prematurely terminating parsing when some track only has a single access unit

Avoid prematurely terminating parsing when some track only has a single access unit

Patch was contributed by teng.hong@nxp.com

Change-Id: I0f56361d839f18627e512d18e86f51b58b9b0e05
related-to-bug: 6240789

Pipe throttle based on requested frame count

Now does throttling based on the requested frame count, but
still permits overfilling pipe up to the rounded-up frame count.
Also still uses the larger frame count for non-blocking arithmetic.

Bug: 6585323
Change-Id: Ic2420a071419905524a14c22ebe1ce0e32d70a9d

am 3bdb4fbf: Merge "audioflinger: fix effect problem during underrun" into jb-dev

* commit '3bdb4fbfc6730b6edc0b1ee980141574ed54ed8d':
  audioflinger: fix effect problem during underrun

Merge "audioflinger: fix effect problem during underrun" into jb-dev

am cc9833b5: Only send port settings changed if the number of channels and sample rate are both known to be valid.

* commit 'cc9833b5db0e96f12daddb90a747fd146627377d':
  Only send port settings changed if the number of channels and sample rate are both known to be valid.

Only send port settings changed if the number of channels and sample rate are both known to be valid.

Change-Id: I9f8813eedc95db202488d8860cf485bdb8e69e11
related-to-bug: 6553089

audioflinger: fix effect problem during underrun

When an audio track underruns, the input buffer of the
corresponding effect chain (if any) must be cleared, otherwise
audio from previous mixer run will be fed again to the effect process
function.

Bug 6551652.

Change-Id: I5cd02196745f756c85af82d6937e9dc54369b37f

Reconcile with jb-release

Change-Id: Ia7adbc708c661ef21a54ae85315de1b061d3cd53

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

Added some informational log messages to track down ANR related issues

Change-Id: Ie991c86e75d58f7eb4c9f524815c4de054d5f262
related-to-bug: 6565826

am 4364d2d1: Merge "Precautionary logging to identify  a potential infinite loop." into jb-dev

* commit '4364d2d1fd712faed7e839b58fedba3f48f626d0':
  Precautionary logging to identify  a potential infinite loop.