Fix duplicate calls of IMS#doStartInput()

This is another follow up CL to my previous CL [1], which had a bug
that InputMethodService#doStartInput() is now always called back twice.

There was a careless mistake when rebasing the CL [1] onto the previous
CL [2], which ended up with dispatching startInput() twice in
IInputMethodWrapper.

In short, InputMethod#dispatchStartInputWithToken() is the new way to
dispatch startInput()/restartInput(). We do not need to call both of
them.

 [1]: I9921b381e02106dbffff5e0b3d13f0a1245ce807
      6db3bfe33d92127d203ec872a0b353585a99f256
 [2]: I476d0cf8cbb0a0134941854f9337d9ad15e66a71
      f7526b58960608887b064670bb42f41aa28b8f41

Test: Set true to InputMethodService#DEBUG and make sure startInput()
      and restartInput() are called in the following scenario.
      1. Complete the setup wizard.
      2. adb shell am start -a android.app.action.SET_NEW_PASSWORD
      3. Proceed to "Choose your password" page
      4. Make sure startInput() gets called only once.
      5. Type "aaaa" then hit "CONTINUE" button.
      6. Make sure restartInput() gets called only once.
Bug: 35079353
Fixes: 35442939
Change-Id: Ifde16c3fad5b787d4c9a11bd886943dfe96f9eb9

Merge changes Ia33ab1ab,I1d01ab5d

* changes:
  AOD: Add toggle for waking up instead of ambient display on sensors
  AOD: Refactor DozeMachine to allow waking up

Merge "Removing PiP from SysUI tuner."

Merge "Add extra key for app-specified outgoing call timeout" into nyc-mr2-dev am: 4d3435637f  -s ours
am: dbf6f9263d  -s ours

Change-Id: I403da9dbf4c9cd9f1e9a69f7152bba4832b734e7

Merge "Ensuring that there are per-user last stack active times."

Merge "Merge "Harden against jobscheduler client app misbehavior" into nyc-mr2-dev am: 38f1b6dbc9 am: c475730d22"

Merge "Add escrow token API in DevicePolicyManager."

Merge "Harden against jobscheduler client app misbehavior" into nyc-mr2-dev am: 38f1b6dbc9
am: c475730d22

Change-Id: I026c74906f8f43c2eb7ca529a59b1e6be5696351

Merge "Use SystemConfig to control background location throttling whitelist."

Merge "Add extra key for app-specified outgoing call timeout" into nyc-mr2-dev
am: 4d3435637f  -s ours

Change-Id: I2ed7250a9fb8c5026b209ad5b3384a401b1d4d16

Merge "Access IMMS#mSettings only inside the lock"

Merge "Harden against jobscheduler client app misbehavior" into nyc-mr2-dev
am: 38f1b6dbc9

Change-Id: I44d16150abee9dd0ec7961d1ce847d9793937354

Merge "Add extra key for app-specified outgoing call timeout" into nyc-mr2-dev

Merge "Various QS fixes and cleanup"

Merge "ShortcutManager should support MaskableBitmap Icon type"

Merge changes from topic 'settings_advanced_expand_button'

* changes:
  Add logging for advanced button expand action
  Add FIELD_CONTEXT to track source of action in eventlog.

ShortcutManager should support MaskableBitmap Icon type

Note: https://ag/1872683/ is where initial code review happened.

Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest1 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest2 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest3 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest4 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest5 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest6 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest7 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest8 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest9 -w com.android.frameworks.servicestests
Test: adb shell am instrument -e class com.android.server.pm.ShortcutManagerTest10 -w com.android.frameworks.servicestests

Test: cts-tradefed run cts --skip-device-info --skip-preconditions --skip-system-status-check com.android.compatibility.common.tradefed.targetprep.NetworkConnectivityChecker -a armeabi-v7a -m CtsShortcutHostTestCases -t 'android.content.pm.cts.shortcuthost.ShortcutManagerMultiuserTest'
Test: cts-tradefed run cts --skip-device-info --skip-preconditions --skip-system-status-check com.android.compatibility.common.tradefed.targetprep.NetworkConnectivityChecker -a armeabi-v7a -m CtsDevicePolicyManagerTestCases -t 'com.android.cts.devicepolicy.LauncherAppsProfileTest'

b/34196580

Change-Id: I3b983c29387e474c7a8c687a524105ceaceac81e

Merge "Use correct packageName for checking if it belongs to callingUid."

Merge "Harden against jobscheduler client app misbehavior" into nyc-mr2-dev

Merge "Add functionality to overwrite the /etc/preloaded-classes file."

Removing PiP from SysUI tuner.

Bug: 35143109
Test: Go to tuner, observe
Change-Id: I5c4eadc33a303e7bf7d99aa772296405d39ee251

Merge "Fail safely on invalid ScanResults when creating NetworkKey"

AOD: Add toggle for waking up instead of ambient display on sensors

Bug: 30876804
Test: Open SysUI tuner -> Ambient Display -> Toggle wake up on sensors setting, verify that double tap and lift wake up phone instead of triggering ambient display.
Change-Id: Ia33ab1ab5ed532f239262d70e00dac3fec0330e2

AOD: Refactor DozeMachine to allow waking up

Bug: 30876804
Test: runtest -x $ANDROID_BUILD_TOP/frameworks/base/packages/SystemUI/tests/src/com/android/systemui/doze/DozeMachineTest.java
Change-Id: I1d01ab5d8ba3c1bf7b11b0b4911b25f3416ca6e9

Merge "Skip class path checking for secondary dex files"

Merge "AOD: Show fingerprint error messages"

Merge "Apply minimized offset when fetching new aspect ratio bounds."

Merge "New setDataSource API for accepting cookies"

Merge "Add extra key for app-specified outgoing call timeout" am: decc6c9d56 am: 2bc24e252e am: 38f5001c10
am: b521143aa1

Change-Id: I416d7ad1a12a6b8ddb2ea2958356a487e7f10aef

Various QS fixes and cleanup

Test: runtest systemui
Change-Id: If87f58f2af00b23c5e1a8e38a63a6a63d903002c

Merge "Add escrow token support to synthetic password flow"

Add extra key for app-specified outgoing call timeout

Bug: 34474757
Test: manual
Merged-In: I6e7e67c946205b28440a4daeeab3a002f8d6bccc
Change-Id: Ia93d68d6a01308b0b93b88977daf983a0ed3a22c

Merge "Add synthetic password to authentication flow"

Merge "Add extra key for app-specified outgoing call timeout" am: decc6c9d56 am: 2bc24e252e
am: 38f5001c10

Change-Id: I6e7e67c946205b28440a4daeeab3a002f8d6bccc

Merge "Add extra key for app-specified outgoing call timeout" am: decc6c9d56
am: 2bc24e252e

Change-Id: I1912bfa8856e78f375587d94bf589ef2fd25bee8

Merge "Add extra key for app-specified outgoing call timeout"
am: decc6c9d56

Change-Id: I58a70b265f4b5af735e93c9004616bd44d5c51de

Merge "AOD: Show High notifications on AOD"

Merge changes from topic 'dual_role_chargers' am: 37b939a180 am: 0a3efc2622 am: d4a847495e
am: ac0bb1f7b2

Change-Id: I9647bbeea800b9e7d73748474f8c5bb83620df39

Merge "Revert "remove app-ephemeral""

Merge "Revert "Per user setting for instant app""

Ensuring that there are per-user last stack active times.

- This fixes an issue where the last stack active time would be clobbered
  when switching between users.  With the policy in the phone/stack
  recents, this is fine, but with the grid recents, it no longer only
  applies when out of the historical window, so it is always wrong (it
  would normally be wrong if switching back from another user after the
  historical time of six hours).

  This CL will migrate the last stack active time to a per-user secure
  setting, which will be used going forward.

Bug: 35375206
Test: On the Ryu, launch some tasks, switch users, launch more tasks, and
      return to the original user

Change-Id: I9941526de5d1dd52d1f9003e795995389064b19d

Revert "Per user setting for instant app"

Bug: 35390781

This reverts commit 2f5811dcfd840e149851a9333e27ef3cdddf7a46.

Change-Id: Ibb1c8dacbdc6908fc7fa2bc5dca664f2455162bf

Revert "remove app-ephemeral"

Bug: 35390781

This reverts commit 4eb7cea562cd117e598a327ab4d93142589c1ae6.

Change-Id: I94567d333b40f99f7a2573125259f4ab575a0519

Use SystemConfig to control background location throttling whitelist.

Test: manual

Change-Id: I51285f9a463381855f2bb3fa9af34af1930b8ebd

Merge "Add extra key for app-specified outgoing call timeout"

Merge changes from topic 'revert-anr-revert'

* changes:
  Process queuedWork on main thread and add logging
  Revert accidential merge 492a058d58e991445996cb8cc8dcb24494ac20c5

Add logging for advanced button expand action

Change-Id: I648984686e2dda053f3d0648954c252edd805342
Fix: 35356464
Test: Builds

Add FIELD_CONTEXT to track source of action in eventlog.

Change-Id: Id32aaab4b4445e5db318d2b48d198b2a815b7cd4
Fix: 35359289
Bug: 35356464
Test: builds

Merge "Fixed that an icon could become invisible"

Merge "New system for versioning sysui plugins"

Merge "Add logs, reorder bugreport."

Merge changes from topic 'dual_role_chargers' am: 37b939a180 am: 0a3efc2622
am: d4a847495e

Change-Id: Icadc77cb7630e61166ab3c9608f60736c24e52c1

Merge "Tuner updates"

Merge changes from topic 'dual_role_chargers' am: 37b939a180
am: 0a3efc2622

Change-Id: I2c0ba36c46674ff7f9a0b963543e271eec93858c

Merge changes from topic 'dual_role_chargers'
am: 37b939a180

Change-Id: I2b9eda605ae85347eb2e3e4202a6c7ec176778af

Merge "Made the notificationViewStates static"

Merge "Fixed a few coloring issues"

Merge "Fix test failures on treehugger?"

Merge changes from topic 'dual_role_chargers'

* changes:
  Introduce debounce to filter rapidly toggling type-c ports
  Fix handling usb dialog for wall chargers.

Add escrow token API in DevicePolicyManager.

Take advantage of the new authentication flow in LockSettingsService
and allow PO or DO to provision escrow tokens on the device. The
escrow token grants them the ability to change device lockscreen
(if used by DO) or work profile challenge (if used by PO). The
new password reset mechanism is even usable before user unlocks,
and it preserves authentication-bound keys in keystore.

Test: runtest frameworks-services -c com.android.server.SyntheticPasswordTests
Test: runtest frameworks-services -c com.android.server.devicepolicy.DevicePolicyManagerTest
Test: cts-tradefed run cts-dev -m CtsDevicePolicyManagerTestCases -t com.android.cts.devicepolicy.MixedDeviceOwnerTest#testResetPasswordWithToken
Bug: 33126620
Change-Id: Iaa684c51946f726cbd909e9ac70ad3e9ca3de1ac

Add escrow token support to synthetic password flow

Escrow token provides an alternative way to derive synthetic password for a
given user. In the new flow, a pre-provisioned escrow token
should be able to do anything the user password can do, since they both
derives the synthetici password which is the master key in the new auth flow.

Test: runtest frameworks-services -c com.android.server.SyntheticPasswordTests
Bug: 33126414
Change-Id: Ib5ee38fd61f66de3245427ce992ebc12f1873a26

Fixed that an icon could become invisible

A icon could be invisible because we were not aborting the animation
in certain cases. This should be fixed now.

Change-Id: I8caf35034704a0df3d205205086c4622b60e0da9
Fixes: 35385932
Test: runtest systemui

Add synthetic password to authentication flow

The user password is used to unlock a per-user synthetic password which
serves the purpose of what the user password previsouly achieves (protect
keystore, vold disk encryption, auth token generation).

Test: runtest frameworks-services -c com.android.server.SyntheticPasswordTests
Test: manual
    1. Start with fresh device, enable synthetic password with "adb shell cmd lock_settings sp 1"
        1.1 add device lock, reboot and verify (positive & negative); change device lock, reboot and verify.
        1.2 Inflate a work profile, reboot and verify device lock. check SID with "adb shell dumpsys lock_settings"
        1.3 Un-unify and add work challenge, reboot and verify work challenge and SID.
        1.4 Re-unify work challenge, reboot and verify.
        1.5 Clear device lock, reboot and verify lock and SID.

    2. Start with a fresh device, add a device lock and inflate a work profile.
        2.1 Enable synthetic password, note current SID
        2.2 Reboot and unlock device. Verify synthetic password is generated and SID remains.
        2.3 Clear device lock, reboot and verify (SID should be cleared)

    3. Start with a fresh device, inflate a work profile, add separate work challenge
        3.1 Enable synthetic password, not current SID
        3.2 Reboot and unlock device and profile. Verify synthetic password is generated.
        3.3 Clear device lock only, reboot and verify (work profile SID should remain)

    All steps tested on marlin (FBE) and bullhead (FDE)

Bug: 33126414
Change-Id: Idb9ebfc7bba2fe40670c5fee2189e873d9704540

Merge "remove app-ephemeral"

Process queuedWork on main thread and add logging

- The queued work processing thread might be sleeping while the main
  thread is waiting for it to do work. Hence process the work in the main
  thread.
- Carefully add logging so that slowness can be tracked.
- Fix usage of the wrong lock (sWork instead of sLock).
- Increase the time of the delay between apply and write to make
  possible side-effects more visible

Test: SharedPrefencesTest, looked at logging
Bug: 30662828
Change-Id: Ie8a5d531e180dacec29c947ba0b59b170facf782

Merge "Per user setting for instant app"

Merge "TIF: Add browsable field to programs"

Merge "Always flush renderstate in DeferredLayerUpdater::apply"

Merge "updateTexImage before bitmap readback of DeferredLayerUpdaters"

Fix test failures on treehugger?

Test: runtest systemui
Change-Id: I0660729aade05cd63b240eed13ed9d5c9ff8de91

Tuner updates

Some UX updates to the tuner to feel better and align with settings
UX patterns.

Test: visual
Change-Id: I9c57194b9649c61e0d6d3f9e21a51d9c4a5c1286

Introduce debounce to filter rapidly toggling type-c ports

Type-c ports can quickly toggle between connected/disconnected
states. Introduce debounce to prevent sending spurious notifications.

Bug: 34972898
Test: notification should not be queued for a pixel-c charger not connected
to the power outlet.
Change-Id: I4aa19f9f864fe5b77e65f6a07a3184d8aba1f5fc

Fix handling usb dialog for wall chargers.

UsbPort.POWER_ROLE_SINK is orthogonal to the type of the charger
attached. POWER_ROLE_SINK would be the case for AC charging and
USB charging. Therefore query BatteryManager for the charger
type.

Bug: 34972898
Test: Charging notification should not show for pixel-c chargers.
Change-Id: I8dddcd7727b6af973bd173d2c6e325aa4be2ca3a

Merge "Fix for cancel to remove both posted and enqueued notifications"

New setDataSource API for accepting cookies

Bug:    34736056
Test:   Manual through the test app
Change-Id: Ibd48d5e292dda490d9e4e4528589b2b7ba97a4b4

updateTexImage before bitmap readback of DeferredLayerUpdaters

Bug: 34206215
Test: hwui unit tests passing

Change-Id: I4360c0d94a831b4d2ebfd4020e4241658bfa190a

Always flush renderstate in DeferredLayerUpdater::apply

Bug: 34206215
Test: hwui unit tests passing, manual test of TextureView video playback

Always flush renderstate to the GlLayer's texture, regardless of
whether updateTexImage has ever been called.

Change-Id: I3974dce9d90633a0299e6bc4259b76c622717c90

Apply minimized offset when fetching new aspect ratio bounds.

Bug: 35091993
Test: Minimize a PiP activity, and then force it to change aspect ratio
Change-Id: I0edafcdefcb68224ca6b476b13819847b6b932d9

Fix for cancel to remove both posted and enqueued notifications

Bug:34287447
Test: runtest systemui-notification
Change-Id: I7deb3a06416a6eb1b8421273f7dd5115d9b7a05c

Add logs, reorder bugreport.

Bug: 32986903
Bug: 34227881
Test: cts hostside
Change-Id: Ic5dfe4be5632c6bd7c70115c6a60b060d23be9bd

remove app-ephemeral

instant apps are no longer stored in a separate folder. they
are now stored along side other full apps in the apps directory.

Bug: 25119046
Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.EphemeralTest
Change-Id: I6669be797987169a0b9cca78f80539c908812f9e

Merge "Zygote: Add support for explicit preloading of resources."

Merge "TextView/LinkMovementMethod/ClickableSpan - touch up revert"

Merge "Revert "Use context's userId in ContentResolver class.""

Revert "Use context's userId in ContentResolver class."

This reverts commit 2b44387f5d706e477418795c2d57d0909dc7ed22.

Bug: 35374151
Change-Id: Ifdde7a3b4e9141d7e283478aff49971c6a5dcaa2

Merge "Add dynamic font support"

Zygote: Add support for explicit preloading of resources.

Add a --preload-default command that instructs the zygote to preload
resources. The command is a no-op if resources have already been
preloaded.

Test: manual.
Change-Id: I4a846a7d911fa929af472d9071ffbff6df424176

Merge "Zygote: Reset niceness if lazy preloading is enabled."

Merge "Remove color attribute from MaskableIconDrawable foreground/background tag Test: manual test"

Merge "Ensure we set the notification marker position by calling updateMarker() just after the AudioTrack is initialized. This solve the issue with cached synthesis in which updateMarker was called before AudioTracker initialization and we never got the callbacks."

Merge "Enable auto-fill when activity is marked as FLAG_SECURE."

Access IMMS#mSettings only inside the lock

This is a preparation CL before addresssing Bug 35361128.

IMMS.MyPackageMonitor#isChangingPackagesOfCurrentUser() needs to be
called after taking a lock for IMMS#mMethodMap because it internally
relies on InputMethodManagerService#mSettings.

Test: Made sure that the device can boot and the IME is available.
Bug: 35361128
Change-Id: I66752a275f20f6ba33bc211ffb8d6fd269250c0d

Made the notificationViewStates static

We shouldn't hold an implicit reference to the
row, let's fix that.

Bug: 35345204
Test: runtest systemui
Change-Id: Ifa0ba0bd07850ae575059b6368791d60d1091616

Fail safely on invalid ScanResults when creating NetworkKey

Added tests for NetworkKey#createFromScanResult

Bug: 34671341
Test: runtest --path
frameworks/base/core/tests/coretests/src/android/net/NetworkKeyTest.java
runtest --path
frameworks/base/services/tests/servicestests/src/com/android/server/NetworkScoreServiceTest.java

Change-Id: Ideac1615917649cd3934421228a82aeed37e7ad1

Use correct packageName for checking if it belongs to callingUid.

In AccountManagerService.getAccountsAsUser, we check if opPackageName
belongs to calling uid by calling AppOpsManager.checkPackage. But when
AccountManagerService.getAccountsAsUser is called from
AccountManagerService.addSharedAccountsFromParentUser, we're using the
opPackageName from system context instead of calling context.

Bug: 35258008
Test: cts-tradefed run singleCommand cts-dev --module CtsMultiUserHostTestCases \
      -t android.host.multiuser.CreateUsersPermissionTest#testCanCreateRestrictedUser
Change-Id: I5c425d9314beb86f7c64a5b5c64b7d879711879a

Add functionality to overwrite the /etc/preloaded-classes file.

* Requires root, disable-verity, reboot, root before use.

Test: manual through UI and sequencing
Change-Id: I68965334776e130b8220a5814b2525109cf96800

Skip class path checking for secondary dex files

Pass '&' marker to dex2oat classpath when compiling secondary dex files.
This will skip class path checking when loading the oat files.

Test: adb shell cmd package compile -m speed -f --secondary-dex com.google.android.gms
      oatdump --header-only --oat-
file=/data/user/0/com.google.android.gms/app_chimera/m/00000006/oat/arm64/DynamiteModulesC_GmsCore_prodmnc_alldpi_release
      check that the class path contains the special '&' marker

Bug: 32871170
Change-Id: If0628aeb4e3f5717604bfc4a87a4b1d438e5fa65

Enable auto-fill when activity is marked as FLAG_SECURE.

BUG: 34077687
Test: CtsAutoFillServiceTestCases passes
Change-Id: I2d7a565a53a32792d6daeceba3f3a7beebdf052c

Merge "audio: deprecate AudioManager.isBluetoothA2dpOn()"

Add extra key for app-specified outgoing call timeout

Bug: 34474757
Test: manual
Change-Id: Ia93d68d6a01308b0b93b88977daf983a0ed3a22c