Move local declarations to point of first use

Change-Id: I8a078386e371caab5174fc07b47f48fb9f21609a

am 7f8f7585: am fd477974: AudioFlinger: more fixes for offload audio lost

* commit '7f8f758506a95d6fe5c0e52aa7b5ef82fb18861a':
  AudioFlinger: more fixes for offload audio lost

am fd477974: AudioFlinger: more fixes for offload audio lost

* commit 'fd4779740ec3e9e865d5514464df26d015354388':
  AudioFlinger: more fixes for offload audio lost

AudioFlinger: more fixes for offload audio lost

mStandby being false was not a reliable indication that
something had been written to audio HAL.

Considering the last track in mActiveTracks vector as the
latest activated track was wrong as this is a SortedVector.

Bug: 11247103.
Change-Id: I397d011c5afcdd779def3fe25aaae7669a472cfc

am 241be8d5: (-s ours) am 9cae2170: Assign blame for playback wakelocks.

* commit '241be8d5874023761121043b8709a6062baac6f1':
  Assign blame for playback wakelocks.

am 9cae2170: Assign blame for playback wakelocks.

* commit '9cae217050aa1347d4ac5053c305754879e3f97f':
  Assign blame for playback wakelocks.

Assign blame for playback wakelocks.

Set a work source for the playback wakelock, so that playback is
counted against the requesting app instead of the media server.

Cherrypicked from master.

b/9464621

Change-Id: I7329f88a288a95a582a78005a1c3d16a5a611e31

Assign blame for playback wakelocks.

Set a work source for the playback wakelock, so that playback is
counted against the requesting app instead of the media server.

Change-Id: I7329f88a288a95a582a78005a1c3d16a5a611e31

am de5f7cd6: am be567be3: Merge "Check for NULL buffer in repeat-latest" into klp-dev

* commit 'de5f7cd61df74027834cd1ed2f1e482238f3ed4b':
  Check for NULL buffer in repeat-latest

am 57c1f758: am f3a6234b: Merge "Fix kAutoRampDurationUs overflow issue" into klp-dev

* commit '57c1f758d8811024aa1298d2ba244577c44ef7b9':
  Fix kAutoRampDurationUs overflow issue

am be567be3: Merge "Check for NULL buffer in repeat-latest" into klp-dev

* commit 'be567be3f92026f5964c1c9ecce00e30bb7df412':
  Check for NULL buffer in repeat-latest

Merge "Check for NULL buffer in repeat-latest" into klp-dev

Check for NULL buffer in repeat-latest

Repeating the most-recent buffer appears to fail when the producer
disconnects.  This adds a NULL check to avoid crashing.

Bug 10607920

Change-Id: Ied036004425f046fcf9c26303823681f3fddb99e

am f3a6234b: Merge "Fix kAutoRampDurationUs overflow issue" into klp-dev

* commit 'f3a6234ba21640cf2bc893dbcb98fb9b0fa47481':
  Fix kAutoRampDurationUs overflow issue

Merge "Fix kAutoRampDurationUs overflow issue" into klp-dev

am c20636f7: am e7329bd5: am d5255d1d: Merge "If dlopen("libwvm.so") fails, log the reason why."

* commit 'c20636f7a4053c6f93a8808617252afcd5dc8d50':
  If dlopen("libwvm.so") fails, log the reason why.

am e7329bd5: am d5255d1d: Merge "If dlopen("libwvm.so") fails, log the reason why."

* commit 'e7329bd564e7c5f671b786dc7d0cc684d6d11fca':
  If dlopen("libwvm.so") fails, log the reason why.

am d5255d1d: Merge "If dlopen("libwvm.so") fails, log the reason why."

* commit 'd5255d1de125d15afae6cfc35701a5255b5e2933':
  If dlopen("libwvm.so") fails, log the reason why.

Merge "If dlopen("libwvm.so") fails, log the reason why."

If dlopen("libwvm.so") fails, log the reason why.

Bug: 11342992
Change-Id: I99986b6181df6d7615fdc1520d2edaadaccd9bb4

am 173f0125: am 9857473f: am 0f4b0d23: Merge "libeffects: do not use GNU old-style field designators"

* commit '173f0125bdd54ba325068d51baa5c31c3e685b1e':
  libeffects: do not use GNU old-style field designators

am 9857473f: am 0f4b0d23: Merge "libeffects: do not use GNU old-style field designators"

* commit '9857473f48abf1333ddd3454ab3c916b4e69daa4':
  libeffects: do not use GNU old-style field designators

am 0f4b0d23: Merge "libeffects: do not use GNU old-style field designators"

* commit '0f4b0d238613cd98fe0b8931f7e6cbcc8b9ad7b1':
  libeffects: do not use GNU old-style field designators

Merge "libeffects: do not use GNU old-style field designators"

libeffects: do not use GNU old-style field designators

Not using a GNU extension improves portability

Change-Id: I4e8a0376c089828ca4fc7f1aaf3914bfd917d4c5

am 3f93c8f8: am 998c4e88: am 0219856e: Merge "libvideoeditor: fix M4OSA_Clock header guard"

* commit '3f93c8f8dbfb8b7cfed2cd21210f875497cbb39c':
  libvideoeditor: fix M4OSA_Clock header guard

am 998c4e88: am 0219856e: Merge "libvideoeditor: fix M4OSA_Clock header guard"

* commit '998c4e88f962336472c5e1c3afbf0ed8d6d3a971':
  libvideoeditor: fix M4OSA_Clock header guard

am 0219856e: Merge "libvideoeditor: fix M4OSA_Clock header guard"

* commit '0219856e5e45c0bdd20f2199297ec92f11f0875f':
  libvideoeditor: fix M4OSA_Clock header guard

Merge "libvideoeditor: fix M4OSA_Clock header guard"

libvideoeditor: fix M4OSA_Clock header guard

CLOCK was mispelled as CLOCH

Change-Id: Ie7163fc8723d7a5b9df113f55a4a550e5465d9b5

am 77177fa2: am 95c03858: Merge "Allow releaseBuffer after flush" into klp-dev

* commit '77177fa20773d02b4f9c4147ecb98107f019fa7d':
  Allow releaseBuffer after flush

am 95c03858: Merge "Allow releaseBuffer after flush" into klp-dev

* commit '95c03858e2ab4fb693a2bfe47b3caa806e43c044':
  Allow releaseBuffer after flush

Merge "Allow releaseBuffer after flush" into klp-dev

Fix kAutoRampDurationUs overflow issue

When kAutoRampDurationUs multiplies with mSampleRate, it overflows to int64.
Type cast it to int64 to make sure RampDurationUs properly.

BUg: 11162491
Change-Id: I4f93bc9acc8456e25623a9255ca7a5b206425009

Allow releaseBuffer after flush

After AudioTrack start checks for pending flush,
allow releaseBuffer on any previously obtained buffer.
For example, this can happen if the resampler has obtained
a buffer but not released the whole buffer yet.
Note that the resampler will be reading obsolete data.

Bug: 11285590
Change-Id: I0614fbb62e43604aac3089cce4b7797c87a306b5

am 6edd8ca5: am 6a51d7ed: audioflinger: fix track terminated before playing

* commit '6edd8ca5b7e3d169db4314d2b994bc9d48e9e082':
  audioflinger: fix track terminated before playing

am 6a51d7ed: audioflinger: fix track terminated before playing

* commit '6a51d7ed7062536ccc892c8850a34ed55cbc8d5c':
  audioflinger: fix track terminated before playing

audioflinger: fix track terminated before playing

When a track is terminated before any audio was written,
we should not send a drain request to the HAL. The drain
will be ignored and we will wait the async callback for ever.

Also fix a potential deadlock by not deleting a track with the
thread mutex held: mPreviousTrack changed from a sp to a raw pointer.

Bug: 11247103.
Change-Id: I0a4b47b099204c7c326d45d700234ab8f72d8eae

am e45796de: am dedc7b0f: Merge "Fix flush() followed by start()" into klp-dev

* commit 'e45796dee9c072cdb13a00b407e68be2cd55e2f2':
  Fix flush() followed by start()

am dedc7b0f: Merge "Fix flush() followed by start()" into klp-dev

* commit 'dedc7b0fa23c21b525dc8abb572bb8161ad66a9b':
  Fix flush() followed by start()

Merge "Fix flush() followed by start()" into klp-dev

am 694d0295: am 6fa20187: Make sure we get to discover that the cache is sufficiently full again

* commit '694d0295b986e4ed8f91e71cc5ac8af0341be1de':
  Make sure we get to discover that the cache is sufficiently full again

am 6fa20187: Make sure we get to discover that the cache is sufficiently full again

* commit '6fa20187e4cf9ace5692979a84741484f19e3f57':
  Make sure we get to discover that the cache is sufficiently full again

Make sure we get to discover that the cache is sufficiently full again

by posting buffering events while in the CACHE_UNDERRUN state.

Change-Id: Ic60c8d57f0ae502aa2b6eec7160dc4224b989604
related-to-bug: 11262689

Fix flush() followed by start()

Bug: 11259166
Change-Id: I72526cdfd4e3ff4b06e6a317e577f57cc285205d

am b2059ff3: am f8f15b05: Merge "Fix race condition in AudioTrack::pause followed by start" into klp-dev

* commit 'b2059ff384eee8ffb70a7ec8fc5570405201c734':
  Fix race condition in AudioTrack::pause followed by start

am f8f15b05: Merge "Fix race condition in AudioTrack::pause followed by start" into klp-dev

* commit 'f8f15b05fe051009945c9042a1a9260280e0feb2':
  Fix race condition in AudioTrack::pause followed by start

am fc24dc1a: am 787c0d04: merge in klp-release (no-op)

* commit 'fc24dc1a2e2ba8d74bfadfadb01d12a542828561':
  Camera: Skip AE precapture when possible
  Cleanup openRecord error handling
  TimedEventQueue takes a wake lock
  fix offload audio effect proxy implementation uuid
  MediaCodecInfo: report supported color formats for non-native-window mode
  Camera2 API: Hack: Always use async mode with GLConsumer
  AudioTrack: fix music resume

am dddafa84: am 3defdab8: Merge "Camera: don\'t do hw support check for ZSL" into klp-dev

* commit 'dddafa84dcbd85c11a732e7553d0b884e5a08c27':
  Camera: don't do hw support check for ZSL

Merge "Fix race condition in AudioTrack::pause followed by start" into klp-dev

am 787c0d04: merge in klp-release (no-op)

* commit '787c0d04ac4f9c938a1311f8e30d9d7ac2b8a6da':
  Camera: Skip AE precapture when possible
  Cleanup openRecord error handling
  TimedEventQueue takes a wake lock
  fix offload audio effect proxy implementation uuid
  MediaCodecInfo: report supported color formats for non-native-window mode
  Camera2 API: Hack: Always use async mode with GLConsumer
  AudioTrack: fix music resume

am 3defdab8: Merge "Camera: don\'t do hw support check for ZSL" into klp-dev

* commit '3defdab829de7df3a343393b2b277e99d071744c':
  Camera: don't do hw support check for ZSL

merge in klp-release (no-op)

Fix race condition in AudioTrack::pause followed by start

Bug: 11148722
Change-Id: Iec88f00c8510363d4418e4b8d5b34feb06ecf04d

Merge "Camera: don't do hw support check for ZSL" into klp-dev

am 7f8c3973: am 120a8847: Merge "AudioTrack: fix head position after restore" into klp-dev

* commit '7f8c397378a7ee5abd395413be71388ad36d3ed2':
  AudioTrack: fix head position after restore

am 120a8847: Merge "AudioTrack: fix head position after restore" into klp-dev

* commit '120a88471a607c85c4d60300d73c3be0a1e8f8c8':
  AudioTrack: fix head position after restore

am ace8947d: am d5766875: AwesomePlayer: fix concurrent pause and teardown

* commit 'ace8947dbcd18e86c6a089291ffe1ce32fb42508':
  AwesomePlayer: fix concurrent pause and teardown

Merge "AudioTrack: fix head position after restore" into klp-dev

am d5766875: AwesomePlayer: fix concurrent pause and teardown

* commit 'd576687570f19f3956d91c1d3da10d965e34c407':
  AwesomePlayer: fix concurrent pause and teardown

AudioTrack: fix head position after restore

The head position transfered to the new track
by restoreTrack_l() must take into account the frames that
are dropped from the old track to avoid a non recoverable
offset in the playback head position returned to applications.

Bug: 11230062.
Change-Id: I51143a08b95e8f264ed709ae2054360315f2b8b1

Camera: don't do hw support check for ZSL

Only use camera.disable_zsl_mode to control disable/enable ZSL.

Bug: 11258054
Change-Id: Ibf10b959d0913f7dedb59d89b571e9bf66fe978a

AwesomePlayer: fix concurrent pause and teardown

There was a race condition if a pause request was received
while we were executing a teardown event. Although we hold
a mutex while processing the teardown event, there is a step
of the prepare sequence where we release the mutex if streaming.
In this case, the pause request is executed but ignored because
the player state is still preparing. At the end of the teardown
event processing we restore previous playback state and resume.

The fix consists in clearing the saved teardown playback state
when a pause request is received while processing a teardown event.

Bug: 11225491.
Change-Id: If0e61855ce5a336322f1ba8e5559bdc190beeb76

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

am 8580679e: am 014e7fa2: Make Audio wakelock names less unique.

* commit '8580679e1780617d978d28a43e932314b86608ce':
  Make Audio wakelock names less unique.

am 014e7fa2: Make Audio wakelock names less unique.

* commit '014e7fa2e90827d911c37bb0ce4d2e10e14d0bb3':
  Make Audio wakelock names less unique.

am b930056e: am b3cb72a1: SoundPool: handle new audio track event

* commit 'b930056ed4c6e1f8288d4f2cef828c3c02940482':
  SoundPool: handle new audio track event

am 1810a287: am 15ad2470: Camera: Fix deadlock in Camera3OutputStream

* commit '1810a287ecc81c243983051f9b1e32ff92ef1365':
  Camera: Fix deadlock in Camera3OutputStream

Make Audio wakelock names less unique.

These wakelocks now show up in the client process, where they
might cause wakelock overflows if AudioFlinger threads die often.

The client process should be agnostic of audio flinger threading,
so don't make the wakelock names rely on audioflinger thread names.

Wakelock names now depend only on the audio flinger thread type, and
not the ID itself.

bug: 11200551

Change-Id: Ia1f71b569cb7502d617b77596abc245f461a6ec0

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

am b3cb72a1: SoundPool: handle new audio track event

* commit 'b3cb72a17d9a472883e9e2faa18b42eac533fe99':
  SoundPool: handle new audio track event

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

SoundPool: handle new audio track event

If the AudioTrack is torn down, SoundPool will never
receive the buffer end event and the track will stay active
for ever.

The fix consists in stopping the AudioTrack when a new audiotrack
event is received.

Bug: 11193583.
Change-Id: I9876eb2a8f75c601368f669acd67b0accf6e2736

am 15ad2470: Camera: Fix deadlock in Camera3OutputStream

* commit '15ad2470b2f2ac34473eb568b606ad75e8e63ac6':
  Camera: Fix deadlock in Camera3OutputStream

Camera: Fix deadlock in Camera3OutputStream

process capture request thread, process capture result and setParameter
binder threads can run into circular locking situation when acquiring
StreamingProcessor lock, Camera3Stream lock, and bufferQueue lock.
Releasing the Camera3Stream lock briefly in process capture request
thread getbuffer call can break this deadlock.

Bug: 11016037
Change-Id: If08d4b134c26be26039b1d5363759e60f911bad6

am 92d27553: am d5ed2263: Merge "Camera: Skip AE precapture when possible" into klp-dev

* commit '92d275536ce3a7f1055627ac087b0f3a9957ad68':
  Camera: Skip AE precapture when possible

am c989d2d4: am 1dd08b30: Merge "Camera2/3: Add more tracing for API1" into klp-dev

* commit 'c989d2d4f6f7e63b1572673b390a5e89fa5c5702':
  Camera2/3: Add more tracing for API1

am d5ed2263: Merge "Camera: Skip AE precapture when possible" into klp-dev

* commit 'd5ed2263a485d6e633fe08f033d04fad75daec6f':
  Camera: Skip AE precapture when possible

am 1dd08b30: Merge "Camera2/3: Add more tracing for API1" into klp-dev

* commit '1dd08b30e2366cabc50d883885d6229dab8218f2':
  Camera2/3: Add more tracing for API1

Camera: Skip AE precapture when possible

Skip AE precapture when AE is already converged for still capture use case. This
could save still capture latency 6-7 frame time for AE already converged case.

Bug: 10712902
Change-Id: Ie5512618b76e5d87c62c57c3d96d2004c604e29e

Merge "Camera: Skip AE precapture when possible" into klp-dev

Merge "Camera2/3: Add more tracing for API1" into klp-dev

Camera: Skip AE precapture when possible

Skip AE precapture when AE is already converged for still capture use case. This
could save still capture latency 6-7 frame time for AE already converged case.

Bug: 10712902
Change-Id: Ie5512618b76e5d87c62c57c3d96d2004c604e29e

Camera2/3: Add more tracing for API1

- AF state
- Async autofocus completion
- Async takePicture completion
- Stages of takePicture

Bug: 10570497
Change-Id: Ida9fedd81aa4ee3ae3fb8dfada858a3bc3c213a3

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

am b9c0bd8a: am cc27e117: Camera: Save 1 frame time for takePicture call

* commit 'b9c0bd8ac65e8984e63a7e170b198e638fd3467f':
  Camera: Save 1 frame time for takePicture call

am cc27e117: Camera: Save 1 frame time for takePicture call

* commit 'cc27e117ed01c9a2b4def5a9c7a3103af83ee47e':
  Camera: Save 1 frame time for takePicture call

Camera: Save 1 frame time for takePicture call

Move the latest request signal call right before submitting request to HAL,
which avoids unnecessary 1 frame time wait.

Also fix request status constructor issue, and the log tag for Camera2Client.

Bug: 10570497
Change-Id: I0b1818658bee9983b117e49a5a00f1aab515c567

am a48ea70f: am b6028c42: merge in KFS78N (no-op)

* commit 'a48ea70f10bc05ef508f7125bd2fb7f940a48a75':

am c55d9e72: am 828363da: Merge "camera2: Don\'t race while dumping last frame metadata" into klp-dev

* commit 'c55d9e72f8d12e42a1f036badacd245fa99680e6':
  camera2: Don't race while dumping last frame metadata

am 8a6c3ba4: am a676e96c: merge in KQS81M

* commit '8a6c3ba4c0fff03b700cf6b1d1386b3c9fc2f3f6':
  Camera2/3: For still captures, use widest FPS range available.
  camera2: Tell all streams to ignore global device UI rotation
  Workaround slow AudioTrack destruction
  Partial fix for SoundPool not terminating
  Fix MediaDrm cts tests

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

am b6028c42: merge in KFS78N (no-op)

* commit 'b6028c4243fb7e4e16e41d893c1d48679d28ca6c':

am 828363da: Merge "camera2: Don\'t race while dumping last frame metadata" into klp-dev

* commit '828363dada5878fca6e0cb1553fbea96f38671c8':
  camera2: Don't race while dumping last frame metadata

merge in KFS78N (no-op)

Merge "camera2: Don't race while dumping last frame metadata" into klp-dev

am a676e96c: merge in KQS81M

* commit 'a676e96c6ed71846bdf60448bd63100ee2b357fb':
  Camera2/3: For still captures, use widest FPS range available.
  camera2: Tell all streams to ignore global device UI rotation
  Workaround slow AudioTrack destruction
  Partial fix for SoundPool not terminating
  Fix MediaDrm cts tests

merge in KQS81M

am b488b251: am a5209c2e: Merge "Camera3: Improve resilience to stream failures" into klp-dev

* commit 'b488b25148b8e3814a6c4854e6e8cbea6ea33735':
  Camera3: Improve resilience to stream failures

am a5209c2e: Merge "Camera3: Improve resilience to stream failures" into klp-dev

* commit 'a5209c2e6a89ecd8d3bec79cb0e82a677301d794':
  Camera3: Improve resilience to stream failures