Merge changes from topic 'libutils-remove-string'

* changes:
  RegionHelper.h: include <limits> directly.
  opengl egl_object.h: add <atomic> include.

Merge "Use std::string for profile operations instead of const char" am: 7c7d888d94 am: ccaa1f3f78
am: 0045d606d1

Change-Id: I6ad63a777811dbc5e5d74870441d8c76dc3273af

Merge "Use std::string for profile operations instead of const char" am: 7c7d888d94
am: ccaa1f3f78

Change-Id: If09e67000ca5818c27bbfe521554c22209c0d04f

Merge "Use std::string for profile operations instead of const char"
am: 7c7d888d94

Change-Id: I44676d7bc0b9297ccb920066284f371fa2e40c3c

Merge "Use std::string for profile operations instead of const char"

Merge "surfaceflinger: simplify HWC buffer cache clean up"

Use std::string for profile operations instead of const char

Will make things cleaner when adding secondary dex profile support.

Also, add tests to verify profile path creation.

Bug: 26719109
Test: /data/nativetest64/installd_utils_test/installd_utils_test
Change-Id: I8cb1a11cefee21f5001d2b729110696d52fc8323

Merge changes I20d546fd,I15363803 am: 632dbe2c77 am: dc9fcf4218
am: 0c90e1c416

Change-Id: I7f976a58fea69e98bbc3126c8b5288c082422363

Merge changes I20d546fd,I15363803 am: 632dbe2c77
am: dc9fcf4218

Change-Id: I6f091a21f326682deaec7527b40a235fecc4c267

Merge changes I20d546fd,I15363803
am: 632dbe2c77

Change-Id: I5fe86348657aa5e7f733be63c2134af4fe4a5365

Merge changes I20d546fd,I15363803

* changes:
  Allow "android" package name.
  Update is_valid_package_name() to match framework.

Allow "android" package name.

Test: /data/nativetest64/installd_utils_test/installd_utils_test
Bug: 36062649

(cherry picked from commit ab7ac8d5a04bd3f38b85ce20ae5bb382f2a26585)

Change-Id: I20d546fdd20813e96bcb5f061ca5b352dff44450

Update is_valid_package_name() to match framework.

The framework has a new format for /data/app directory names in OC
to protect user privacy.  This change updates the validation logic
in installd to support the new format.

For consistency the logic now mirrors the package name validation
logic used by PackageParser.

Test: /data/nativetest64/installd_utils_test/installd_utils_test
Bug: 33705143

(cherry picked from commit 367ace2f77105f7a09b34a50bc875fd4fa591177)

Merged-In: I41d270942479c52e063c29a2cb9552d7b238b86e
Change-Id: I15363803fb45310697e29be810e800dc1b1902d0

Merge "Re-add mistakenly removed lock in SurfaceFlinger::removeLayer."

Merge "[SurfaceFlinger] Use sf phase offset from CS"

Re-add mistakenly removed lock in SurfaceFlinger::removeLayer.

Lock was mistakenly removed with "Add detachChildren transaction"
when refactoring the weak-pointer to a strong-pointer.

Test: I will try and follow up with a stress-test in Transaction_test.
Bug: 36117490
Change-Id: I94171c86334bb52b974392186e1c3ebbf9f33a05

surfaceflinger: simplify HWC buffer cache clean up

When a Layer is no longer connected, we destroy the associated HWC
layers on next call to SurfaceFlinger::rebuildLayerStacks or when
the Layer is destroyed.  There is no need to listen to
onBuffersReleased.  Besides, we need to perform the cleanup from the
main thread as we only talk to HWC process from the main thread.

While at it, move HWComposerBufferCache to its own files.

Bug: 35320590
Test: manual
Change-Id: Ifa32f24076b094c8fa9cda8572b03d5bfb8e0b93

RegionHelper.h: include <limits> directly.

Includes are transitively imported by <string> in String8.h + String16.h
but that include is being removed.

Test: pass
Change-Id: I378d533ee5445bc0286c7535581a491f55ca36c9

opengl egl_object.h: add <atomic> include.

Includes are transitively imported by <string> in String8.h + String16.h
but that include is being removed.

Test: pass
Change-Id: I6176389d41aa5eb6fdf8b8afe3378eeb877bc525

Merge "Revise virtual touchpad interface."

Remove debug flags

I accidentally left the debug flags on my last commit.

Bug: None
Test: Still compiles and works.
Change-Id: I9fc652e643aa53ebb531332c3b156b8f001837d8

Merge "Fix DummyNativeWindow to respond to new query"

Merge "Reland: Remove NativeWindow and unused DVR functions"

Fix DummyNativeWindow to respond to new query

The `NATIVE_WINDOW_IS_VALID` query must now return true in order for
`eglCreateWindowSurface` to succeed. Technically this isn't true for our
dummy window, but we only need it to create the context, it gets dropped
immediately afterwards.

Bug: 36102224
Test: Created a simple test application. Also ran a Vr application with
      these changes applied.

Change-Id: I2d559962d28db4b1cb6ad188269e4e54cf47245d

Reland: Remove NativeWindow and unused DVR functions

1/ Remove the NativeWindow-backed DVR DisplaySurface implementation.
2/ Unused API such as: dvrCreateDisplaySurface and friends are removed.

Bug: 36033302
Test: Built system, ran basicvr, ran particles on marlin and lucid. Also
updated graphics_app_tests to stop testing deprecated API.

Change-Id: I586a6933f18a5c116bcffdf390ada122fadb9545

Merge changes from topic 'igraphicbufferallocator'

* changes:
  get rid of IGraphicBufferAlloc
  Remove a dependency of EGL on ui/GraphicBuffer.h
  remove EGL dependency on libbinder

Merge "Disable input-vdex when doing speed-profile." am: 028ad63e80 am: a5582c290c
am: a15409a340

Change-Id: I3261131e0f71f6305cdb177ef227ab114be23c60

Merge "Disable input-vdex when doing speed-profile." am: 028ad63e80
am: a5582c290c

Change-Id: I297b481892c75babde5b239bf6b1fe4e3701244e

Merge "Disable input-vdex when doing speed-profile."
am: 028ad63e80

Change-Id: I3ec636cd526bcdb749fc3af87f2ac0d301fbcafd

Merge "Disable input-vdex when doing speed-profile."

Disable input-vdex when doing speed-profile.

input-vdex and dexlayout are incompatible with each other.

bug: 36084905
bug: 36076104
bug: 36059948
bug: 35872504

Test: device boots, vdex works on OTA, input-vdex is not passed
      for speed-profile.

Change-Id: I194311041899399db422867670f5e788d6b9ceeb

get rid of IGraphicBufferAlloc

buffers can now be allocated in-process.

Test: compile & run
Bug: cleanup
Change-Id: I3d4317a9bed20a6d8be2b7ac8fbb85738efb3657

Remove a dependency of EGL on ui/GraphicBuffer.h

the conversion to a ANativeWindowBuffer is now
handled by libnativewindow.

Test: compiled & run
Bug: cleanup
Change-Id: I52dd64574afbc1eff5bdbefacec234a79f8acd97

remove EGL dependency on libbinder

This gets rid of the
eglCreateNativeClientBufferANDROID extension,
which is replaced by NDK’s AHardwareBuffer.

Bug: 35101209

Test: Not tested yet
Change-Id: I7905eb875139781a23e9fa0b3447ea1ad25c71cf

Merge "Create libhwc2on1adapter"

Merge changes from topic 'vintf_arch' am: 45370128a5 am: c2b36f5696
am: 628dafc85e

Change-Id: Ib17fb1e919fdc5a1d3ce652debf1f0d530fde57f

Merge changes from topic 'vintf_arch' am: 45370128a5
am: c2b36f5696

Change-Id: Id6a10176042b63fcefe8298dae2df4c5ac4c5297

Merge changes from topic 'vintf_arch'
am: 45370128a5

Change-Id: Ia8e62d1a64e5a893b651c46e5872854e2f52034f

Merge changes from topic 'vintf_arch'

* changes:
  Update to use bitness info in VINTF.
  Fix forEachTable will copy the table.

Merge "check that the NativeWindowType is valid in eglCreateWindowSurface"

Create libhwc2on1adapter

Extract the adapter out of SurfaceFlinger library to allow code
behind HAL to use it.

Test: Manual
Change-Id: I415aa2674564eccb2c38f76086807f834177f80f

Update to use bitness info in VINTF.

Bug: 35966597
Test: lshal -ir has bitness info for passthrough ref section
Test: lshal --init-vintf has bitness info in the manifest.
Change-Id: Ie9f52bb1ca30ce69856dbf44fab8287563014550

Fix forEachTable will copy the table.

std::initializer_list<T> has T deduced to Table, which will
copy the table. Use pointers instead.

Test: pass
Change-Id: I7da40eb0f5d0171244a9b9caacf37b41fe0af304

Merge "Delete typo in SurfaceFlinger documentation"

Merge "Revert "Remove NativeWindow and unused DVR functions""

Merge "Allow "android" package name."

check that the NativeWindowType is valid in eglCreateWindowSurface

This will avoid certain crash (null dereference),
and return an error instead. This is useful for
app developers as java language apps are not
expected to crash when passed bad arguments

This doesn’t fix bug 36065206, but was prompted
by it.

Test: compile / run
Bug: 36065206
Change-Id: I90d25730b3dd292d756b4c38b51320a466fa6d48

Delete typo in SurfaceFlinger documentation

Test: None (this is a comment)
Change-Id: I0876a40697fe5bb5245513520bc1f9a4bf1139f7

Merge "Fix the usage of EIGEN_STATIC_ASSERT_SAME_MATRIX_SIZE"

Merge "A helper header for matrices."

Revise virtual touchpad interface.

- Explicit start and stop, outside of which the evdev devices
  don't exist.
- Permission test (not compiled by default pending build & SELinux
  support for temporarily retaining a second copy of the service
  for vr_wm).
- Enforce a single user of the touchpad.
- Support 'dumpsys'.

Bug: 36051900
Test: log inspection
Change-Id: I038ed2632d5adf50a3565a981031691d5dc5f7cd

Revert "Remove NativeWindow and unused DVR functions"

This reverts commit a001af5ae07e156f9c31dcdb24fb7ddead4d9f1e.

Change-Id: Ieb60bc8e8e7caefee30b0fbd13cda15298c32075

Fix the usage of EIGEN_STATIC_ASSERT_SAME_MATRIX_SIZE

  - Eigen::internal::traits is not specialized for Eigen::MatrixBase, it
  should only be used with the Derived types. Just as Eigen's own tests
  do.

Bug: 34161771
Test: mm
Change-Id: I997b26b4a470c4ba851afe6edc5c602ae450c222

Allow "android" package name.

Test: /data/nativetest64/installd_utils_test/installd_utils_test
Bug: 36062649
Change-Id: I9b41a34822a1a194d256f5b99d5955e51716029c

Merge "Remove NativeWindow and unused DVR functions"

Merge "Interface for multiple virtual touchpads."

Merge "libbinder: use sysconf(_SC_PAGESIZE) to get pagesize" am: b638bf8d3f am: f83403fdd2
am: 391becb8a7

Change-Id: I11546e02a736bd6c6ece606831462f7ef85c4ec4

Merge "libbinder: use sysconf(_SC_PAGESIZE) to get pagesize" am: b638bf8d3f
am: f83403fdd2

Change-Id: If136ef494a22fb750321272302db7a0d8144eb95

Merge "libbinder: use sysconf(_SC_PAGESIZE) to get pagesize"
am: b638bf8d3f

Change-Id: I90088d0769cfe337b2c2c5c308f5fa7e832ca4df

Merge "libbinder: use sysconf(_SC_PAGESIZE) to get pagesize"

Merge "Update is_valid_package_name() to match framework."

Update is_valid_package_name() to match framework.

The framework has a new format for /data/app directory names in OC
to protect user privacy.  This change updates the validation logic
in installd to support the new format.

For consistency the logic now mirrors the package name validation
logic used by PackageParser.

Test: /data/nativetest64/installd_utils_test/installd_utils_test
Bug: 33705143
Change-Id: I41d270942479c52e063c29a2cb9552d7b238b86e

Merge "Fix enum ordering in sensor NDK"

Merge "Temporarily disable vr_wm visibility while we figure out what triggers it to come up at all time. This is to quickly unblock everyone."

Fix enum ordering in sensor NDK

Test: compiles
Change-Id: Ie186270af2f51d723855cdc62f0b09ffe1ab6ecf

Merge "Sensor direct report with HardwareBuffer channel"

Remove NativeWindow and unused DVR functions

1/ Remove the NativeWindow-backed DVR DisplaySurface implementation.
2/ Unused API such as: dvrCreateDisplaySurface and friends are removed.

Bug: 36033302
Test: Built system, ran basicvr, ran particles on Marlin. Also updated
graphics_app_tests to stop testing deprecated API.

Change-Id: I6ea6063c56fabacddcc301bc91567cc1b3f5d67e

Merge "Disable Eigen static assertion for tests/numeric_test.cpp"

Temporarily disable vr_wm visibility while we figure out what triggers it
to come up at all time. This is to quickly unblock everyone.

Bug: 36036583
Test: Compiled
Change-Id: Ibf045e7354ee96b7609af1c47b1e64b63659feb9

Merge "Add ASensorEventQueue_registerSensor and default error value"

Disable Eigen static assertion for tests/numeric_test.cpp

  - Clang complains with the Eigen static assertions in the tests, after
  Eigen 3.3.3 rebase.
  - Disable the static assertion until it is fixed in Eigen / Clang.

Bug: 34161771
Test: mm
Change-Id: I25056cf9df9bbb32f979390e02df1f23ac760bab

A helper header for matrices.

Bug: 34720474
Test: No functional change.
Change-Id: Ibcb8f11feeeadf6f484396d66c0b0cd1ef751f94
(cherry picked from commit 6fff8f0580e352fd81ecd7a0b6b58a6604bcbed7)

[SurfaceFlinger] Use sf phase offset from CS

If present, use sf phase offset from the config Store.

Test: Manual
Change-Id: I2dd7ecf984b7484f6b447699680f54c7c8e221a0

Merge "Add MiniFence to drop HWC2on1Adapter libui dep"

Merge "[SurfaceFlinger] Comment SurfaceFlinger HWC2"

Sensor direct report with HardwareBuffer channel

Implement creation of sensor direct channel in sensor
service for sensor direct report feature.

Bug: 30985702
Test: cts-tradefed run cts --module CtsSensorTestCases
      --test android.hardware.cts.SensorDirectReportTest
Test: cts-tradefed run cts --module CtsSensorTestCases
      --test android.hardware.cts.SensorNativeTest

Change-Id: I52350811df538986cea7f5e1fb336fad53234c86

Add ASensorEventQueue_registerSensor and default error value

ASensorEventQueue_registerSensor is new function created in 2016 but
is never exported at NDK level. In this CL the function signature is
formalized and the function name is added to NDK function export
table. Internal class SensorEventQueue is also changed to adapt.

Default error values are also defined.

Test: cts-tradefed run cts --module CtsSensorTestCases \
          --test android.hardware.cts.SensorNativeTest
Change-Id: I74dc72e50ea4bae6d2d8944ce6d2e191ed2f495e

Add MiniFence to drop HWC2on1Adapter libui dep

This class will soon become a library used by devices with no support
for HWC2. They will link against libhwc2on1adapter. Drivers should have
no depedencies on libui.

Test:Manual
Change-Id: Iabd2aa89fc3b737a999632a16c4f6c30464787c4

Merge "split libsensor our of libgui "

Interface for multiple virtual touchpads.

This passes the touchpad selection across binder, but does not yet
actually implement multiple touchpad devices.

Bug: b/35992608
Test: primary only in VirtualTouchpad_test
Change-Id: I7529735362e0e38805b9c0e52e07b5efc86bd535

split libsensor our of libgui

Test: compile & run
Bug: treble cleanup
Change-Id: I5b2c2a14f7d3b364d25695cf0901c5b08cfb0fc9

Merge "Installd: Clean up code related to foreign dex use" am: 62b34b57c4 am: 9d4819ff2e
am: 9181bdb57f

Change-Id: I2257654b3b73e74a92a179e1068f771e2b6af186

Merge "Installd: Clean up code related to foreign dex use" am: 62b34b57c4
am: 9d4819ff2e

Change-Id: I21299c36def0a5aa5e202d381abd4ad184c2e7b6

Merge "Installd: Clean up code related to foreign dex use"
am: 62b34b57c4

Change-Id: Id7beb09ed3d3b79b3b818dd5192b457605f5a54c

Merge "Installd: Clean up code related to foreign dex use"

Merge "Fix the compiler error for the ternary operator, caused by Eigen rebase."

Merge "Disable HDR extension for now"

libbinder: use sysconf(_SC_PAGESIZE) to get pagesize

pagesize is not always 4KB, use sysconf(_SC_PAGESIZE) to get
real pagesize.

Change-Id: Ib2c82c3a842257601a5c304da3a1f0b07c6ab8c0
Signed-off-by: Ganesh Mahendran <opensource.ganesh@gmail.com>

Merge "InputDevice warning fixes and make warnigs into Error"

InputDevice warning fixes and make warnigs into Error

use const char * for string literals in InputDevice
Remove unused variable in Keyboard

Bug: 35993509
Test: mm in frameworks/native/libs/input builds without the warning
Change-Id: I11242835eae855386c47e56d6c36e5a2f912a28f

Disable HDR extension for now

Test: adb shell vkinfo
Test: Should not see VK_EXT_hdr_metadata in extension list.

bug: 36007564

Change-Id: I96a365d75ddec75856af2fda834d3a4f8368ebb0

Fix the compiler error for the ternary operator, caused by Eigen rebase.

Bug: 34161771
Test: build
Change-Id: I3c0c1b3667a3d1f56052b247c94c6255e9e12004

[SurfaceFlinger] Document layerStack

Test: None, these are only comments.
Change-Id: Id5d45a5847d253bf92263091e5b322a1577fabe1

Merge "Double check hidl service at init + meaningful abort when hidl crash"

Merge "Make the AHardwareBuffer extension compatible with C."

Merge "lshal: update descriptions in output." am: 074c1cd157 am: 105e9391ac
am: 76ccc8cbd1

Change-Id: Iadb1ea22c087d643d0664919411a3e2408d12fd1

Merge "lshal: update descriptions in output." am: 074c1cd157
am: 105e9391ac

Change-Id: I9dfa8644300c91bc2149e9f7f4ba5c978b85ecf5

Merge "lshal: update descriptions in output."
am: 074c1cd157

Change-Id: Iaa60e243566133d1674b2591becdbc9ead403754

Double check hidl service at init + meaningful abort when hidl crash

Sensor hidl service will kill itself when it detects
double-connection(*) from framework sensor service in order to avoid
potential deadlock.  This cl adds a no-op call to hidl service at
initialization of framework service to allow early detection on hidl
service side. It also adds retry mechanism in this case to avoid
another system server crash.

Log messages is added when hidl call transport failure to help better
identify problem. There is no sane recovery at this point and fatal
log is preferred.

* Double-connection to hidl service happens when framework sensor
service crash but leave a lingering connection in sensor hidl service.
Auto restarted framework service will attempt to connect hidl service
again which result in a double connection.

Test: manually killing sensor hidl service and/or restart runtime,
      with and without sensor service in restricted mode, observe
      system recover to good state.

      Pid of sensor hidl service can be found by lshal
          lshal | grep sensor
      switch to restrict mode
          dumpsys sensorservice restrict a.non-exist.package

Bug: 35727090
Bug: 35843387
Bug: 35702298
Bug: 35919167
Bug: 35848670
Bug: 35903635
Change-Id: Ie5c24300503e125d9de031a70948e2bda1eb6dc6

Merge "Fix <vulkan/vulkan.h> compatibility with C."