tree 5d23e0e21a3f000dd0072d4ff6417d83c06df5c5
parent f9e7c8b8189fcdb8441a0174688ee5cbb9dd5cb2
author Steven Moreland <smoreland@google.com> 1660062466 +0000
committer Cherrypicker Worker <android-build-cherrypicker-worker@system.gserviceaccount.com> 1678983098 +0000

libhidl: check ro.debuggable less

We were always checking this when retrieving services, but
we don't need to (we can check it only when treble testing
override is on).

There are a few things we could consider in this codebase:
- checking debuggability when the testing override is initially
  set (and warning then) VS now when it is read
- avoiding duplicated calls to isDebuggable in
  isTrebleTestingOverride and getRawServiceInternal.
- removing 'kEnforceVintfManifest' since it is always true now.

However, given that we're deprecating HIDL and in the interest
of keeping changes simple, focusing on other things instead
of these improvements.

Ignore-AOSP-First: cherry-pick for tm-qpr-dev
Bug: 193912100
Test: boot
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:3b6a233eba04571ea6178d0c49e126b099de1b30)
Merged-In: I6df328dead00c05dcbd8eedf63fc24a8d6141871
Change-Id: I6df328dead00c05dcbd8eedf63fc24a8d6141871