omx node: patch err code to preserve API contract

Bug: 36789323
Change-Id: Ie1d8dde5e4df8f5f4061cba7b20e1732b1466235
Test: adb shell am instrument -e size small \
  -w android.media.cts/android.support.test.runner.AndroidJUnitRunner

Merge "Show in dumpsys effect libraries that failed to load" into oc-dev

Show in dumpsys effect libraries that failed to load

Bug: 36525163
Bug: 36864417
Change-Id: I120129cb43e9593cc94ba361fe76ade86448e46a

Test: run and use dumpsys audio_flinger

Merge "Remove useless GraphicBuffer() constructor" into oc-dev

Merge "NuPlayer : send the correct streaming info while opening audio sink" into oc-dev

Merge "MPEG4Source: fix fragmented read." into oc-dev

Merge "Revert "Camera: Map between bufferqueue gralloc0 usage, and HIDL gralloc1 usages"" into oc-dev

MPEG4Source: fix fragmented read.

Test: passed CTS test DecoderTest#testDecodeFragmented
Bug: 36571704
Change-Id: I71ad6aaae473b03483f8405899d3178148597bba
(cherry picked from commit ba9af7792dfed6e9b1b216aab91a97e713eec891)

Revert "Camera: Map between bufferqueue gralloc0 usage, and HIDL gralloc1 usages"

This reverts commit 1a2f15daf13552e86324025cf6d31d887cd8fa02.

Test: fix b/36876770
Bug: b/36876770
Change-Id: Ie7b8c9ee444e3e8667154b396e3e6a556b54739a

Merge "Add timeout to lock in MtpServer configure" into oc-dev

NuPlayer : send the correct streaming info while opening audio sink

 -For any offload playback NuPlayerRenderer always open the audio sink
  with isStreaming info as true.
 -Pass the streaming info to the NuPlayerRenderer while opening audio
  sink
Test: cts tests
Bug:36051644
Author: Preetam Singh Ranawat <apranawat@codeaurora.org>
Change-Id: I249e6769ef4587917a13b0225d049a3923544d16
(cherry picked from commit b2444b392d0d5de4118d37a24f4351f9de5c215d)

Merge "NuPlayer: Use deep buffer AudioTrack for video playback" into oc-dev

Merge "Camera: Try to remove inflight requests during error notifications" into oc-dev

Remove useless GraphicBuffer() constructor

Test: compiled & run
Bug: 36869708
Change-Id: I777701b8f47929d2b669d2a990649e58eef528fc

Merge "Camera: add buffer freed notification for output streams" into oc-dev

Merge "Camera: add buffer freed notification interface" into oc-dev

NuPlayer: Use deep buffer AudioTrack for video playback

This reduces power requirements and increases robustness against
audio glitches.

Test: Photos
Bug: 36867565
Change-Id: I200728cfc1bfd01301441c8759816415abe58d48

Camera: add buffer freed notification for output streams

Test: ReprocessCaptureTest fixed
Bug: 34461678
Change-Id: Icd5af79a8f7fc35b0a040c57a639ae6777e71024

Merge "Camera: Map between bufferqueue gralloc0 usage, and HIDL gralloc1 usages" into oc-dev

Camera: add buffer freed notification interface

To cleanup caches of obsolete buffers.
This CL addressed the input stream bit, the output
stream hook will be a followup CL.
Also cleanup some dead API in CameraDeviceBase.h

Test: fix CTS ReprocessCaptureTest
Bug: 34461678
Change-Id: I801cd81c29becaa45630ed0a5c2dab8df1278a6a

Add timeout to lock in MtpServer configure

In situations where MtpServer is in a bad state, this will prevent
the configure call from blocking indefinitely, which will prevent
a few visible problems and allow the state to recover on the next
config change.

Bug: 34873000
Test: Connect to AA test sled
Change-Id: I5daf9cfcec69c967bde6bf3f6fd90e1c434e18bb

Use gralloc0 usage to init BufferItemConsumer

BufferItemConsumer is expecting gralloc0 usage, use
android_convertGralloc1To0Usage.

Also one minor log change: during acquireImageLocked, turn off the
warning when accquired buffer has different dimension than originally
configured on ImageReader. It is actually a legal operation for
IGraphicBufferProducer to change buffers dimension dynamically.

Bug: 35114769
Test: AImageReaderCts now passes with CPU_READ_OFTEN case.
Change-Id: I32015148437be56242a46c8e20ca33497e182ac2

Merge "libaaudio: implement getTimestamp for legacy path" into oc-dev

Camera: Try to remove inflight requests during error notifications

Make it possible for clients to clear the internal inflight queue
as part of error result notifications. For this to work all pending
buffers must be returned previously with the exception of the
result metadata.

Bug: 35652756
Test: Complete Camera/Camera2 CTS without any regressions.
Change-Id: I9d8a6b63364fd254311cda2f8c836b99ee05cacb

Merge "Camera: HAL1 recording batching support" into oc-dev

Merge "MediaRecorder: Handle 24-hour timelapse duration" into oc-dev

Merge "Add Flag for Enabling 64-bit Media DRM Server" into oc-dev

NuPlayer: handle source error correctly.

Test: source error is sent to app.
Bug: 35025627
Change-Id: I668bab0de20cf24def028a6a467efceaed54af2b
(cherry picked from commit 231fd353518502e3a05021473ae60e7addd9a46c)

Camera: HAL1 recording batching support

Test: Angler HAL1 + batching normal recording mode
Bug: 35997432
Change-Id: If088715ccdb40163f44150c16deed62432692c07

Merge "Nuplayer logging empty records to media.metrics" into oc-dev

MediaRecorder: Handle 24-hour timelapse duration

Handle FPS in double / 64-bit integer and allow room for error.

Bug: 34870132
Test: cts-tradefed run cts-dev --module CtsMediaTestCases --test android.media.cts.MediaRecorderTest
Change-Id: I371d22d9ae70e2d77b08bd6c501e8b34a71592d1

Merge "Clearkey plugin didn't handle negative test cases" into oc-dev

Merge "Remove dependency on persist.hal.binderization." into oc-dev

Merge "media: Handle STOP action properly." into oc-dev

Merge "libstagefright: Add error handling in mediacodecsource" into oc-dev

libaaudio: implement getTimestamp for legacy path

Fix: 36659408
Test: CTS test_aaudio.cpp
Signed-off-by: Phil Burk <philburk@google.com>
Change-Id: Ic73e69a3747b2daa1df5757650f897a58511e571

Clearkey plugin didn't handle negative test cases

Part of VTS tests for drm

Change-Id: Ia89aa51ec9018e633fd8b04bfeceb5fe4e631daa
related-to-bug: 32815560

Merge "libaaudio: change aaudio_audio_format_t to aaudio_format_t" into oc-dev

libstagefright: Add error handling in mediacodecsource

During camcorder recording, if encoder throws an error,
it intimates error to media codec source.
In such error cases, added functionality to stop puller
and clean up the graph to avoid memory leaks.

Bug: 34870133
Test: Video Recording.

Author: Uma Mehta <umamehta@codeaurora.org>

Change-Id: I487d96f855f541033e8ec7872e86d00bafb96daf

media: Handle STOP action properly.

Release all buffers and set mEndOfStream when handling STOP action.

Bug: 36682265
Test: Video Recording and MediaRecorder CTS test.
Change-Id: I2f40f0dca4e0b062ab43bfc5e6a646f649fdc524

Merge "libeffects: Fix EQ's 5th band upper limit" into oc-dev

Merge "Allow MediaExtractor to create FileSource" into oc-dev

Nuplayer logging empty records to media.metrics

Changes to ensure that nuplayer always reported a particular set
of metrics through android.media.MediaPlayer.getMetrics() changed
the shape of what we should have considered as an empty record.
This caused some noise in the reported media metrics logs.

Also removed a no-longer-wanted ALOGD() diagnostic.

Bug: 36658856
Test: read 'dumpsys media.metrics' output
(cherry picked from commit e4ae17109893b6388299a1f46585fb5d00a0a70e)

Change-Id: I2400e3636bb395aa5f46faf0142b2767f4fad610

libeffects: Fix EQ's 5th band upper limit

The old value: "1" is invalid, as follows from the code that
uses these values.

Bug: 36651594
Change-Id: I0516272241ffc1c4d346d8d66b06169b981bae2c
Test: VTS test for Audio Effects
(cherry picked from commit 3cf264448dba3b547babef35b2fc63c13047f491)

Merge "Fix reclaim sessions (DO NOT MERGE)" into oc-dev

Remove dependency on persist.hal.binderization.

Test: Manual use of Camera, Movies, Photos and YouTube apps.

Test: With CtsMediaTestCases.apk installed,
adb shell am instrument -e size small -w
'android.media.cts/android.support.test.runner.AndroidJUnitRunner'

Bug: 34274385
Change-Id: I5251fea337f33460d1f0db174cedc465822f2a47

fix another missing include

Test: built
Bug: 36603618
Change-Id: I3dd55a96fb29e42fc7accb40a471d11a37cb3296

libaaudio: change aaudio_audio_format_t to aaudio_format_t

Test: no functional change, it compiles
Change-Id: I3bdb1472a09ef2a01cbca0b1059257b72a980e1a
Signed-off-by: Phil Burk <philburk@google.com>
(cherry picked from commit 50a3ad82bcca719eebe71bece07cb4745f5e2c6d)

Camera: Map between bufferqueue gralloc0 usage, and HIDL gralloc1 usages

Otherwise some bits aren't where they're supposed to be.

Also stop using HW_CAMERA_ZSL; we need to only set HW_CAMERA_READ, and it's
confusing to set a producer flag on the consumer usage side.

Test: Camera CTS passes
Bug: 35215313
Change-Id: I23e6e60bf875fe9d8f2d7a1f805d2ef854c16b97

Merge "fix missing include" into oc-dev

Fix reclaim sessions (DO NOT MERGE)

[ Merged from master ag/2027840 ]

The SESSION_RECLAIMED event needed to be propagated to the jni native
interface.

Test: Verified by
  com.google.android.media.gts.DrmSessionManagerTest#testReclaimSession
  (GtsMediaTestCases) test

b/36589391

Change-Id: Ia3f17e20be19bc2e7bb88a2354d231917bfce1e8

Add Flag for Enabling 64-bit Media DRM Server

We are ready for devices to begin migrating to a 64-bit Media DRM
Server. However, not all devices are ready to make this jump yet. A
device needs to have all of its DRM Plugins ready to run as 64-bit
before it can do this. This includes having 64-bit hardware
integrations.

So that devices can turn on 64-bit support as they get ready, I am
adding flags that can be set in a device's device.mk file. There are two
parts:

* ENABLE_MEDIADRM_64 enables building the mediadrmserver as 64-bit.

* Setting drm.64bit.enabled=true in PRODUCT_PROPERTY_OVERRIDES switches
  mediadrmserver to searching for DRM Plugins in the 64-bit directory.

Test: Validated that turning on these flags enables a 64-bit
      mediadrmserver that loads 64-bit DRM Plugins from the lib64
      directory (after setting DISABLE_TREBLE_DRM to true to load this
      code as opposed to the Legacy DRM Plugin HAL)
Bug: 36076017
Change-Id: I141cfd57ff75b40c8b0cb40a5bf1439a1ab5b766

Allow MediaExtractor to create FileSource

Remove binder call to MediaServer for reading files.

Test: Photos, Play Music, Play Movies, Youtube
Bug: 29125703
Change-Id: Id6abf592bf6d70b81158f6038b1982f0a537b9b9

Merge changes from topic 'heapbase' into oc-dev

* changes:
  DRM: more fixes for heap base mapping -- DO NOT MERGE
  Revert "Revert "Fix decoder instantiation during playback"" -- DO NOT MERGE

DRM: more fixes for heap base mapping -- DO NOT MERGE

Heap base for the same heap could be mapped to different values
after they go across binder to CryptoHal. So we can't use heapbase
to index the heaps.

Since each ACodec instance allocates all its shared memory buffers
from the same memory dealer, we let CryptoHal assign a sequence
number to the ACodec when it calls setHeap. In subsequent calls
to CryptoHal::decrypt, reference the heap by the seq num, and ignore
the heap base address.

Bug: 36479980
Bug: 36209723
Bug: 36660223

Test: the above bugs don't repro

Change-Id: I2f519a689a5891447385d1bf9d6e668bb3b4dbe2

(cherry-picked from bf628da1e231e2e4d6bf61f9884e120bae3f9156)

Revert "Revert "Fix decoder instantiation during playback"" -- DO NOT MERGE

This reverts commit de7268d8e20b883ec88a7ff19ad560a665373484.

Bug: 36479980
Bug: 36209723
Bug: 36660223

Change-Id: I8d83305a28dc35cee16f8f1068c90fdd0b9effff
(cherry picked from commit 6aa5c0662c86c7f7b2890577a207086eeecbd177)

fix missing include

Test: built
Bug: 36603618
Change-Id: I836e2458203600768aa199f243206d5e0d9d4d39

Camera: Update 'indexOfKey' return check

Method 'indexOfKey' will return 'NAME_NOT_FOUND' error
status in case it doesn't find any values matching the
given key. Checking for anything other than this error
code could lead to instabilities.

Bug: 35925482
Test: Manual using application
Change-Id: Ie72eb29776b27a6d485f6e42ee7e62c62795ca9e
(cherry picked from commit 4219c290e526c6cb1767038776fff4f9b5d74cc7)

fix missing include

Test: built
Bug: 36603618
Change-Id: I739a13d3072e1fc168479cb26479cdbb4d19c2dc

Merge "stagefright: fix casting error in AMessage::findAsInt64"

Merge "AudioFlinger: Improve Thread logging."

Merge "Camera: fix FlashLightTest"

AudioFlinger: Improve Thread logging.

Test: dumpsys media.audio_flinger
Bug: 30572472
Change-Id: I43f72354a6ece045f5f9f664946a406166974258

stagefright: fix casting error in AMessage::findAsInt64

Change-Id: I7d9452e55611565cc4208d1835872a0de6f5a351

Merge "Make CameraStatus::cameraId to use String16."

Merge "Enable CFI for libstagefright_foundation."

Make CameraStatus::cameraId to use String16.

Java strings are passed as UTF16 and attempt to pass cameraId
with alpha symbols via CameraStatus leads either to corrupted
or null strings.

Test: build/flash/boot
Test: verified together with ag/1841664 that alpha camera ids
are passed correctly.

Change-Id: Ic6045322137c156aa03bbcdccfe81f8f0ecb0618

Camera: fix FlashLightTest

Fix issues related Treble HAL1 path.

Test: FlashLightTest pass on Angler HAL1/module 1.0
Bug: 35674539
Change-Id: I6bd749464c0e6e437a4743d2d66a0a5f54cdba38

stagefright: add support for named vendor extension parameters for MediaCodec

Relay vendor extension params to the component and populate vendor extension
parameters to the format. Failure to set these parameters is propagated to
the caller. Component may decide whether to fail on invalid/incomplete
settings.

Also add required support:

- propagate more errors from OMX to client, and be specific about
errors we ignore in MediaCodec.configure().
- add AMessage::findAsInt64

Bug: 32746650
Change-Id: I67ee8a5c1d3e65816cdadc28d228721925e2e327

Enable CFI for libstagefright_foundation.

Adds CFI (and diagnostics for it) to libstagefright_foundation.

Bug: 36219323
Test: ENABLE_CFI=true m -j40 # builds and device boots

Merge "NdkMediaCodec: add AMediaCodec_signalEndOfInputStream"

Merge "NdkMediaCodec: expose surface APIs in manifest"

Merge "Revert "ACodec: update native window crop rectangle""

Revert "ACodec: update native window crop rectangle"

This reverts commit c7b2b233d26e80ecf823078a7504e458ba9efe8c.

Fix: 36553910
Fix: 36541617
Test: manual
Change-Id: I5f0023dee45bd9d574ef9a27c3456ca40ed75f0a

NdkMediaCodec: add AMediaCodec_signalEndOfInputStream

Bug: 32746065
Change-Id: I20cdb02a93dfffdb2379db885f50738fb3a502d9

Merge "stagefright: properly handle crypto object reset"

NdkMediaCodec: expose surface APIs in manifest

Bug: 32746065
Change-Id: Ie2c9ce33c839f38fffb1d96f92a773b1c3357249

stagefright: properly handle crypto object reset

Bug: 36449774
Test: Run PoC in the bug
Test: adb shell am instrument -e size small -w 'android.media.cts/android.support.test.runner.AndroidJUnitRunner'
Test: Play an encrypted content on Play Movies
Change-Id: Ia7b38ed586460ad51fa66a6959211a7dfffa3493

Merge "Add support for USB headset device"

Merge changes from topic 'preview_video_surface_share'

* changes:
  Camera: Do not crash if detachNextBuffer fails
  Camera: Allow different kinds of IMPLEMENTATION_DEFINED surfaces to share

Merge changes from topic 'ims-o'

* changes:
  NdkMediaCodec: implement setParameters to signal dynamic parameters
  NdkMediaCodec: implement createPersistenInputSurface and setInputSurface
  NdkMediaCodec: implement createInputSurface

Add support for USB headset device

Add missing rules for USB headset devices in routing, volume and effects
management.

Test: playback and catpure regression tests.
Change-Id: I42cd157f483deba2371a05d332cd11dceb2b2fa3

Merge "ACodec: update native window crop rectangle"

ACodec: update native window crop rectangle

In setupNativeWindowSizeFormatAndUsage, set the crop rectangle
with the output port's crop value.

Fix: 34869730
Test: adb shell am instrument -e size small -w 'android.media.cts/android.support.test.runner.AndroidJUnitRunner'
Change-Id: I415be069d02d88dcd6b7c2460dda6df191ee7434

Merge "MediaRecorderClient: Fix binder dereference while connecting to camera"

Merge "NBLog exchange author and timestamp positions in log"

Merge "Move mediacodec to vendor partition."

Merge "Revert "Fix decoder instantiation during playback""

Revert "Fix decoder instantiation during playback"

This reverts commit 33327c7f0f04bcce3f8e8ee23953f87efc423037.

Bug: 36479980
Bug: 36209723

Change-Id: I8ff9422086f5ba9ba5df60135d20b9d8bfc0be52

Merge "media: Fix a bug in recording pause/resume."

media: Fix a bug in recording pause/resume.

Bug:35950379
Test: Camera recording and keep pause, resume, pause...
Change-Id: I5c81016e51f40e36430553990724fef3ccb23bb5

Camera: Do not crash if detachNextBuffer fails

Test: Camera CTS
Bug: 33777818
Change-Id: I965d741df6ce6bdb701dbe1d721cf42ea53c7a40

Camera: Allow different kinds of IMPLEMENTATION_DEFINED surfaces to share

Do not guard against mismatching usage flag and data space for
IMPLEMENTATION_DEFINED formats, so that different formats can share
the same stream if HAL support is available.

Test: Camera CTS
Bug: 33777818
Change-Id: I62a1f458c06ae417c9a2c407d433319a1158fdcc

Register new NDK API

Bug: 35114769
Test: Build and CTS (android.media.cts.NativeImageReaderTest)
Change-Id: I8ecad0b30dd5621d41e9d337d63abed234dbbabe

Implementing new NdkImageReader APIs

Bug: 35114769
Test: Build (should have no functional change from headers change), and
      run CTS test CtsNativeImageReaderTestCases on a Pixel.
Change-Id: Icb40196363d9f10a5d853c17f0bc378b6c27b3f6

Extending AImage/AImageReader headers

This introduces new API that allows AHardwareBuffer access via
AImage/AImageReader.

Bug: 35114769
Test: Build (should have no functional change from headers change), and
existing CTS tests run using cts-tradefed on a Pixel.

Change-Id: I7b6505ca3b9489d86eb564d1ae5707fd3fa25668

seccomp-x86: allow sched_yield and gettid syscalls

Addresses:
media.extractor: libminijail: blocked syscall: sched_yield, gettid

Bug: None
Test: make USE_CLANG_PLATFORM_BUILD:=true SANITIZE_TARGET=address -j42
      media extractor complains no more.
Change-Id: I3d499b0400df10ede08801fe6b608e6003727734

Merge "audiohal: Exit instead of crashing when HAL factory can't be obtained"

audiohal: Exit instead of crashing when HAL factory can't be obtained

Replace LOG_ALWAYS_FATAL with ALOGE + exit, so the process still
restarts, but without a crash. This avoids confusion in bug triaging
and also helps to eliminate noise in bug reports statistics.

Bug: 36424777
Change-Id: I623cbc6a5b0d3351f849810dd67708d421efd61a
Test: reproduced error condition, checked that audio server restarts

MediaRecorderClient: Fix binder dereference while connecting to camera

IServiceManager.getService can return NULL. Check for NULL and
return NO_INIT on failure.

Bug: 34870127
Test: adb shell am instrument -e size small -w 'android.media.cts/android.support.test.runner.AndroidJUnitRunner'
Change-Id: I6d71f8c0e89a556e64f5250ba1ab742e485f1ebc