After upgrading libc++, Clang fails to compile DumpManifest.cpp.

prebuilts/clang/host/linux-x86/clang-r498229/include/c++/v1/__memory/unique_ptr.h:63:19: error: invalid application of 'sizeof' to an incomplete type 'aapt::CommonFeatureGroup'
    static_assert(sizeof(_Tp) >= 0, "cannot delete an incomplete type");
                  ^~~~~~~~~~~

Fix the problem by deferring the instantiation of ~unique_ptr, by
making the ManifestExtractor constructor out-of-line and moving it and
the initialization of commonFeatureGroup_ to a point after
CommonFeatureGroup has been defined.

Bug: b/175635923
Test: treehugger
Change-Id: I9ab51f29724fded24773344aa36763ffeea02d00
Merged-In: I9ab51f29724fded24773344aa36763ffeea02d00

Merge "Revert "Move @ViewDebug.ExportedProperty annotations from private fields to public methods"" into main

Onboarding new clients to the flagging infrastructure will require updating this file frequently.

Test: updating OWNERS file
Bug: 298653405
Change-Id: I3c752a1b7785f1443572516119cf8090f30e3946

This reverts commit 7f2a6227.

Reason for revert: Verifying if it is culprit for 299257121

Change-Id: I854221ccc99ecb43628b03f06d6365937c703c83

Merge "Move @ViewDebug.ExportedProperty annotations from private fields to public methods" into main

Bug: 295394788
Test: make checkbuild
Change-Id: I370f66c469de73064dec2e42c539dc236dd69d1e

Today, the top-level DecorView is missing some useful pieces of data in its dump, for example the "V"isible and "E"nabled and "I"nvalidated and "D"irty flags. Add these back in.

Today it looks like:

```
    View Hierarchy:
      DecorView@41b7f45[MapsActivity]
        android.widget.LinearLayout{a1c3dbb V.E...... ......ID 208,0-1920,1080}
          android.view.ViewStub{c5878e4 G.E...... ......I. 0,0-0,0 #1020194 android:id/action_mode_bar_stub}
```

Note how the DecorView doesn't have the deep debug info the other views do.

This change updates toString to work the same as the other views, but including the extra title suffix information.

This is a new version of aosp/2276228, but on the main branch.

Change-Id: I35415f7625dc99a80412f6656800504eea876d91

Test: m checkbuild
Bug: 295394788
Change-Id: I9488bc5632cbd47c83f6b5f2df4c87eb324a1e8e

Merge "Align OWNERS for /core/java/com/android/server/security/ and /core/java/android/security/" into main

From java doc: If timeout is zero, however, then real time is not
taken into consideration and the thread simply waits until notified.

Bug: 298456150
Test: m
Change-Id: Id5cdf6a52a97fddb24d2fcf2dcaa6e7385eb7c46

Bug: 291751402

Change-Id: Ifb0bd87e7daf7577e4f37881613580d3566cebde

Bug: 295328116
Test: m framework-minus-apex
Change-Id: I8d03c997fdfaea237d732391443903a4a7508b6a

Delete proguard files that are either unused or redundant with
global or aapt-generated proguard flags. This change is a no-op.

Bug: 248580093
Change-Id: I6dcb3fb9584e9a15ac66104df6696780c166054d
Test: m + verify identical builds

Prefer the @UsesReflection annotation to generate the keep rules
necessary to preserve LocalTransport derived class names. These
class names should be kept as they are used to generate a name that
may be referenced by other packages. Other classes currently in the
proguard config are automatically kept by manifest-generated keep
rules, or are otherwise unnecessary.

This change is effectively a no-op under the current shrinking
configuration for both LocalTransport and EncryptedLocalTransport. If
obfuscation is enabled, this annotation will ensure continued
correctness.

Bug: 248580093
Test: m + ensure all classes kept
Change-Id: I867693861e5b1ca9765c926d676c0b18b16cc340

This empty method is different from the old one - it aligns with
std::string definition.

Bug: 295394788
Test: make checkbuild
Change-Id: I1900f2285ee3c3b1eb57d2cef3990896a64dae01
Merged-In: I1900f2285ee3c3b1eb57d2cef3990896a64dae01

Bug: 289776578
Test: presubmit
Change-Id: Id077cff3825cea0da2ea9fad06c53eeb4f8655a2
Merged-In: Id077cff3825cea0da2ea9fad06c53eeb4f8655a2

mUserIsTrusted is no longer used.

Test: atest TrustTests
Change-Id: I122987d4c2c9a8b4161d9f9edd360450b7bb9962

Change-Id: I33639360cc179f6840a887ad723d14464525c4af

Change-Id: I5d9c263cd33a5405d5909aa485ded8e6f85555b3
Bug: N/A
Test: N/A

* changes:
  TrustManagerService: remove ENABLE_ACTIVE_UNLOCK_FLAG
  Fix timer bug in TrustManagerService.