Merge "Fix tiny memory leak"

Merge changes from topic 'vendor_available_libeffects'

* changes:
  Change libeffect from vendor:true to vendor_available:true
  Add libeffects_headers.
  libeffects: move to vendor

Merge changes Iabcb5469,Id8302f88,Iadfa594e,If0b10ed9

* changes:
  Add libhardware_headers to various modules
  libdrmclearkeyplugin is linked against libstagefright_foundation_vendor
  provide libstagefright_foundation_vendor to vendors
  Move libstagefright headers to media/libstagefright/include

Change libeffect from vendor:true to vendor_available:true

libaudioflinger is still using the lib in the aosp/master branch.

Bug: 33241851
Test: BOARD_VNDK_VERSION=current m -j
Merged-In: I11b54a5f050f914afc234e15f680581d15064c82
Change-Id: I9808ea87ea037a3caef03b2beabd1a717f153a6d

Add libeffects_headers.

Also had to fix the location and symlink of them.

Test: works with BOARD_VNDK_VERSION := current
Bug: 33241851
Merged-In: Idcfe1eaaf7be10a183b69bafa07f035ab9ca1e4e
Change-Id: Idcfe1eaaf7be10a183b69bafa07f035ab9ca1e4e
(cherry picked from commit a3b4ef8b5f99e3161684bb56b1e77f94903e194a)

libeffects: move to vendor

Bug: 33241851
Test: equalizer works in google play music
Test: CleanSpec works
Merged-Id: I11b54a5f050f914afc234e15f680581d15064c82
Change-Id: I11b54a5f050f914afc234e15f680581d15064c82
(cherry picked from commit c2e0f33032b223013645d8d0838abd9b00197318)

Add libhardware_headers to various modules

Bug: 33241851
Test: build target and pass CTS
Test: With BOARD_VNDK_VERSION, mma works
Merged-In: Iabcb5469c4fd936c2d7c1f9a1113c557c8beab88
Change-Id: Iabcb5469c4fd936c2d7c1f9a1113c557c8beab88
(cherry picked from commit da6a063a9ec520373443cdd4796eb643fcbffe34)

libdrmclearkeyplugin is linked against libstagefright_foundation_vendor

Since libdrmclearkeyplugin is in vendor partition, it is linked against
the vendor-variant of the libstagefright_foundation.

Bug: 37343126
Test: sailfish builds and boots
Test: BOARD_VNDK_CURRENT=current m libdrmclearkeyplugin is successful

Merged-In: Id8302f88978fc91282c54e0a679bcd8978898f8d
Change-Id: Id8302f88978fc91282c54e0a679bcd8978898f8d
(cherry picked from commit 24aa2adf677c71cc8f3c519ce83f4cf3363e640f)

provide libstagefright_foundation_vendor to vendors

libstagefright_foundation_vendor is a vendor-only variant of the
libstagefright_foundation. Vendor libs and other VNDK libs should link
against this _vendor.so library. This vendor-only variant does not have
dependency to libpowermanager which is not available to vendors.

Headers are moved from the global include dir to module-local include
dir. This is because global include dir (e.g. frameworks/av/include) is
not provided when building with BOARD_VNDK_VERSION=current. Instead the
headers are exported via the module's export_include_dirs directive.

Short summary on the directory movement:

"some" libstagefright headers are moved from libstagefright/include to
libstagefright/include/media/stagefright. This is because the headers
are included as <media/stagefright/HEADER_NAME,h>.

libstagefrigut_foundation headers are moved from
libstagefright/include/foundation to
libstagefright/foundation/include/media/stagefright/foundation. Again
this is because the headers are included as
<media/stagefright/foundation/HEADER_NAME.h>.

Furthermore, in order to prevent sudden breakage, symlinks to the
module-local header directories are added to the global include
directory.

frameworks/av/include/media/stagefright ->
<libstagefright>/include/media/stagefright

frameworks/av/include/media/stagefright/foundation ->
<libstagefright_foundation>/include/media/stagefright/foundation

Bug: 37462569
Test: sailfish builds and boots
Test: BOARD_VNDK_VERSION=current m libstagefright_foundation_vendor is
successful

Merged-Id: Iadfa594e885e0bd39837ccb282993d3ba5b62852
Change-Id: Iadfa594e885e0bd39837ccb282993d3ba5b62852
(cherry picked from commit 093f0b7ab981049e7a6e6bbcf1e0712e1995ba68)

Move libstagefright headers to media/libstagefright/include

This is a backport of the libstagefright portion of the change to move
headers from include/ into the appropriate directories.

include/media/stagefright -> media/libstagefright/include

Adds a symlink from the old location to the new one.

Bug: 33241851
Test: mmma -j frameworks/av/media/libstagefright

Merged-In: I0ecf754a2132640ae781a3cc31428fb8c0bd1669
Change-Id: If0b10ed98e06a068985cf8d84a089e539a7cde5f

Merge "fix potential mediaserver crash in ACodec::getPortFormat"

Merge "libmedia: Improve charset detection."

libmedia: Improve charset detection.

Tags are first put together before being detected. When different tags
are encoded in different charset, the result could be very bad. So when
the best result is not good enough, we should not accept it. We give it
another chance by detecting tags separately.

For the test samples we collected, 6 songs out of 13 are now displayed
correctly.

Test: transfer mp3 to devices and see if the tags' charsets are detected
correctly.
Signed-off-by: caozhiyuan <cao.zhiyuan@zte.com.cn>
Change-Id: I90d11a612b9f9e8896696df85635c4a46d067f09

fix potential mediaserver crash in ACodec::getPortFormat

https://issuetracker.google.com/issues/62428782

if rect.nLeft or rect.nTop is less then 0, in the error log it
will still check the integer overflow which probably cause abort

Test: NA

Change-Id: Ibe9a6edd0a57267b8dc7a29cdbdb2645e44c9af7
Signed-off-by: panshuangquan <panshuangquan@xiaomi.com>

Merge "Add OWNERS in frameworks/av"

Fix tiny memory leak

Bug: 36103283
Change-Id: Ib33905fc64b997da80b4b26346db7c531ff8bb8e

Add OWNERS in frameworks/av

* Owners are selected from top CL approvals or owners.
  They will be suggested to review/approve future CLs.
* OWNERS files are recognized by the new find-owners plugin, see .md files in
  https://gerrit.googlesource.com/plugins/find-owners/+/master/src/main/resources/Documentation/

Test: build/make/tools/checkowners.py -c -v OWNERS
Change-Id: I7c848855a2d7a0d7f33123ea4ef5c2d03977b495

Merge "Workaround for Clang emitting spurious Wunused-lambda-capture warning"

Workaround for Clang emitting spurious Wunused-lambda-capture warning

Bug: 38349491
Test: Build
Change-Id: I642108d456a1a9bd548cdeaf1ad46b98fc447c8a

Merge "Android.mk: remove $TOP in LOCAL_C_INCLUDES"

Android.mk: remove $TOP in LOCAL_C_INCLUDES

Test: make systemimage

Change-Id: I5f83786afe3f2c65819b127b041cb74fc4c989ac
Signed-off-by: YOUNG HO CHA <ganadist@gmail.com>

Merge "libeffects: move all libraries to vendor"

libeffects: move all libraries to vendor

Bug: 37490685
Test: equalizer works in google play music
Test: CleanSpec works
Change-Id: I9c1d19b5b5c5099edc9c20ff7728adc830ff4786

Merge "Convert libmediandk to Android.bp"

Convert libmediandk to Android.bp

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

Test: m -j checkbuild
Merged-In: I1d0aa1ba2da52a430d93b4c64bebf900f5af3a78
Change-Id: I1d0aa1ba2da52a430d93b4c64bebf900f5af3a78

Merge "Convert libstagefright to Android.bp"

Convert libstagefright to Android.bp

See build/soong/README.md for more information

Test: builds
Change-Id: I74cfc651d07cbbaddeae7c4da9da6f2527aee691
Merged-In: I74cfc651d07cbbaddeae7c4da9da6f2527aee691
(cherry picked from commit 6f82b4d3e970469de830d9fffcfcbea800e4637e)

Merge "Replace LOCAL_C_INCLUDES with header lib dependency."

Merge changes I937ac6ab,Ib94f78f7

* changes:
  Fix compiler warnings.
  Fix warnings in bitrotted module

Replace LOCAL_C_INCLUDES with header lib dependency.

Test: works with BOARD_VNDK_VERSION=current
Bug: 33241851
Change-Id: I8d3c47fe3d4b7dfe69d04e1f9ea1263cde067174

Merge "Convert libmtp to soong."

Merge "Use %zu to print size_t"

Merge "Move libmediandk and libcamera2ndk headers"

Fix compiler warnings.

Test: mmm

Change-Id: I937ac6ab2da5530ee249e6e87a90d18e9bd3de6e
Merged-In: I937ac6ab2da5530ee249e6e87a90d18e9bd3de6e
(cherry picked from commit 87eef41a5b81391b10a0f4ad874f6f04e261ed84)

Use %zu to print size_t

Test: m -j checkbuild
Change-Id: I93a1da7241169e0e08d310ad7fad68f227629be6
Merged-In: I93a1da7241169e0e08d310ad7fad68f227629be6
(cherry picked from commit cb8f367327286714617c0ccd7b356abd3a8aa15b)

Fix warnings in bitrotted module

Test: builds
Change-Id: Ib94f78f7baffbc63cff708ca5d81233618f30015

Move libmediandk and libcamera2ndk headers

Move libmediandk and libcamera2ndk headers into their module
and export them from there.  Also move them to the same path
relative to the export as used in the NDK, media/ for
libmediandk and camera/ for libcamera2ndk.

Test: builds
Change-Id: I0b424e0e48bfb5e2b762694a97d799de08f5dda3
Merged-In: I0b424e0e48bfb5e2b762694a97d799de08f5dda3
(cherry picked from commit 7e8d4ba5887affda749978d787b898bf31fda729)

Convert libmtp to soong.

Bug: 37512442
Test: native test
Change-Id: Idf0e96a206faadc5a3179f0df564ab1d4daacda2

Merge "Convert libdrmframework and drmserver to Android.bp"

Merge "Convert libmedia to Android.bp"

Merge "libeffects: move all libraries to vendor"

Convert libdrmframework and drmserver to Android.bp

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

Also moves a few headers into the same library as their
implementation, and fixes a few includes to go through the
global frameworks/av/include path instead of manually adding
frameworks/av/include/drm.

Test: m -j checkbuild
Change-Id: I0c2d2f2262e7ed78d80a0d44795705c64b797ff0

Convert libmedia to Android.bp

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

Test: m -j checkbuild
Change-Id: I7326405e50dd847ba9528aee5e5cd79f204c44af
(cherry picked from commit eeb436397a78aa80e41a4929a5f379319eefe0c3)

Merge "Mark functions in headers with static inline"

Merge "Remove reference to libc_logging."

libeffects: move all libraries to vendor

Bug: 37490685
Test: equalizer works in google play music
Test: CleanSpec works
Merged-In: I323f4048bce8a512d8aaa1760aef9d176a282c2d
Change-Id: I323f4048bce8a512d8aaa1760aef9d176a282c2d
(cherry picked from commit df57a30938ee4cc9b1c32f81ec54445466cbb858)

Merge changes I155f9f77,Ibb1aa07a,If460a6ef,Ifabff1c2

* changes:
  Fix trivial warnings hidden by -isystem
  Fix non-trivial warnings hidden by -isystem
  Rename SurfaceMediaSource::dump to dumpState
  HACK: work around both android-base/logging.h and ADebug.h defining CHECK

Mark functions in headers with static inline

Functions in headers that are not static inline will cause duplicate
symbol errors when linking if the header is ever included twice
into different compilation units of the same module.  It also causes
-Wunused-function warnings if they are not used in the cpp file, which
were hidden by the use of -isystem frameworks/av/include.

Bug: 31751828
Test: m -j checkbuild
Change-Id: I36f258bb4f5c3df54e874280beab344e63c519d5
Merged-In: I36f258bb4f5c3df54e874280beab344e63c519d5
(cherry picked from commit 413adafb04005fc5cdfdcbb3fe1ee4cfee9765b1)

Merge "libcpustats: Android.mk -> Android.bp"

Fix trivial warnings hidden by -isystem

Fix the trivial warnings that are hidden by the use of -isystem to
include frameworks/av and caught by -Werror.

Test: m -j checkbuild
Bug: 31751828
Change-Id: I155f9f772ce0a84b364313814cb7cf528b9de4eb
Merged-In: I155f9f772ce0a84b364313814cb7cf528b9de4eb
(cherry picked from commit b8c35f94470d1518e2def0582aaec4e038c92af0)

Fix non-trivial warnings hidden by -isystem

These warnings were hidden by -isystem.

Bug: 31751828
Test: m -j checkbuild
Change-Id: Ibb1aa07a6827fcbd29fc15291fcd90280c0f608d
Merged-In: Ibb1aa07a6827fcbd29fc15291fcd90280c0f608d
(cherry picked from commit 6f51c153ea5e4187a313e47ac8bad9a1a999d4e1)

Rename SurfaceMediaSource::dump to dumpState

A method named dump() is already exported by Binder.  The warnings
were hidden by the use of -isystem to include frameworks/av/include.

Bug: 31751828
Test: m -j checkbuild
Change-Id: If460a6efc7c7b108243c3ef068c28619482f599d
Merged-In: If460a6efc7c7b108243c3ef068c28619482f599d
(cherry picked from commit f461f38e08ca8afb89e1dc33beae43905f47f9f8)

HACK: work around both android-base/logging.h and ADebug.h defining CHECK

One version of the CHECK_* macros are defined by
android-base/logging.h, and another by ADebug.h.  Eventually
the ADebug.h ones should be replaced with the libbase ones, but
that ripples out into a long dependency chain that needs to link
against libbase.  The redefined macro warnings were hidden by the
use of -isystem to include frameworks/av/include.  Hack around it
for now by undefining the libbase ones if they exist.

Bug: 31751828
Bug: 37782039
Test: m -j checkbuild
Change-Id: Ifabff1c2bc6bac198804c951e98f3fee282cd018
Merged-In: Ifabff1c2bc6bac198804c951e98f3fee282cd018
(cherry picked from commit 1b1686db53d3e8c9f4c1b8c7bb360bb6437337ab)

libcpustats: Android.mk -> Android.bp

Test: links
Change-Id: I4b7394299ad0f879ea8cc34ba95a8c6041513d53

Remove reference to libc_logging.

The libc_malloc_debug_backtrace includes the log functions so there
is no need to also add the libc log library.

Test: Built and booted a bullhead device.
Change-Id: I00457ad0c321b385e68232ee87575588a8188db7

Merge "Convert libmediadrm to Android.bp"

Merge "Fix warnings hidden by -isystem"

Convert libmediadrm to Android.bp

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

Test: m -j checkbuild
Change-Id: Icfc13bb5f55fa156ef72d3cf3a327e38739d775b
Merged-In: Icfc13bb5f55fa156ef72d3cf3a327e38739d775b
(cherry picked from commit 53f2035f1d78499df816e2ef6ddb90421b0bfd72)

Fix warnings hidden by -isystem

Fix -Wundefined-var-template and -Wsign-compare warnings that were
hidden by the use of -isystem to include frameworks/av.

Test: m -j native
Bug: 31751828
Change-Id: I28e07b238d5bc913ea2f3dfa09fa02f577d5a250
Merged-In: I28e07b238d5bc913ea2f3dfa09fa02f577d5a250
(cherry picked from commit d995c0186ee0fc13f397a35808a1bc8045ab48d0)

Merge "Remove unused #include."

Remove unused #include.

Bug: N/A
Test: builds
Change-Id: Ia493566414e7d08ae12bb9ba754d5c207eed8d3e

Merge changes Ib249b276,Ica1d0286,If4914340

* changes:
  LOCAL_CLANG := true is now the default
  stagefright: move several static methods out of ACodec
  stagefright: move MediaDefs from libstagefright to libmedia

Merge "MockDrmCryptoPlugin: Don't truncate log output"

LOCAL_CLANG := true is now the default

Test: verify that build has no failures
Change-Id: Ib249b276c6295e55eb54fa37b6d15486415dfbed
Merged-In: Ib249b276c6295e55eb54fa37b6d15486415dfbed
(cherry picked from commit 5f5ee1110dd85001b43db7637a97474acfe8d681)

stagefright: move several static methods out of ACodec

Bug: 24761407
Change-Id: Ica1d0286b1e800f5d4d0f120c043c0a556dab4c2
Merged-In: Ica1d0286b1e800f5d4d0f120c043c0a556dab4c2
(cherry picked from commit 1ad5b240bfd49aa918092dbdc18c1a24c5e6c3a4)

stagefright: move MediaDefs from libstagefright to libmedia

Bug: 24761407
Change-Id: If4914340a645c6f3c14eea0c873c162926d72ca2
(cherry picked from commit ce4df5e534eb86748fe80c84efb6b7fc0b6b5b46)

MockDrmCryptoPlugin: Don't truncate log output

A function argument which is an array with a fixed size is not
actually a fixed size array, but rather just a pointer.  Thus,
when we were using sizeof() for 'key' and 'iv', we were mostly
likely only logging the first 4 or 8 bytes of the array, and
not all 16.

We change this code so we're at least passing the correct size
to the arrayToString method, and thus logging all the bytes.

This API is still fragile, as there is nothing which assures
callers to decrypt are passing in arrays which are exactly 16
bytes.  However, this API is equivalent to before, so we aren't
introducing any compatibility issues.

The compiler gave a warning about this.

Test: Compiled and confirmed there's no longer this compiler warning.
Change-Id: Ie3a32eef0882ea95bccc3e22ffcab7336779bd4c

Merge "mtp: Preserve last modified time of files."

mtp: Preserve last modified time of files.

Bug: 6325175
Test: Transfer files
Change-Id: I54af3205575f2bb83c732bb70907f60e0b785e91
Signed-off-by: caozhiyuan <cao.zhiyuan@zte.com.cn>

Merge "Allow fstatfs64 syscall for mediacodec/extractor"

Allow fstatfs64 syscall for mediacodec/extractor

This syscall is used by the linker to detect files located on tmpfs

Bug: http://b/37245203
Bug: http://b/37590327
Test: boot an angler and check that mediacodec does not crash
Change-Id: I7147063db7d6b8ec39a25d39b60e8e6882addf6e
(cherry picked from commit 1516b02c1a4ab5739bffea9d75ad9b52fc1d5b91)

Merge changes I0dc38288,I48b713cd

* changes:
  libmediautils: mk -> bp
  Clean up mediadrm dependencies

libmediautils: mk -> bp

Test: module built and works with scheduleservice
Bug: 29251823
Change-Id: I0dc38288b66f03919dbb9ab8abcc76dec19b0b30
(cherry picked from commit 22ee46008d6fb8eb4cbe8aa62acf3899e5da3a92)

Clean up mediadrm dependencies

Move various things around so mediadrmserver doesn't need
libstagefright and libmedia

Test: build/boot
Change-Id: I48b713cd679b992cb6ad1262c20394a9892d7027
Merged-In: I48b713cd679b992cb6ad1262c20394a9892d7027
(cherry picked from commit 75571e4bc76bd2defd4309c7154120840396329d)

Merge "Convert libcamera_client to Android.bp"

Merge "DO NOT MERGE: Move camera headers to camera/include"

Convert libcamera_client to Android.bp

See build/soong/README.md for more information

Test: m -j checkbuild
Change-Id: Ie3a82e929ea1e2c271e362bf0fa85f79fa9e9004

DO NOT MERGE: Move camera headers to camera/include

In preparation for moving libcamera_client to Soong, this is a backport
of the camera portion of the change to move headers from include/ into
the appropriate directories.

include/camera -> camera/include/camera

Adds a symlink from the old location to the new one.

Bug: 33241851
Test: mmma -j frameworks/av/camera
Change-Id: Ic508c2d0100f4e84b63e745d50eca1402b6fed6c
(cherry picked from commit d008fd1e4f4cadd300355ece1547ec56d80227dc)

Merge changes If9daee8f,I09ab3bfc,I854760bd

* changes:
  Convert libradio to Android.bp
  Convert libsoundtrigger to Android.bp
  Convert libimg_utils to Android.bp

Convert libradio to Android.bp

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

Test: m -j checkbuild
Change-Id: If9daee8fea744a5d70cf0ea6efb2a806b6cffa36

Convert libsoundtrigger to Android.bp

See build/soong/README.md

Test: m -j checkbuild
Change-Id: I09ab3bfc8649bda8bad66c0898e342865f8b4a92

Convert libimg_utils to Android.bp

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

Test: m -j checkbuild
Change-Id: I854760bd1f7ab054feb2496d2d18eee35ad8b9f2

Merge "Fix clang static analyzer warnings."

Fix clang static analyzer warnings.

Fix the following warning:
frameworks/av/media/mtp/MtpServer.cpp:1098:5: warning: Assigned value is
garbage or undefined [clang-analyzer-core.uninitialized.Assign]

BUG:None
TEST:Warnings fixed.
Change-Id: If4e54ebd9b865e2e6cdc7666da0c9987a918df7c

Merge "Remove useless code in aacenc"

Remove useless code in aacenc

Remove useless code in aacenc for better understanding

Test: mm -j8
Change-Id: I6f5a1dcfea5dd3642d1c607be68103039cbb8c53

Merge "Remove .(ll)ndk suffix from (ll)ndk_library"

Remove .(ll)ndk suffix from (ll)ndk_library

Soong handles these automatically now.

Bug: 33241851
Test: Android-aosp_arm.mk is the same before/after
Test: build.ninja is the same before/after
Test: build-aosp_arm.ninja is the same before/after
Merged-In: I792e4647b38988db5d9f591d69688479ce9dc62b
Change-Id: I792e4647b38988db5d9f591d69688479ce9dc62b

Merge "Do not seek when switching to background playback"
am: 3db1bb97a7

Change-Id: I2c894349941e1a9bcfe329f4c40b0caea85987de

Merge "Do not seek when switching to background playback"

Merge "Remove old software avc encoder"
am: fb20749bfa

Change-Id: Iaedc1950b61aca0cb87e808606f6e743419b371d

Merge "Remove old software avc encoder"

Remove old software avc encoder

It hasn't been used since Lollipop

Test: build

Change-Id: I7652973b50426b3bcf8c10dcc77deb97366b003c

Merge "audio policy: add rule for ringtone duplication over speaker + SCO"
am: dc671c4e33

Change-Id: Ic19309906b00121c4cd02ff8ed620416d5b9678f

Merge "audio policy: add rule for ringtone duplication over speaker + SCO"

audio policy: add rule for ringtone duplication over speaker + SCO

Bug: 19171297
Test: make, testplans/79248

Change-Id: Ic539256e53782806d5dbbca5fb6ebe9f2ca5ec76

Merge "parameter passed to BnHDCP::onTransact() is incorrect when calling decrypt function."
am: 5052e9cf9f

Change-Id: I4eaf334bd11e1b91d68d40c0958f14364ee9b471

Merge "parameter passed to BnHDCP::onTransact() is incorrect when calling decrypt function."

parameter passed to BnHDCP::onTransact() is incorrect when calling decrypt function.

Change-Id: Iba8e74a0043877a167e4b290b2257fcd990076b8
Signed-off-by: Ming Zhou <b42586@freescale.com>
Reviewed-on: http://androidsource.ap.freescale.net/project/504
Reviewed-by: Jian Li <jian.li@nxp.com>

Merge "Add support for MTP_RESET_DEVICE operation."
am: 52afe9d2cf

Change-Id: I36c8e325e2b2dde85cb2a3546508c95e5da77e57