Merge "Fixes volume dialog in dark mode."

Merge "Fixed volume logging on sticky broadcasts"

Merge "Fix unused new stack when bring existing activity to front"

Merge "Release SurfaceControl native resource when test finishes"

Merge "Removed references to ActivityRecord and TaskRecord in AM (24/n)"

Merge "Remove obsolete AndroidTestCase"

Merge "Fix security exception when reading usb serial num"

Merge "Fixes crash when getting panel before setting components"

Merge "Support monitor input per display (2/2)"

Release SurfaceControl native resource when test finishes

By replacing factory methods, records all SurfaceControl and
Transaction objects created while test is running. Then release native
resources of those after test finishes. This will mitigate resource
presure while running WindowManagerService tests.

Bug: 116449554
Test: atest :presubmit in frameworks/base/services/core/java/com/android/server/wm
Change-Id: Iac99f80ee995bd64a82b965e4ea8a3135b0a5bd5

Remove obsolete AndroidTestCase

This CL also utilize google truth assertion helper.

Test: atest FramewroksServiceTests:ActivityManagerTest
Test: atest FramewroksServiceTests:TaskPersisterTest
Test: atest FramewroksServiceTests:UserControllerTest
Bug: 116449554
Change-Id: I95c3cc2c56be2ce53b95c030edb6370634546b10

Merge "MediaPlayer2: pass positions to native"

Merge "Add dwkang to OWNERS"

Merge "Skip calculating launch params if there is no root activity."

Merge "Removed unused variable (ViewState.mSession)."

Merge "Remove IInputMethodManager#finishInput(), which is NOP"

Merge "Fix incorrect proto apk loading"

Fix security exception when reading usb serial num

Bug: 117881221
Test: run projection on mojave board
Change-Id: Ife4c5e18b6d8f7c20b363f40119dfefd8ab008ef

Merge "Moved appNotResponding handling into ProcessRecord (23/n)"

Merge "Removing douple lookup when calling startActivityAsUser"

Merge "Revert "Revert "refactor radioPowerstate"""

Revert "Revert "refactor radioPowerstate""

This reverts commit 05188f3555723bd26522b410109bb1de1b7fbb8c.

Reason for revert: <INSERT REASONING HERE>

Change-Id: Id170063bfdb1e3325f67e8680231ab4deda7d65d

Bug: 117349311

Merge "Add a new config table to the settings provider for remotely configured parameters. This includes the minimum number of changes necessary to make the table work, but no API surface yet."

Add dwkang to OWNERS

Test: n/a

Change-Id: I98245fe411295b6112cdc73a13baf2ac70edeb9d

MediaPlayer2: pass positions to native

Test: cts
Bug: 112549021
Change-Id: I7576611f94f6538bbb96b95d44cb1d9624c77710

Removed unused variable (ViewState.mSession).

Bug: no, thanks!
Test: atest CtsAutoFillServiceTestCases

Change-Id: I8810f24e79da5f0d2bc65756280d604cb4f7494c

Fixes volume dialog in dark mode.

This change removes the caching of color values in VolumeDialogImpl such
that the dialog will change its colors correctly when dark mode is
toggled.

Change-Id: I241edc284a33f60e60ec5e54f2e4d6bb0a0eadfd
Fixes: 116860768
Test: manually

Removing douple lookup when calling startActivityAsUser

Test: atest \
      com.android.cts.devicepolicy.LauncherAppsMultiUserTest \
      com.android.cts.devicepolicy.LauncherAppsProfileTest \
      com.android.cts.devicepolicy.LauncherAppsSingleUserTest
Bug: 117888136

Change-Id: Ia25058b3bb2fb6cf57eedc5eecae4cb99275b4e7

Merge "Removed AMS.mStackSuperivor (22/n)"

Merge changes I3070e95f,Ia500dea6

* changes:
  Fixed groups with min priority children
  Added logging to debug the notification view hierarchy

Merge "Apply requirements when enabling provider"

Skip calculating launch params if there is no root activity.

Bug: 117810579
Test: atest FrameworksServicesTests:TaskLaunchParamsModifierTests
Change-Id: Ic4c506ab3221cf8082b15b714c1cd82b26e969d0

Merge "Add RingtoneManager.openDefaultRingtoneUri()"

Merge "Guarded against ArrayOutOfBounds"

Merge "Update CoreSettingsObserver internal state correctly."

Merge "Enterprise Policy for Private DNS Setting"

Add a new config table to the settings provider for remotely configured parameters. This includes the minimum number of changes necessary to make the table work, but no API surface yet.

Test: local tests

bug: 113100523

Change-Id: I47f89f5e6657a2a347e62cb40924bba4547f7dd9

[automerger skipped] DO NOT MERGE resolve merge conflicts of c6a1cfda93aac7fc004da73e3e5921b838e931a7 to pi-dev-plus-aosp
am: bae081d202  -s ours

Change-Id: I604ad31279cb133446327890f516becb3774714d

Removed references to ActivityRecord and TaskRecord in AM (24/n)

Once ActivityRecord and TaskRecord are moved to the WM package they
can no longer be accessed by classes in the AM package. This CL fixes
up some of the access before the move.
- Moved activity relaunch reasons to ActivityTaskManagerService and made
them public to allow access from the AM side in the future.
- Use task id to reference task in AppErrorDialog and AppErrors vs. the
TaskRecord object.
- Also, removed out-of-date code that tries to re-start a task if it
can't start in through recents. The reent start also looks for the task
in the stacks now so if the task exists it will be started through that
channel and we no longer need this fallback.

Test: Existing test pass
Bug: 80414790
Change-Id: I8df76a4f107fa526d726d7522eb5fcee6958b45d

Moved appNotResponding handling into ProcessRecord (23/n)

Allows for better seperation of AMS and ATMS, also the method mostly
accessed internal variables in ProcessRecord so it make sense for it
to be in that class.
Also, move inputDispatchingTimedOut back to AM side because it involves
lots of process stuff.

Test: Existing test pass
Bug: 80414790
Change-Id: I45b98dc550ff121e9df4bf004b2667af2426b79d

Removed AMS.mStackSuperivor (22/n)

The stack supervisor object should only be accessed through ATMS.

Test: Existing test pass
Bug: 80414790
Change-Id: I0648a95161a6a5c4ad714264d217c7f5e55852d7

Merge "Snapshot memory state for some native processes"

Fix incorrect proto apk loading

If a proto apk is missing resources.pb, AAPT2 would fail to detrmine the
apk format because of an incorrect check. This fixes that check and
removes the checking fuction from the LoadedApk header.

Bug: 117820549
Test: removed resources.pb from proto apks and confirmed that loading in
apks still works

Change-Id: If3628a821e7b59c7dfcbefb502b6080be083cec1

Merge "Move ViewGroup#mChildren[Count] to dark-grey list"

DO NOT MERGE resolve merge conflicts of c6a1cfda93aac7fc004da73e3e5921b838e931a7 to pi-dev-plus-aosp

Change-Id: I1a18690e85f66ff2a25329f9a050cb038fc0bcfb
Bug: None
Test: I solemnly swear I tested this conflict resolution.

Guarded against ArrayOutOfBounds

In some monkey tests, getAdapterPosition returns -1 (for example, if the
ViewHolder is not bound). In this case, we shouldn't proceed with the
moving operation.

Test: manual
Change-Id: I1eccc2c60136ddf5cc0175d88a3fddbe1d2a5a97
Fixes: 117769352
Fixes: 116871219

Merge "DO NOT MERGE Fixes clipping on PagedTileLayout" into pi-dev

Fixed volume logging on sticky broadcasts

Ringer mode changed is a sticky broadcast and is currently being logged on broadcast attach for normal and vibrate state (not silent). This change instead uses the sticky broadcast to initialise the current state rather than logging.

Test: manual
Change-Id: I09e91ec4b71fbbbfae71d188d9c858675a7e774d
Fixes: 117315582

Fix unused new stack when bring existing activity to front

If a non-top reusable activity is found when starting activity on
secondary display, the condition in
ActivityStarter.setTargetStackAndMoveToFrontIfNeeded will get launch
stack by the reusable activity. But it actually creates a new stack
with the target display which won't be used and seems as a leakage.
That may also disturb focused stack checking when it is on the top of
display (e.g. other tasks are moved to back).

This change gives the preference to use the stack of candidate task
if it already exists on target display. Then it will have the same
behavior as default display that reuses the existing stack.

Bug: 117799939
Test: atest ActivityStarterTests#testBringTaskToFrontOnSecondaryDisplay
Change-Id: I65213386d9802f39ac7ba0d377a06711913c672d

Merge "Do not move home task if Recent is not the previous top"

Merge "Revert "refactor radioPowerstate""

Revert "refactor radioPowerstate"

This reverts commit 700c8a0389dc45e729e3797433d03d875556325a.

Reason for revert: Broken master build

Change-Id: Ia0171ff9250228cc03685e7f03da8dd9e374826f

Snapshot memory state for some native processes

Which processes to snapshot is controlled by a whitelist.

Benchmark for taking the snapshot:
https://docs.google.com/spreadsheets/d/1vG9ku8Uu8104CmKbO4cNeEKVeeByvHY--p0_dK1GAdA/edit?usp=sharing
(The difference between the first two sheets.)
~20ms constant cost plus ~4ms per process.

Bug: 115968899
Test: manually verified that statsd is included in the report
Change-Id: Iba680531c563ba28fae849e44044313866b2103f

Support monitor input per display (2/2)

To support system gesture per display,
monitor Input should be separated by display.
So each DisplayContent would contain its own
PointerEventDispatcher to detect system gesture.

Bug: b/115602714
Test: atest DisplayContentTests ActivityManagerMultiDisplayTests
Test: atest inputflinger_tests
Change-Id: I7889128af82f0d61dbdc89a84d6ccf0b337e38b4

Merge "Private DNS: Add user restriction"

Remove IInputMethodManager#finishInput(), which is NOP

Currently InputMethodManagerService#finishInput() does nothing. Until
we fully understand what is the right approach on how and when
InputMethodService#finishInput() (Bug 9216494), let's remove this
unnecessary IPC from the IME client to InputMethodManagerService.

Bug: 9216494
Test: atest CtsInputMethodTestCases CtsInputMethodServiceHostTestCases
Test: atest FrameworksCoreTests:android.view.inputmethod.InputMethodManagerTest
Change-Id: I614050d20f4a7d9611dc0502e55e6ca3458a836e

Merge "Move displayId into MotionEvent"

Merge "Change the feature flag for Hearing Aid behavior"

Merge "Polish InputMethodManagerService#startInputUncheckedLocked()"

Merge "iorap: Remove blank newline in TEST_MAPPING" am: 22fe21df93 am: cbe655e781
am: d40cb164d5

Change-Id: I6637b6233ce90ec19437465050394232731540bb

Merge "iorap: Remove blank newline in TEST_MAPPING" am: 22fe21df93
am: cbe655e781

Change-Id: If091c4a00b32263691f4f0f28be6c58e53917715

Merge "iorap: Remove blank newline in TEST_MAPPING"
am: 22fe21df93

Change-Id: Id1b72f79c242f827ff04acd0deff2c4c9d4fe65d

Do not move home task if Recent is not the previous top

We shouldn't move the home task to front if Recents was
not the previous top activity on the display where the
task lands.

Bug: 111363427
Test: atest ActivityStackSupervisorTests

Change-Id: I39b51d5eec897bf83b2380dd8b7947c13cc7dd57

Merge "Unextract InputMethodManagerService#startInputInnerLocked()"

Polish InputMethodManagerService#startInputUncheckedLocked()

This CL aims to simplify logic recently added to IMMS to support
multiple displays (Bug 111364446).

With my previous CLs [1][2], now we can simply review what
InputMethodManagerService#startInputUncheckedLocked() is actually
doing.  Things we found and this CL does address are:

 1. Redundant if conditions against null IMMS#mCurMethodId.
 2. Timing to check IMMS#mSystemReady.
 3. Timing to check display id access
 4. Timing to update IMMS#mCurTokenDisplayId.
 5. Unnecessary complexity due to IMMS#mCurFocusedWindowClient, which
    is now guaranteed to equal to |cs| in that method.

Although this CL is not a mechanical safe refactroing, the new
behavior should be more efficient, solid and easier to maintain.

 [1]: I52f6c4cd1e02be3a59e9a87e33b0a44f4ba8d80b
      f91a2b102b4bd28036e7f31f5888f6f4629d9d70
 [2]: I71e259fa447dd06ff02b9ef8c958dc70bbce86ea
      caaa1d3d2d3248dcc1ad1ea4c65d5a8949fb4219

Bug: 117730713
Test: atest ActivityManagerMultiDisplayTests
Test: atest CtsInputMethodTestCases CtsInputMethodServiceHostTestCases
Test: atest FrameworksCoreTests:android.view.inputmethod.InputMethodManagerTest
Change-Id: Ic9232d9c3ec9802101df5f0bc511c55465b5bbe6

Unextract InputMethodManagerService#startInputInnerLocked()

With my previous CL [1], IMMS#startInputInnerLocked() is called only
from IMMS#startInputUncheckedLocked().  By unextracting
IMMS#startInputInnerLocked() again, it would be much easier for us to
understand what IMMS#startInputUncheckedLocked() is actually doing.

To decouple actual behavior change from cosmetic code refactorings,
this CL does nothing except for merging IMMS#startInputInnerLocked()
into IMMS#startInputUncheckedLocked().

There should be no behavior change.  Non trivial behavior
simplifications will be done in subsequent CLs.

 [1]: I52f6c4cd1e02be3a59e9a87e33b0a44f4ba8d80b
      f91a2b102b4bd28036e7f31f5888f6f4629d9d70

Bug: 117730713
Test: atest ActivityManagerMultiDisplayTests
Test: atest CtsInputMethodTestCases CtsInputMethodServiceHostTestCases
Test: atest FrameworksCoreTests:android.view.inputmethod.InputMethodManagerTest
Change-Id: I71e259fa447dd06ff02b9ef8c958dc70bbce86ea

Merge "MediaPlayer2: remove hidden api, setMetadataFilter and getMetadata"

Merge "iorap: Remove blank newline in TEST_MAPPING"

Merge "audio: apply volume upon hearing aid device connection" into pi-dev am: 9872b257cd
am: cf7cca0a43

Change-Id: I16d10f8ad3b10c934028cc35ed0fba57de0df3fa

[automerger skipped] Merge "Make SubscriptionManager.getSlotIndex public" am: 719310cd2c am: a781751b32
am: 59c8b87ad4  -s ours

Change-Id: I8db95a7b701c71caffc78f03ca618036c84cdf5f

Merge "audio: apply volume upon hearing aid device connection" into pi-dev
am: 9872b257cd

Change-Id: Ibb0a31b7e4750e904568d328384983979cf774a0

Merge "Make SubscriptionManager.getSlotIndex public" am: 719310cd2c
am: a781751b32

Change-Id: Ib7d3dd0f7ce06edf736c493cde6778365c366ae2

Merge "Stop using DIST_DIR directly, use dist-for-goals instead" am: a8f2601d28 am: 9b767e275e
am: fa5b204029

Change-Id: Ic052f5d1de317c2f271f5a2df83c733c79978679

Merge "audio: apply volume upon hearing aid device connection" into pi-dev

Merge "Make SubscriptionManager.getSlotIndex public"
am: 719310cd2c

Change-Id: Ia942ecfefef8f26ff0b1f140c0692c81aebd11fa

Merge "Stop using DIST_DIR directly, use dist-for-goals instead" am: a8f2601d28
am: 9b767e275e

Change-Id: I64a04f0c7b9220a8de872526ba37590b71fba458

Merge "Stop using DIST_DIR directly, use dist-for-goals instead"
am: a8f2601d28

Change-Id: Iabb2ddb086dda71887c3d50152db50f813c9556a

[automerger skipped] Merge "audioservice: fix a2dp connection race condition" am: e568805209  -s ours am: 6466847321  -s ours
am: d32dca9663  -s ours

Change-Id: I257b73130b71e0223a1c81d217c758a2a7bc1c96

Merge "Make SubscriptionManager.getSlotIndex public"

Fixed groups with min priority children

Previously  min priority  notifications would be
empty on the lockscreen. We're now showing them
if they are part of a group.

Bug: 117349034
Test: add group with min priority children on lockscreen
Change-Id: I3070e95f63c3b2cf227fbefdb53e3a3d09927ae9

Merge "Set EXTRA_USER instead of EXTRA_USER_ID for ACTION_SHOW_ADMIN_SUPPORT_DETAILS"

Merge "Clean-up removed RecoveryController API"

[automerger skipped] Merge "audioservice: fix a2dp connection race condition" am: e568805209  -s ours
am: 6466847321  -s ours

Change-Id: I583507cfc59c088fab3efa6f71998eb96169c903

Merge "[GraphicsEnvironment] Add whitelist support."

Merge "Stop using DIST_DIR directly, use dist-for-goals instead"

Merge "refactor radioPowerstate"

[automerger skipped] Merge "audioservice: fix a2dp connection race condition"
am: e568805209  -s ours

Change-Id: Ifbb5cc5a8422f73e2f2eb2236d3f2865e4553986

Merge "audioservice: fix a2dp connection race condition"

Merge "Do not bind IME until an IME client is recognized"

Merge "Minor Autofill improvements:"

Merge "iorap: Add presubmit/postsubmit support" am: 7cbabff3f0 am: ff3a8bf0f7
am: eaef91f7aa

Change-Id: Ifa811049b34a90ea607e8d18d423890217dd4b0c

Merge "Make SubscriptionManager.getSlotIndex public"

Merge "iorap: Add presubmit/postsubmit support" am: 7cbabff3f0
am: ff3a8bf0f7

Change-Id: Iee75da6b590be3d04c591acce53389ed420190b1

Merge "Stop waiting when elapsed time is equal to or greater than the timeout, rather than strictly greater than. This avoids a rare problem where the loop manages to complete at exactly the timeout time and then makes a call to wait(0), which waits indefinitely until another thread notifies it instead of timing out after 0 ms."

Merge "iorap: Add presubmit/postsubmit support"
am: 7cbabff3f0

Change-Id: I5f5afb05cde2894fb62728e67943a3dc38f26594

audioservice: fix a2dp connection race condition

In case of fast (< 1 second) A2DP disconnect/connect sequence, there
is a potential race condition where the delayed disconnection message
(due to becoming noisy intent) is received after the not delayed
connection message.

- Make sure all messages related to device connection/disconnection
(including MSG_A2DP_DEVICE_CONFIG_CHANGE and MSG_BTA2DP_DOCK_TIMEOUT)
are handled by queueMsgUnderWakeLock().
- Make sure messages are processed in the same order as API calls.
- Add check for null address in makeA2dpDeviceUnavailableNow().

Bug: 109903807
Bug: 78837311
Test: repro steps in bugs. A2DP connection and playback

Change-Id: Ib81b3805f945f7206b1a60de74e9bbeeef89bdd0
(cherry picked from commit 3c4636c7127cd34728050d051ef3bd62829e877d)
Merged-In: Ib81b3805f945f7206b1a60de74e9bbeeef89bdd0

Apply requirements when enabling provider

Test: Manually tested on device
Bug: 113087608
Change-Id: I508d3495aca38a45637b2d7d53f3fcf16a3a90c1
Fixes: 113087608

Merge "iorap: Add presubmit/postsubmit support"

iorap: Remove blank newline in TEST_MAPPING

Change-Id: I378e86327951daa05512efd2ac491d27f9995ddd

Added logging to debug the notification view hierarchy

Adding more logging to the systemUI tree to get useful
state to debug visual issues.

Test: grab bugreport
Bug: 80525283
Change-Id: Ia500dea632bc66f08c1ce350914ead033aba7383