Add logging of written frame counts

Bug: 31591209
Change-Id: Ife5917aa65b02c3b0791dfe1975c70ccf58f9a75
(cherry picked from commit 2c45393d842a5d2968ef78fa7a08667efdf15f2a)

Merge "AudioFlinger: Account for frames dropped in suspend mode"

AudioFlinger: Account for frames dropped in suspend mode

These frames must be added to the kernel position in timestamps.

Bug: 30025777
Change-Id: I30c14503c9467059c3782e5c96c01ae1cf3709f6
(cherry picked from commit 238fa3deff26d7e4d9e81bd0a88c936f16226c4e)

Merge "Fix potental memory leak (compiler warning)."

Fix potental memory leak (compiler warning).

This CL fixes the following warning:

EffectProxy.cpp:93:8: warning: Potential leak of memory pointed to by
'pContext'.

BUG=None
Test: WITH_TIDY=1 WITH_STATIC_ANLYZER=1 mm

Change-Id: I7f127042eb50d3d6297473bfd90a5402b182cc85

Merge "WAVExtractor: support ambisonic subformat"

Merge "Fix issues with metadata handling when format change happens"

Merge "Fix dereference null pointer warning."

Fix dereference null pointer warning.

This fixes
warning: Access to field 'region' results in a dereference of a
null pointer (loaded from variable 'config')

Test: The warning is gone.
Change-Id: Iac531e879a6768d8638e77fc03dadb08481c486a

Merge "Rely on the platform -std default."

Rely on the platform -std default.

Bug: http://b/32019064
Test: builds
Change-Id: I535e2fb041cf11c3c4eeaf3367c54072e9208ebe

Fix issues with metadata handling when format change happens

When a video format changes, the discontinuity will clear the ATSParser
queue and format. This means that AnotherPacketSource::getFormat() will
return NULL until a new format is found. The discontinuity casues the
decoder to be shutdown and NuPlayer will start scanning sources.
But since ESQueue has not yet found any format, the call to the
StreamingSource::getFormat() will return an error (BAD_VALUE) since the
convertMetaDataToMessage() fails upon NULL input.
This error code causes the scanning of sources to stop which means that
the video will be stoped.

Instead returning -EWOULDBLOCK while waiting for a new format will keep
the scanning of source.

Bug: 32007921
Test: Manual - Install SVT Play app, start live streaming
Change-Id: Ia237825361755bc63ba34b7089e7804c187fafd8

Merge "Convert to Android.bp"

Convert to Android.bp

See build/soong/README.md for more information.

Test: mmma -j frameworks/av/media/libnbaio
Change-Id: Ifd5f13ad6ab1c9590623c3b58adea5ef3ea6af60
Merged-In: Ifd5f13ad6ab1c9590623c3b58adea5ef3ea6af60

Merge "Convert to Android.bp"

Convert to Android.bp

See build/soong/README.md for more information

Test: mmma -j frameworks/av
Change-Id: I1b8deeec65e2cfa1633354712004ef87f3ac8b3e

Merge "mediaextractor: Skip setting  memory limits on ASan builds"

mediaextractor: Skip setting  memory limits on ASan builds

ASan-ified libraries have a larger memory footprint which counts
towards a process's RLIMIT_AS. Detect ASan at runtime and skip
setting memory limits.

Test: Builds and boots. "MediaUtils: running with ASan, not setting
memory limits" appears in logcat, mediaextractor is no longer
crashing.

(cherry picked from commit 4bc7ad9e0ea04928e7ed9767251696cc4a4e5d15)

Bug: 31831213
Change-Id: I6705baa825cbfd45695def2cd80d8d2261b05339

Merge "Fix struct vs. class mismatch in forward definitions"

Merge "Rename dump methods to dumpState"

Merge "Prevent crash for malformed ID3 COMM frame"

Prevent crash for malformed ID3 COMM frame

When trying to retrieve the full text string from a malformed
ID3 COMM frame, there is a risk of getting a negative index
value if the size of the frame is too small.

Adding a size check to avoid negative indexes.

Bug: 31815576
Test: Playback content with malformed ID3 COMM frame
Change-Id: I21253e104cd76a65f5ab5d03f58490e0f4a4cb65

Fix struct vs. class mismatch in forward definitions

The warnings were being hidden by the use of -isystem to include
frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I0eea5b59e7bd696d9aff44c8680361c99474e634

Rename dump methods to dumpState

The dump methods shadow the IBinder dump method.  The warnings were being
hidden by the use of -isystem to include frameworks/native/include.

Bug: 31752268
Test: m -j
Change-Id: I78136a4baf71630455010e884bc198314cecca7a

Merge "Migrate libmediandk to ndk_library."

Merge "libeffects: fix enum conversion warnings."

libeffects: fix enum conversion warnings.

warning: implicit conversion from enumeration type 'LVDBE_Fs_en' to
different enumeration type 'LVM_Fs_en' [-Wenum-conversion]

Thanks to Luis Lozano for the patch!

BUG: None
Test: Compiles without warnings.
Change-Id: I5bb0b5fc2e4e23080f61a209ab0ea95f423bafb1

Merge "Add missing liblog dependency"

Add missing liblog dependency

Bug: http://b/31289077
Bug: http://b/27171986
Test: remove liblog from libcutils and recompile
Change-Id: Ic2e8a3d9e62d778d32863a4b794dc4c80b1b1b2b

Migrate libmediandk to ndk_library.

Test: make native
Bug: http://b/30465923
Change-Id: Id62eac2d5cc518465445d87ce204fa918f0ddb68

Merge "IMediaSource: Pack alignment of ReadOption struct"

IMediaSource: Pack alignment of ReadOption struct

int64_t aligns differently in x86 vs x64, which presents
problems when communicating across Binder.
Use packed struct to avoid problems.

Bug: 30472583
Change-Id: Iacdad792e19c19899cde5289238b5aa8981a9a7a

Merge "Migrate libcamera2ndk to ndk_library."

Merge "Fix build when #define LOG_NDEBUG 0"

Merge "stagefright: fix finding hardware codec"

Fix build when #define LOG_NDEBUG 0

A few variables have changed type and when building with LOG_NDEBUG 0
the code no longer compile. Adjust the ALOG_ASSERT strings to use the
correct types.

Test: make with LOG_NDEBUG enabled
Bug: 31584070
Change-Id: Ib47f369f8be9fa0c12fa4f69c294a719df8ad018

Merge "Remove incorrect forward declaration"

Remove incorrect forward declaration

ANativeWindow is a struct, not a class, and the incorrect forward
declaration causes a warning when -isystem is removed from
system/core/include.

Test: m -j native
Bug: 31492149
Change-Id: I3516aa98defe1944a9776c07f52f4ee580366988

Merge "Stop including libcore/include/ in frameworks/av."

Migrate libcamera2ndk to ndk_library.

Test: make checkbuild tests
Bug: http://b/30465923
Change-Id: Iab800fd20beeef64250abc7d1c966d12fa8bfbb5

Merge "media: Fix graphic buffer leak."

media: Fix graphic buffer leak.

Bug: 30882996
Test: build and boot
Change-Id: I673c06b225f55f0d9f15800edb8113abc237c4ce
Merged-In: I571a3a9d1705233029abb92f3752416d55ed1056

Stop including libcore/include/ in frameworks/av.

Change-Id: I7c0ffa84071c2080b5bfe2ce9ad7e1822fe863c7

Merge "Remove unnecessary pathmap usage"

Remove unnecessary pathmap usage

speex is being removed by the pathmap because libspeexresampler already
exports that include dir automatically if you link against it.

Test: mma
Change-Id: If3d07234696cd24ff0060d829f55d9d50a3df849

Merge "Fix google-explicit-constructor warnings in camera."

Merge "Fix google-explicit-constructor warnings in audioflinger, audiopolicy."

Merge "Fix google-explicit-constructor warnings in media."

Merge "Use symbolic name for return value in legacy audio policy client"

Merge "Convert to Android.bp"

Fix google-explicit-constructor warnings in media.

* Add explicit keyword to conversion constructors.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I141b22bcb3caf386400b6b0403d1b231a8e7e6c4

Fix google-explicit-constructor warnings in camera.

* Add explicit keyword to conversion constructors.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I61f9b5b2b6e13e0c6561327bc8dfe69c9cfc5ba5

Fix google-explicit-constructor warnings in audioflinger, audiopolicy.

* Add explicit keyword to conversion constructors.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I3b73f0cd73f47be7735cc374306c02301c20588f

WAVExtractor: support ambisonic subformat

Support ambisonic subformat

Change-Id: I4fb35cbac3dd8b97bdda0e4faf4cfc356e2de928

Convert to Android.bp

See build/soong/README.md for more information.

Change-Id: Ib5abad8bcb63bd3f3aa19a08817276fe21902515

Merge "Mark unused parameters as UNUSED in BRILLO."

Merge "Add bound checks to utf16_to_utf8"

Mark unused parameters as UNUSED in BRILLO.

Bug: 31068444
Test: `make dist` in Brillo.

Change-Id: Ib0229ad78894fc703e9c5ad7bda4ab2466641f77

Merge "Camera3: don't let dequeueBuffer block indefinitely"

Camera3: don't let dequeueBuffer block indefinitely

The synchronous consumers (e.g. ImageReader) may be very slow when the
clients have computational intensive image processings. When system
load is high, these processing will be even much slower. This could
starve the producer side and then cause dequeue/attach buffer block
indefinitely. If clients intends to close the capture session, right
after a capture request is submitted, the waitUntil drain could be
blocked indefinitely if the capture request dequeue buffer call is
blocked indefinitely, as the request thread will never become idle until
the last dequeue buffer is done and the request is sent.This indefinite
getBuffer() blocking could easily trigger the waitUntilDrained 5s timeout
thus put the camera device into error state although there is nothing
bad happening in the HAL.

Introducing the timeout will avoid such bad situation. When consumer is
slow, there will be no new request being sent to HAL, as there is really
no new buffer available.

Bug:30404840
Change-Id: Ibf08d2745911203bce6f31130800707f36d7f985

Merge "Silence warning about unused variables in BRILLO."

Add bound checks to utf16_to_utf8

Bug: 29250543
Change-Id: I3518416e89ed901021970958fb6005fd69129f7c
(cherry picked from commit 1d3f4278b2666d1a145af2f54782c993aa07d1d9)

Silence warning about unused variables in BRILLO.

This patch moves the call to validateClientPermissionsLocked() outside
the #if so it is executed in Brillo builds anyway, but the result is
ignored. This fixes the warning about unused parameters which was
causing the build to fail.

Bug: 31068444
Test: `make dist` in Brillo

Change-Id: I8a6648bf2221406068c9f5a1850a59ae0bde03ca

stagefright: fix finding hardware codec

When findMatchingCodecs(.., kHardwareCodecsOnly,..) was called,
it wouldn't actually match any hardware codecs because of a typo:

  if (!(TRUE_prefer_HW_codec && !FALSE_is_SW_codec))
    ==> if (!(TRUE && TRUE))

Fix logic and clarify statements

Change-Id: Ice725f043475caedd8f7ab961e0bc985db2eba6f

Merge "Fix for tone glitch produced in ToneGenerator"
am: 46f1c24c5a

Change-Id: If7da9dbfe106fdda640fc817a4fffe05b2ed5dff

Merge "Fix for tone glitch produced in ToneGenerator"

Merge "Fix google-explict-constructor warnings in mtp."
am: b480ab9365

Change-Id: I7285319a159b5d1d42145e6eb9f51849c53df9d3

Merge "Fix google-explict-constructor warnings in mtp."

Merge "Fix google-explict-constructor warnings in img_utils."
am: c36d401e83

Change-Id: I3cc4f149a715084f36a5e0fa4007e54ec6a7e84f

Merge "Fix google-explict-constructor warnings in img_utils."

resolve merge conflicts of 6ae8c7b to stage-aosp-master

Change-Id: Iabb5d9b93b98e428c53f0504fbfe832ed24d988d

Merge "Fix clang-tidy warnings in stagefright."

resolve merge conflicts of 70066d9 to stage-aosp-master

Change-Id: I9af1ace904388c91e6ce885e8a6d3540eb57bb4a

Merge "Fix clang-tidy warnings in camera."

Merge "Remove RefBase base class from FrameRenderTracker"
am: e4e2a8fc35

Change-Id: Ibf36079f9b9474988e7aed4d15964d89e2a0da49

Merge "Remove RefBase base class from FrameRenderTracker"

resolve merge conflicts of fd923e7 to stage-aosp-master

Change-Id: I75b44b89bae41197a1fd68362d20b8ba2b4dd192

Merge "Fix clang-tidy warnings in audio and playerservice."

Fix clang-tidy warnings in audio and playerservice.

* Add explicit keyword to conversion constructors.
Bug: 28341362
* Use const reference type for read-only parameters.
Bug: 30407689
Test: build with WITH_TIDY=1

Change-Id: I265f3b094e08d5705b506b3fbba51439c134af84
Merged-In: I265f3b094e08d5705b506b3fbba51439c134af84

Fix clang-tidy warnings in stagefright.

* Add explicit keyword to conversion constructors.
* Or add NOLINT to implicit conversion constructors.
Bug: 28341362
* Use const reference type for read-only parameters.
Bug: 30407689
* Use const reference type to avoid unnecessary copy.
Bug: 30413862
Test: build with WITH_TIDY=1

Change-Id: I7a15ac1e37973fe2c661f577a6106b913653de40
Merged-In: I7a15ac1e37973fe2c661f577a6106b913653de40

Fix clang-tidy warnings in camera.

* Add explicit keyword to conversion constructors.
Bug: 28341362
* Use const reference type for read-only parameters.
Bug: 30407689
* Use const reference type to avoid unnecessary copy.
Bug: 30413862
Test: build with WITH_TIDY=1

Change-Id: I71d3008da843ba5f1df1a73a320fb2af6ceffa16
Merged-In: I71d3008da843ba5f1df1a73a320fb2af6ceffa16

resolve merge conflicts of 1c10de5 to stage-aosp-master

Change-Id: I1e395ecb96524c18348b9f65c96d60c9eae9be45

Merge "Fix clang-tidy warnings in mediaresourcemanager."

Fix clang-tidy warnings in mediaresourcemanager.

* Add explicit keyword to conversion constructors.
Bug: 28341362
* Use const reference type for read-only parameters.
Bug: 30407689
Test: build with WITH_TIDY=1

Change-Id: I9aac64b379ddb77a9abb4b03a6648f38e2a756ab
Merged-In: I9aac64b379ddb77a9abb4b03a6648f38e2a756ab

Merge "Fix clang-tidy warnings in drm."
am: e055519839

Change-Id: I99159fb0391af796445ea24265d088d20d178077

Merge "Fix clang-tidy warnings in soundtrigger."
am: 3ba2cb78bf

Change-Id: Ib9481a6bb666c75b8c0c1db5fc8f480cc85a266a

Merge "Fix google-explict-constructor warnings in radio."
am: f53c1d182d

Change-Id: I5973845ffdd43d318ba1058c4c03432d78dbf733

Merge "Fix clang-tidy warnings in drm."

Merge "Fix clang-tidy warnings in soundtrigger."

Merge "Fix google-explict-constructor warnings in radio."

Fix clang-tidy warnings in drm.

* Add explicit keyword to conversion constructors.
Bug: 28341362
* Use const reference type for read-only parameters.
Bug: 30407689
* Use const reference type to avoid unnecessary copy.
Bug: 30413862
Test: build with WITH_TIDY=1

Change-Id: Idf1c48d7bfe3f0da20401cb3ff1983ec00be170d

Fix google-explict-constructor warnings in mtp.

* Add explicit keyword to conversion constructors.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I3a84ae7ffa03b2e23562edef95ef6abefe0ff456

Fix google-explict-constructor warnings in img_utils.

* Add explicit keyword to conversion constructors.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I490f8fcb666e0e1cb7923067857a747e7f529b3b

Fix google-explict-constructor warnings in radio.

* Add explicit keyword to conversion constructors.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I15359f74d6587d691a8cb35445732aff44188268

Fix clang-tidy warnings in soundtrigger.

* Add explicit keyword to conversion constructors.
Bug: 28341362
* Use const reference type for read-only parameters.
Bug: 30407689
Test: build with WITH_TIDY=1

Change-Id: I28b251c99c78286a2da1e039df470a3f0bd8abe2

Remove RefBase base class from FrameRenderTracker

FrameRenderTracker is never s standalone object, so it's unnecessary.
This avoids allocating a few extra heap objects. More importantly
it no longer interferes with my attempts to understand RefBase
objects that are deallocated without acquiring a strong reference.

Change-Id: I008b04b085fbab1efa4e8948efdcca14b26c247f

Fix writeColrBox to write 0x80 instead of 128

Use 0x80 and explicitly cast down to int8_t.

Bug: http://b/29831838
Change-Id: Ibf989863ab2affdc7218f94ca073f37f5192d488
(cherry picked from commit fffb0c8fbe16ac948619b85e1eea9e65a9daa0c9)

Merge \"webm: avoid unnecessary variable-length arrays\"
am: 67fbddfdc2

Change-Id: I7f738a3ab2fadf0794da4b5f47c2be259613bf4d

Merge "webm: avoid unnecessary variable-length arrays"

webm: avoid unnecessary variable-length arrays

Change-Id: Ia72df2c208f2b0ebc9c55099dfaabdca97914791