Fix two shortcut manager issues

- isUserUnlocked check is still racy
We used a local copy of each user state in mUnlockedUsers, and updated
it in the service lifecycle events. However because
SystemService.onUnlockUser() is called on Handler, there was a brief
window between when the user was actually unlocked and the shortcut
manager thought the user was unlocked.
So now check with activity manager for the latest state too.  We still
check with the local copy first, because we want to consider STOPPING
as "unlocked".

- Messenger loses all bitmap icons.
Because we delay-save the shortcuts.xml file, if the device shut down
before we save the XML file but after removing the bitmap files,
we'd lose the bitmaps.

(Apparently SystemService.onCleanupUser() may not be called even when
a device is cleanly shutting down.)

So don't remove bitmap files synchronously, ever, and instead after
saving the XML just run the dangling file cleanup.

Bug 30784267
Bug 30730471

Change-Id: Ie58656efba2dca2b00582e145613bc56266a091e

Merge "Bind fingerprint when we start authentication" into nyc-mr1-dev

Merge "Put the affordance launch animation back on the renderthread" into nyc-mr1-dev

Merge "Rescan apps after restore and re-publish manifest shortcuts" into nyc-mr1-dev

Merge "Add missing null check to AppErrors#handleAppCrashLocked" into nyc-mr1-dev

Rescan apps after restore and re-publish manifest shortcuts

Originally when I wrote backup & restore for ShortcutManager,
there was no manifest shortcuts, so there was no need to
handle preintalled apps specially.

However, now we have manifest shortcuts, which are published
when the user is unlocked, by the time restore happens preinstalled
apps may already have manifest shortcuts, which will be overwritten
by the restored shortcuts.

So we need to re-publish manifest shortcuts after restore.

Bug 30746028

Change-Id: I6afbae7790c9ed38483637f33c381ecb6f854677

Merge "Fix navigation bar in seascape in BackdropFrameRenderer" into nyc-mr1-dev

Merge "Docs: Fixed the link that loads Fragments guide" into nyc-docs am: 5ca44906c6
am: b6962dee09

Change-Id: I5145db9e35199969ada9c38a6c5c972725af178b

Docs: Fixed the link that loads Fragments guide am: a8e1c75bf6
am: dabee01e9b

Change-Id: I6a17d73c3dab8980c57e9f6c9617bb465c48f1a5

Merge "Docs: Changed custom_toast to toast_layout" into nyc-docs am: a70c722c77
am: 3c49a4b761

Change-Id: Ib00803734137a0a4c3a704e45c45c66e246e586b

Docs: Changed custom_toast to toast_layout am: 3b7309d753
am: ca17f5b8e3

Change-Id: Ief6d2b58e26b4510b00ecb07a5efba180276cd37

Merge "Docs: Fixed the link that loads Fragments guide" into nyc-docs
am: 5ca44906c6

Change-Id: I4286b4a8e0766f7c3ca2d7e5ebd5c9f2b296fbb9

Docs: Fixed the link that loads Fragments guide
am: a8e1c75bf6

Change-Id: I26ea32ff32067d86fb5390103eec737b293ac031

Merge "Docs: Changed custom_toast to toast_layout" into nyc-docs
am: a70c722c77

Change-Id: I6e1d0d70d5d4b89b083cb74272f064bebf6a273a

Docs: Changed custom_toast to toast_layout
am: 3b7309d753

Change-Id: I3d85fb321e27643227c568127870c7ad07650c41

Merge "Docs: Fixed the link that loads Fragments guide" into nyc-docs

Merge "Docs: Changed custom_toast to toast_layout" into nyc-docs

Add missing null check to AppErrors#handleAppCrashLocked

BUG: 30692618
Change-Id: Ibe3589e1248520067714d5a20963a17935789066
(cherry picked from commit dc74142c74bf37e0ddcbe6ec7d76f57191815b44)

Merge "Fix @link tags in keystore doc" into nyc-docs am: 3188f77658
am: 7218af3f9a

Change-Id: I302b1dce1cff57b6f1cd5da3d6619984c80fde92

Fix @link tags in keystore doc am: 3f0526ea09
am: 9e8b4ad9c9

Change-Id: Iadb16bd13d4bb525805dd0266aea046c4c472309

Merge "Fix @link tags in keystore doc" into nyc-docs
am: 3188f77658

Change-Id: I4d30c3ad9f1d2d9d949e5f1ebc680f65c6925b99

Fix @link tags in keystore doc
am: 3f0526ea09

Change-Id: I3162cbf57341ab231b84544c04e02d2062dbd304

Merge "Clean up zen rules and permissions immediately on uninstall." into nyc-mr1-dev

Merge "Fix @link tags in keystore doc" into nyc-docs

Merge "Limit number of recents entries for an app to maxRecents allowed." into nyc-mr1-dev

Merge "BootAnimation system property to play sound." into nyc-mr1-dev

Clean up zen rules and permissions immediately on uninstall.

Change-Id: I8d03f4a4347b7d7bffbe267961c75ed5863a4113
Fixes: 30740759

Limit number of recents entries for an app to maxRecents allowed.

Regression in the API caused by the fixes for b/28293748 and b/25460953
which prevented the task entries from been trimmed if the task don't
have the same intent filter (additional documents launched will normally
have different intent action/filter than the base activity) or it allows
multiple task entries.
We now trim when we go over the maxRecents allowed in both cases.

Bug: 30645949
Bug: 28293748
Bug: 25460953
Change-Id: I74c8ecc38f72a5d03c213dea8d3e62261d42f8f6

Fix navigation bar in seascape in BackdropFrameRenderer

Change-Id: I922aaf835011b236f4b4d34f249304e335c3318b
Fixes: 30670517

Bind fingerprint when we start authentication

This fixes a bug where it was possible to authenticate the wrong user.
We now bind the userId when we start authentication and confirm it when
authentication completes.

Fixes bug 30744668

Change-Id: I346d92c301414ed81e11fa9c171584c7ae4341c2

Merge "Handle empty/unspecified crop properly in wallpaper restore" into nyc-mr1-dev

Put the affordance launch animation back on the renderthread

Change-Id: I10f6b7062d9c6ca89e139ef6b67af30cb39adcb2
Fixes: 30484027

Merge "Mark wallpaper as backupInForeground="true"" into nyc-mr1-dev

Merge "Merge "docs: Revised "Optimizing Content for the Assistant" page." into nyc-dev am: 87c2ed97b8" into nyc-mr1-dev

Merge "docs: Revised "Optimizing Content for the Assistant" page. am: d4197234bb" into nyc-mr1-dev

Merge "docs: Revised "Optimizing Content for the Assistant" page." into nyc-dev
am: 87c2ed97b8

Change-Id: I48f91ef763f6591e16751d9ae8fd9d86be277111

docs: Revised "Optimizing Content for the Assistant" page.
am: d4197234bb

Change-Id: Ie85273ed977c06e9b82a77d66a7333e9eca86e10

Merge "Prevent EndLayerOps when Begin was rejected" into nyc-mr1-dev

Docs: Fixed the link that loads Fragments guide

In the class description, fixed the reference link that points to the
Fragments guide.

Bug: 29007808
Change-Id: I3399e5f10c461eb6666a04f8829c6cde774948e0

Merge "docs: Revised "Optimizing Content for the Assistant" page." into nyc-dev

Handle empty/unspecified crop properly in wallpaper restore

Pass 'null' as the crop hint when the crop as expressed in the restored
wallpaper metadata is either unspecified or effectively empty.

Bug 30521402
Bug 30274136

Change-Id: I14e5d2bae1ec30fb27e8fd45b340b2ca87f35a01

Merge "ShortcutManager improve app udpate check" into nyc-mr1-dev

docs: fixed generics in data binding code samples am: b1b5efd3be
am: 7f703bde72

Change-Id: Ibc12639849de4edcbcfdaa1c0f543d6f2eb868f8

Merge "Fix bug where foreground activity fails to cancel auth/enrollment" into nyc-mr1-dev

Merge "docs: add info to SharedPreferences.Editor.putString" into nyc-dev
am: a6ee9c1848

Change-Id: Ief38c0b6b1b61e8fcc060224c298fb3a4711cc94

docs: add info to SharedPreferences.Editor.putString
am: fb48521f41

Change-Id: I948c8091a70180e29a601d6cba40d7fb0cf2d76a

docs: fixed generics in data binding code samples
am: b1b5efd3be

Change-Id: Iecafea06866a4579824435467c5dd31859c0dbd4

Merge "Reverting ag/1288123" into nyc-mr1-dev

Merge "docs: add info to SharedPreferences.Editor.putString" into nyc-dev

Prevent EndLayerOps when Begin was rejected

bug:30537130

BeginLayerOps were being rejected in a way that allowed the associated
EndLayerOps to still be recorded. This was a violation of DisplayList
content expectations, and caused crashes in FrameBuilder when trying to
play these DisplayLists back.

Change-Id: I531b840aa5c4ffb1ee458da3f4b366978eaeafbe

Merge "Fix currentAnimationTimeMillis to use vsync time" into nyc-mr1-dev

ShortcutManager improve app udpate check

- Don't use lastUpdateTime for system apps since it's not reliable.

- Scan downgraded apps too.

Bug 30708050
Bug 30734178
Change-Id: I98253f4c635466197548385275ab08c5e3a1a10b

Merge "Fix infinite AVD not running" into nyc-mr1-dev

BootAnimation system property to play sound.

Optional system property "persist.sys.bootanim.play_sound"
If nonexistent, default behavior plays the sound.

Bug: 30690353
Change-Id: Ie5ed456d891632be5ec8be255ac7c1193753c741

Merge "Restore padding." into nyc-mr1-dev

Merge "Have BatteryStats track and report the running wakelocks." into nyc-mr1-dev

Fix infinite AVD not running

This CL fixed an integer overflow error caused by using integer literal
when it should be a float literal that defines a double.

BUG:30739855
Change-Id: If2bdb0eb7a2e93d28434bfe72ec11e8f1f2a1eda

Restore padding.

Change-Id: I532e5007c2a9152554b8f5130196a2f0cb3c8f1e
Fixes: 30738806

Merge changes Idc4c2c14,Ibea016c6,I49ab58c3,Id911d405,I0f17dc5f, ... into nyc-mr1-dev

* changes:
  Improved the transition when clicking on the last notification
  Fixed a bug that could lead to an empty notification
  Removed contentDescriptions that hindered accessibility usability
  Fixes a bug where setting a null bigLargeIcon would not clear it
  Fixed the transition of the background of the notifications
  Fixed a bug where the notification would fade out in a ugly way

Merge "Don't ensure configuration for activity we are resuming." into nyc-mr1-dev

Merge "Prevents screenshot layer from dismissing IME." into nyc-mr1-dev

Don't ensure configuration for activity we are resuming.

Regression introduced in ag/865988 when ASS.ensureActivitiesVisibleLocked()
was added to the end of AS.completePauseLocked() without specifing the
starting/resuming activity. If the starting/resuming activity isn't specified
the resuming activity can be relaunched with the current configuration which
might be different from the configuration the activity will end up in if it has
fixed orientation.
We now pass in the starting/resuming activity to avoid this.

Bug: 30509213
Change-Id: Idd0be84745bad83a1924e850b9b4020b927a78d8

Merge "QS: Tweak animation paths when few icons" into nyc-mr1-dev

Merge "QS: Fix expanding on closing detail" into nyc-mr1-dev

Merge "Ensure Dequeue/Queue duration is always set" into nyc-mr1-dev

Merge "Import translations. DO NOT MERGE" into nyc-mr1-dev

Merge "Import translations. DO NOT MERGE" into nyc-mr1-dev

Import translations. DO NOT MERGE

Change-Id: Ia177ed731331dbc6420eafe17a1615a7bc65dac2
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: Ib195a4f4c102694bc911d6987dc1506f511cadd4
Auto-generated-cl: translation import

Import translations. DO NOT MERGE

Change-Id: I6b7b641340466819a50ee8838278b352948891f9
Auto-generated-cl: translation import

Merge "Add Connection Event for wifi handover failures." into nyc-mr1-dev

Prevents screenshot layer from dismissing IME.

Add code to adjustWindowLw to force the flag
FLAG_NOT_FOCUSABLE to be set for layer type
TYPE_SCREENSHOT.

Bug: 30485483
Change-Id: I11725eb89fda59b6d50fa1700845cdfe9ffb930b

Merge "Make sure IME focus is synced to View focus" into nyc-mr1-dev

Fix bug where foreground activity fails to cancel auth/enrollment

This fixes a bug where the currently authenticating application
can't cancel itself in onPause(). The problem is that some applications
can set FLAG_RESUME_WHILE_PAUSING (like the home screen), which changes
the life cycle of IMPORTANCE_FOREGROUND to be removed before onPause()
completes.  The result is the currently authenticating activity
can't cancel fingerprint authentication or enrollment.

The fix is to allow apps to always cancel themselves.

Fixes bug 30380234

Change-Id: If65c2028014da8d5129506d0b45cc6c1e25f7860

Merge "Remove animation value change from push staging" into nyc-mr1-dev

Merge "Support repeatMode = reverse in AVD" into nyc-mr1-dev

Merge "Improve handling of crashing a11y services." into nyc-mr1-dev

Have BatteryStats track and report the running wakelocks.

The times reported are in the screen off / on battery time base.

Bug: 29546924
Change-Id: I025b2959becb256a26cf531e944786d4e9a59149

Merge "Reschedule date-change alarm whenever the time is set" into nyc-mr1-dev

Make sure IME focus is synced to View focus

This CL fixes an edge case that my previous CL [1] forgot to handle.

The goal of my previous CL was to avoid InputMethodManager from getting
confused by a false focus-in event from temporarily detached Views.
However, my CL forgot to take care of the case where the temporarily
detached View is still focused even after the temporary detach mode is
done.

The bad news is that such a situation is relatively easy to trigger by
having a ListView that has EditText as follows, which seems to be
known to be a common technique in Android developer community to put an
EditText in a ListView.
  ListView#listView.addHeaderView(new EditText(context), null, true);

If the ListView is initialized as above, and the EditText has input
focus, View focus and IME focus start to disagree immediatelly after the
ListView's layout is re-evaluated.  This is really easy to trigger, for
example just by dismissing the IME window.

In summary, the root cause is that InputMethodManager#focusIn(View) is
now always ignored as long as the View is temporarily detached, under an
assumption that IMM#focusIn(View) will be called back again with a View
that is not temporarily detached when everything is stable.  Hence the
fix is to do so by hooking up View#dispatchFinishTemporaryDetach() to
call IMM#focusIn(View) again when the View is actually focused in the
final state.

 [1]: Ia79bbd8468f768d546354382b47b39dd31ef7bb5
      a4ed0cfcb6885beeb52f701bfc64c393b668f7ba

Bug: 30022872
Bug: 30578745
Bug: 30706985
Change-Id: Iecbdb00dcef8c72e4f7b31035c9bf0f4a40a578f

Merge "Not trying to remove admins already being removed" into nyc-mr1-dev

Improved the transition when clicking on the last notification

In the locked shade, we were returning to the lockscreen when
the last notification was clicked and removed due to autocancel.
We're now not going back there anymore if we're closing.

Change-Id: Idc4c2c145fb2ca308f236b99408cc6096dd02302
Fixes: 30642902

Reverting ag/1288123

- Reverts the change to draw the drop targets opposite of the nav bar
- Fixes the issue with the drop divider being occluded by the nav bar
  when the phone is in landscape orientation.  Instead, expand the
  drag overlay to include the nav bar space.

Bug: 30548794
Change-Id: I70ed3513547e101e49c8eba114a153c5a1f36bee

Reschedule date-change alarm whenever the time is set

Bug 30628519

Change-Id: Ic4180b9f3c822f3dfb7e35199f3d058938acf778

Merge "Switch volume animation to Scene." into nyc-mr1-dev

Merge "Fix WM missing stack/task when activity is force stopped and restarted" into nyc-mr1-dev

Merge "Correctly initialize rotation animation hint." into nyc-mr1-dev

Switch volume animation to Scene.

animateLayoutChanges is too janky.

Bug: 30070005
Change-Id: I88bb94c2de18a8ebc14f0f0aed48877a8b328d8a

Fix WM missing stack/task when activity is force stopped and restarted

When activity process is force-stopped (like in the case we need to stop
it to allow debugger), activity record is removed and a new instance is
added. Depending on timing, adding of new record could happen before
or after old record is removed. If it added after old record is removed,
then the removal could cause AM to also remove the task AND the stack
(if the activity is the last one).

On WM side however, the task/stack removal could be delayed due to
animation. So when AM attaches a new stack, we end up with two stacks
with the same stack id in the DisplayContent. Also, AM reuses the last
used taskId after it's deleted. So WM is confused and didn't add a new
task record. Then when the delayed stack removal fires everything is gone.

We need to use a different task id when creating a new task record, even
if the last created task is already deleted. Also when attaching a new
stack to WM, first check if we already have such a stack in DisplayContent,
if so, reattach it and don't create a duplicate.

bug: 30465601
Change-Id: Ic399a32c54f6acdcbb9a0c6155599331d55df232

Fixed a bug that could lead to an empty notification

With sensitive notifications, notification groups could
become empty.

Change-Id: Ibea016c64287d1952db136e038372b778d8febc2
Fixes: 30641080

Merge changes from topic 'smooth_unlock' into nyc-mr1-dev

* changes:
  Keyguard: Improve trusted unlock while occluded
  Keyguard: Refactoring for improving trusted unlock while occluded

Add Connection Event for wifi handover failures.

Adding a new connection event which Telephony can use to inform upper
layers of a failure to handover to WIFI.

Bug: 30697460
Change-Id: I75da2f956a6b70a9bf4cf7128f29968e08172dc3

Merge "Fix "adb backup -shared"" into nyc-mr1-dev

Correctly initialize rotation animation hint.

In the case of no activity options we won't initialize
it again later, and so we will override the apps request
with a hint of 0 (ROTATION_ANIMATION_ROTATE).

Bug: 30683604
Change-Id: I5eed12adfbb241859503bcd40f15ee82141436d8

Ensure Dequeue/Queue duration is always set

Bug: 30662749
Change-Id: I35a4c26ce78536fc7a506ec2f92a8b9eecb12ec2

Merge "Move EphemeralResolverService" into nyc-mr1-dev

docs: fixed generics in data binding code samples

Generic types, such as List<String>, must be escaped when they are
used in XML. For example, List&lt;String>. This commit adds
escape characters to the XML samples in the data binding page.

Bug: 30691285
Change-Id: If55ed55d13fb11038e2495884c79698e67219108

Merge "WindowManager: Restore update behavior of mLastFrame." into nyc-mr1-dev

Fix currentAnimationTimeMillis to use vsync time

Bug: 30674719
Change-Id: I71f51eea971e43cbe85bb0713a1c457a1b3243b3

Merge "QS: Fix accidental a11y tile removal" into nyc-mr1-dev