DO NOT MERGE Fixing crash in SystemUI fetching task description icons.

When we compose the TaskDescription for the task, we update it with
information from each activity from top down.  If an activity has a
saved icon, the filepath for that icon will be encoded with the user
id of the activity and saved into the task description.  However, if
that activity's user is different from the user id of the root
activity of the task, we will later try to fetch the icon with the
user id of the task which will cause an exception to be thrown.

Instead, because the task description describes the task and information
belonging to the task description is always fetched as the user of
that task, we can save the icon with the task's user encoded instead
of the activity's.

Bug: 34692261
Test: Launch primary user activity with a custom task description icon
      in a task that is rooted with a work profile activity, then restart
      before the primary user activity is finished.

Change-Id: I66240cd488271e677cce7923cfef8e3e0cea5621

Merge changes from topic 'usb_bugfix' into nyc-mr2-dev

* changes:
  Refactored setCurrentFunction and setUsbDataUnlocked into single method.
  Clean up persistent usb state on boot.
  Fixed handling of usb state during adb changes.

Merge "Decrease frequency of high temp warnings" into nyc-mr2-dev

Decrease frequency of high temp warnings

Instead of showing any time the temp is above throttling temp,
if the user has dismissed the notification (or clicked it), the notification
will not reshow until the temperature has gone below the throttling
temp and come back.

Test: manual
Change-Id: I4cef9144dfcff03a5e70ccd5b12efaab1e74d15b
Fixes: 34346625

Merge "MessagingStyle: Fix buggy measure in MessagingLinearLayout" into nyc-mr2-dev

Merge "Make SpannableStringBuilder.getSpans thread-safe" into nyc-mr2-dev

Merge "Fix SpannableStringBuilder getSpans sort" into nyc-mr2-dev

Refactored setCurrentFunction and setUsbDataUnlocked into single method.

This gets rid of an extraneous configuration change when going from
adb to adb + file transfer as previously the config would have been
reset once for functions and once for data unlocked.

It also simplifies some of the code.

Bug: 31814300
Test: manually changing usb configurations
Change-Id: Ica10a195338b2189db13113f44657393db110bee
(cherry picked from commit 7a396be6d5ba8914933a54b5bfac25e118db0e9f)

Merge "Vodafone SIMs should treat Spain as roaming" into nyc-mr2-dev

Merge "Add Settings.Global configs to high temp warning." into nyc-mr2-dev

MessagingStyle: Fix buggy measure in MessagingLinearLayout

Fixes a bug in MessagingLinearLayout where we would not
recompute the height of it after applying the computed
insets to the text views, leading to incorrect padding
and in rare cases a cut off message at the bottom.

Change-Id: If87a527555158e94e501832e9f49e380cc7da2bb
Test: runtest -x core/tests/coretests/src/com/android/internal/widget/MessagingLinearLayoutTest.java
Fixes: 31463075
(cherry picked from commit 4ac8f40d96196a2a36d7bda7d92eee9bbd4ee7e7)

Merge "[DO NOT MERGE] Do not log boot times for secondary users and upgrades" into nyc-mr2-dev

Merge "Don't lower priority of EGL Init thread" into nyc-mr2-dev

Don't lower priority of EGL Init thread

The EGL Init thread spends a lot of time loading shared libraries,
which involves reading pages and adjusting VM pages. Previously it was
a low priority thread, which meant that when it blocked while holding
a VM lock, it might take a while to be scheduled again once ready, so
it would hold the VM lock longer than necessary, blocking other
threads waiting on soft faults, etc. Classic priority inversion.
Leaving the thread at normal priority causes it to interfere *less*
with critical-path Activity launch work.

Bug: 34611670
Test: go/platform-startup, specifically:
    ./tradefed.sh run google/template/local --template:map test
    google/test/performance/app-launch-perf
    --post-boot-command "service call persistent_data_block 6 i32 1"
    --post-boot-command "service call persistent_data_block 7 i32 1"
    --apps Clock alarmclock --trial-launch --launch-iteration 10
    --launch-order cyclic --drop-cache --trace-directory sdcard
    --trace-iteration 2 --simple-perf-cmd "simpleperf stat -a"
    --report-metrics all --force-skip-system-props --alt-dir `pwd`

Change-Id: I17647c657da64d15f40f8f2b97ee3400e1f0be7f

[DO NOT MERGE] Do not log boot times for secondary users and upgrades

Do not log framework_boot_completed/framework_locked_boot_completed
for secondary users, runtime restarts or first boot/upgrade.

First boot/upgrade also applies to boot_system_server_ready/
boot_package_manager_init_ready

Cherry-picked from commit 4ba91b9a879290d3d810330d172161ab1f923da8

Test: Manual update with fingerprint change
Test: runtime restart - not logged

Bug: 34516002
Bug: 32807863
Change-Id: I64b960c96a0e45b4fefaf05547ea5ac5c701c765

Add Settings.Global configs to high temp warning.

Test: manual
Bug: 34480261
Merged-In: Icfd08c1c7dae456920b3978918ee01469e1dec95
Change-Id: I9ed444dfb89f86d94f3a0db699324318f5bf78d0

Merge "Fix typo in the IS_OMPT_VOICEMAIL field." into nyc-mr2-dev

Make SpannableStringBuilder.getSpans thread-safe

Add cached int buffers for sorting.

Test: All CtsTextTestCases

Merged-In: Ice0b3c3fffc541b26aca94c93fd01e30e13efe2e

Bug: 33609774
Change-Id: Ib728654a384cfc072d1c190611e4553aa8957fa8

Merge "Query for the settings package in KeyguardManager instead of hardcoding it." into cw-f-dev
am: 8ac1bff7e2

Change-Id: I2e4f9b4ea73ff5d0491b8cfda889a57aaac413ff

Query for the settings package in KeyguardManager instead of hardcoding it.
am: e256a18dcd

Change-Id: I3eb7f475c44f815ad2f0e8e1f85c39e555774fcc

Merge "Query for the settings package in KeyguardManager instead of hardcoding it." into cw-f-dev

Fix typo in the IS_OMPT_VOICEMAIL field.

This cause the voicemail content provider to crash on query.

Bug: 34610526
Change-Id: I561ae6616fe66ea49630fa6c4eb9280fdc24b153
Fixes: 34610526
Test: VoicemailContentProviderTest

Vodafone SIMs should treat Spain as roaming

Since there are unnecessary "config_operatorConsideredNonRoaming" values
in the following file, UE mistakenly recognizes all Spain network as
non-roaming when Vodafone sim from Spain is inserted. Fix the issue by
removing the values.
 - core/res/res/values-mcc214-mnc01/config.xml

Bug: 30147413
Bug: 33902721
Change-Id: I75b3f4a112a625a80824516bb6ea06a43a4567da

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

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

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Merge "DO NOT MERGE - Require password to FR in carrier demo mode" into nyc-mr2-dev

Merge "Load EGL early in Activity launch, instead of in Zygote" into nyc-mr2-dev

DO NOT MERGE - Require password to FR in carrier demo mode

Bug: 34341567
Test: manual - password not required for regular user factory resets
and is required in carrier demo mode.

Change-Id: If696e5c5c806e31399e33a151b0d36e7a950ceb0

Load EGL early in Activity launch, instead of in Zygote

Preloading EGL in Zygote was originally a memory footprint
optimization, but it turns out to be an important app startup time
optimization as well. Preloading EGL in Zygote is incompatible with
updatable graphics drivers, but we don't want to do it on-demand as
part of drawing the first frame either, since that increases
first-frame latency unacceptably.

This change removes Zygote preload, and instead loads EGL on a
low-priority background thread immediately after choosing which
graphics driver to use. This means it is usually done well before
drawing the first frame, without significantly disrupting other
activity launch work.

Test: observe systrace of Calculator launch on bullhead
Bug: 34404021
Change-Id: I6a0f6b90ade21848a10d51ddae62c936f70151b5
Merged-In: I887aa09bd35b088b16f53a89838a0c7c98f15761

Query for the settings package in KeyguardManager instead of
hardcoding it.

Bug: 33047002
Change-Id: I57bfab4750540789937d50c72ec05bb7a4535ae1

Merge "DO NOT MERGE Finalize device-monitoring dialog strings" into nyc-mr2-dev

DO NOT MERGE Finalize device-monitoring dialog strings

Add Char-limits, do minor style changes and remove STOPSHIP comment.

Related change on master:
ag/1800040
0d08ca7d7fa12217f4a6562764b4ea65f3fa31f2

BUG:33655277
BUG:31207965
BUG:29748723
Test: only string changes
Change-Id: I37d1b509c6344d7f7d144fc0d8950f45faeaaef4

Merge "Add voicemail backup/restore columns" into nyc-mr2-dev

Merge "[DO NOT MERGE] Do not report boot timings on first boot or runtime restart" into nyc-mr2-dev

[DO NOT MERGE] Do not report boot timings on first boot or runtime restart

During first boot after OTA, additional dexopting has to be done
during PM initialization. Timings for OTA are reported separately,
so we should ignore first boot to avoid skewing the metrics.

The following metrics were updated:
 - framework_locked_boot_completed
 - framework_boot_completed

Cherry-picked from commit 1d87e40d4214a7f6d20a58d6f27cca174f0d974e

Test: manual
Bug: 32807863
Change-Id: I9d545cf38118f45f3f13597df2949d0ae4abd26a

Merge "2D Recents: round task thumbnails properly." into nyc-mr2-dev

Add voicemail backup/restore columns

BACKED_UP and RESTORED are required for dialer voicemail backup/restore
feature for N MR2. As there are no API bump the fields will be hidden
in NMR2. These fields will be made public in O.

ARCHIVED and IS_OMTP_VOICEMAIL is not used for NMR2, but added in
advance to avoid multiple database upgrades.

IS_OMTP_VOICEMAIL will be used in O to dedupe multiple apps implementing
VisualVoicemailService. All voicemail inserted by the service should
set this field to 1. The UI should only display voicemails with this
field set to 1 when the source_package is also the current active
VisualVoicemailService, as other sources represents the same source of
truth(the carrier) but is outdated.

For example, the query should be
WHERE (is_omtp_voicemail == 0) OR
(is_omtp_voicemail == 1 AND source_package = "current.vvm.package")

Test: VoicemailProviderTest
Bug: 34463609
Change-Id: I0f3e58fabff2102adf5bc29d81dac46c7e71d410

Use separate switch ops for runtime permissions
am: e710a6fc66

Change-Id: I86cd237bffdde972de69b91fa086ac5f5ec349e8

2D Recents: round task thumbnails properly.

Test: Checked that on local sw600dp devices, task views are rounded with
8dp. Also checked that Recents works properly on phones.
Bug: 32101881

Change-Id: I6f7a09a12c3e3f1f7270bed1ae25b291fadfb8f2

2D Recents: handle keyboard config changes.

While attaching or detaching a physical keyboard, we should handle the
configuration change in RecentsActivity, so that the activity isn't
relaunched and causes abnormal behaviors.

Test: Checked that while keyboard is attached or detached, Recents
behaves normally.
Bug: 32101881

Change-Id: Ibd185aeb48b31dbde032e235fdcaa17fbcdd9c8a

Fix SpannableStringBuilder getSpans sort

Sort function was broken for odd number of elements.

Test: Added new CTS tests.

Bug: 33567024
Change-Id: I7457dee6ac279c6aab5b92431b347dc1f16a8fa0

Merge "Add arrow key navigation support to Grid-based Recents." into nyc-mr2-dev

Use separate switch ops for runtime permissions

If permissions review is enabled we allow individual
control of fine graned permissions in the SMS group.
This change ensures these permissions use the
corresponding app op as a switch to allow indifidual
control.

Change-Id: I83cd78a78a8266df8324b8a10cd9e36c04ff3112
(cherry picked from commit 99e4d51584202433f1628faa3441e5ced5cf7a0f)

Merge "Eschew unspecified IP addresses for DNS servers" into nyc-mr2-dev

Merge "VPN reconnection fails after manually disabling VPN" into nyc-mr2-dev

Eschew unspecified IP addresses for DNS servers

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-wifi passes (though no test covers this code yet)
Bug: 34210527
Change-Id: I2d44445982dfb3221f7b6394daf0479c1fa8a6e1

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

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

VPN reconnection fails after manually disabling VPN

When disabling VPN manually, there was no trigger for ipsec-tools to
send “delete message” to VPN server. Therefore, connection information
is left in VPN server and next connection fails.

Fix this issue as below:
- Add “delete message” sending via flush in ipsec-tools when racoon
  daemon stops
- Keep daemon alive when VPN.java exit() to let it finish sending to
  VPN server
- Move close(socket) and stop(daemon) in VPN.java execute()
  and monitorDaemons() to run() to gather cleaning block.

(cherry picked from commit 047454c759b46bbadb87ee3b64bf3e29afda48d6)

Change-Id: Ibfbd389b17de5b5a5d23cba59c8d1e05fbe12c15
Bug: 28279646
Bug: 33467086

Merge "Validate custom pointer icons aren't null." into cw-f-dev
am: 05b2d3eb0d

Change-Id: I3a11e06dfe3f476b791838241bfb36e5362063ab

Validate custom pointer icons aren't null.
am: 35068079c8

Change-Id: I37fdde3c1ea735c4888760f3aab43caf7711dd0e

Merge "Validate custom pointer icons aren't null." into cw-f-dev

Merge "2D Recents: polish the app -> recents transition" into nyc-mr2-dev

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Merge "Remove the requirement of the --checkin flag." into nyc-mr2-dev

2D Recents: polish the app -> recents transition

* Do not draw the header as part of the transition thumbnail
  since the header is already shown. Just let the thumbnail
  slot into the task view already drawn on screen.
* Shift the target rectangle by the header height so that
  the end state of the animation corresponds exactly to the
  task view layout.
* Remove unused local variable.

Bug: 32101881
Bug: 33752154
Test: Checked/screencasted recents -> app -> recents animations on local sw600dp device
Change-Id: I4c8ce52b378567114735e6bd582572124a2a354c

Merge "2D Recents: use a different background color for drop targets" into nyc-mr2-dev

Merge "2D Recents: to and from home, animate task views as a block" into nyc-mr2-dev

Merge "DO NOT MERGE - Fix demo mode reset notification" into nyc-mr2-dev

Merge "Add isChildDocument to MtpDocumentsProvider." into nyc-mr2-dev

Merge "Not create document under the device." into nyc-mr2-dev

Add arrow key navigation support to Grid-based Recents.

Test: Checked that arrow key navigation works in Recents on local
sw600dp devices. Also checked that Recents works properly on phones.
Bug: 32101881

Change-Id: I8ece3fc4c9ae82916efcbbde72d40805e43cc70d

Merge "Add keyboard support to Grid-based Recents." into nyc-mr2-dev

Merge "2D Recents: Fix issue with undocking only one task" into nyc-mr2-dev

Merge "DO NOT MERGE ANYWHERE Skip review permissions if package is installed with -g." into cw-f-dev
am: b8c8b67789  -s ours

Change-Id: I502a61cc0c1c6caaf18a7d0c03aa558541b3c17f

DO NOT MERGE ANYWHERE Skip review permissions if package is installed with -g.
am: 4a601f972a  -s ours

Change-Id: Id66c49cfa6010ab327f46e85a6940bd52102eeea

Merge "DO NOT MERGE ANYWHERE Skip review permissions if package is installed with -g." into cw-f-dev

2D Recents: use a different background color for drop targets

Bug: 32101881
Test: Checked layout and animations in Recents, on local sw600dp device.
Change-Id: I628bd0866691e5badd7a37c88020e1e10e1902f7

Validate custom pointer icons aren't null.

Bug: 33853287
Test: cts
Change-Id: I844a9aec6e37e306a77eee0644a774f6470cf5df
(cherry picked from commit b004b5137e12bffae9a3d3ae97440c8c24bdc07a)

2D Recents: to and from home, animate task views as a block

Bug: 32101881
Test: Checked animations to/from home on local sw600dp device
Change-Id: I74b8b02bb39be6d4e4553f7d74550ff177d47a5e

DO NOT MERGE ANYWHERE Skip review permissions if package is installed with -g.

Bug: 34193954

Change-Id: Id744fd190ef78e498e635c2ab9936f12d0bbc917
(cherry picked from commit 6130fb741c86d7d4f0f86e3a875c3eca1fe8aa51)

Fix issue with saving admins before finishing loading. am: f98ed6863a am: b275a205af am: ee9f03d1ed
am: 1894632864

Change-Id: I4328578f929875fd59f48681b8ae2c7c4d3bd623

DO NOT MERGE - Fix demo mode reset notification

Bug: 34358856
Test: manually verified that tapping the Reset notification
triggers a reset after initial provisioning.

Change-Id: I7cf4db729384630a85b22ca9b4947198974a8fbf

Fix issue with saving admins before finishing loading. am: f98ed6863a am: b275a205af
am: ee9f03d1ed

Change-Id: Ic724588cbab8d4282eeb5bc806f8c67791189a05

Fix issue with saving admins before finishing loading. am: f98ed6863a
am: b275a205af

Change-Id: I01487d9c39f4e82b63961cdd43df238f2bfd2b89

Fix issue with saving admins before finishing loading.
am: f98ed6863a

Change-Id: I7f816c164548df86f9607c86772902efd0d53620

svc nfc should support watches too
am: 1876583bfb

Change-Id: Ic3f0277c2c13dbd2c4f8b6ec5dbe9f9dc34ef297

svc nfc should support watches too

Check for both FEATURE_NFC and FEATURE_NFC_HOST_CARD_EMULATION in the
"svc nfc" command.  Watches may only have the latter, but should still
support the nfc enable/disable command.

See b/24532713#comment7 for context.

Bug 32386425

Change-Id: I307798f00728c6b437aebfcf58115af7265e3692

Fix issue with saving admins before finishing loading.

Saving device policy managers settings to clear out
password stats was happening before initializing mAdminList
so could wipe active admins.

Test: manual - flash with N2G05C add google account with dmagent flash wth this fix, check dmagent is still an active admin, reboot check admin is still active.
Test: runtest -c com.android.server.devicepolicy.DevicePolicyManagerTest frameworks-services

Bug: 34277435
Change-Id: I13660b47f30e9aba001eb13f2e457c3b3f36da3e
(cherry picked from commit adbda7474cc1968b66e9948aee566dc346e71340)

Merge "Using an increasing request counter instead of system uptime to avoid duplicates when sorting pending updates." into nyc-mr2-dev

Merge "Fix issue with saving admins before finishing loading." into nyc-mr2-dev

Fix issue with saving admins before finishing loading.

Saving device policy managers settings to clear out
password stats was happening before initializing mAdminList
so could wipe active admins.

Test: manual - flash with N2G05C add google account with dmagent flash wth this fix, check dmagent is still an active admin, reboot check admin is still active.
Test: runtest -c com.android.server.devicepolicy.DevicePolicyManagerTest frameworks-services

Bug: 34277435
Change-Id: I13660b47f30e9aba001eb13f2e457c3b3f36da3e

Add keyboard support to Grid-based Recents.

Test: Checked that pressing tab, shift + tab, alt + tab, alt + shift +
tab work for task navigation in Recents on local sw600dp device. Also
checked that Recents on phones work properly.
Bug: 32101881
Change-Id: I3e4cd212d2900523ece30a85cae7fb73a9594efb

Merge "[DO NOT MERGE] Prevent crash from early Toast surface destruction." into nyc-mr1-dev am: 332762aaf0
am: df459cb2ea

Change-Id: I1239cde924e6d31222bf108c6d55ff19a78019ad

[DO NOT MERGE] Prevent crash from early Toast surface destruction. am: 016c9c8cb5
am: a7ff8fbe74

Change-Id: Ic372436ee48a9f725c178a72c057a2192aad9267

Merge "[DO NOT MERGE] Prevent crash from early Toast surface destruction." into nyc-mr1-dev
am: 332762aaf0

Change-Id: I20cc197cc2f1d4d9dd0b6b6c6fee07e71dd4a1d3

[DO NOT MERGE] Prevent crash from early Toast surface destruction.
am: 016c9c8cb5

Change-Id: Ib33d75a050a1b79de9590f2dbe03609c6e2b4463

Merge "Not allowing foreground services to be cleared as children" into nyc-mr2-dev

Merge "[DO NOT MERGE] Prevent crash from early Toast surface destruction." into nyc-mr1-dev

2D Recents: Fix issue with undocking only one task

While there's only one task and we undock it, it's not in the Recents
task list. In this change, we just reset the transform and return it.

Test: Checked that Recents works properly while docking and undocking a
task and it's the only task running.
Bug: 32101881

Change-Id: I1207b546ff2864018c8d9bf2520f4690762b2d27

DO NOT MERGE: Propagate revoked permissions in permission review mode
am: 4518d0be16

Change-Id: Id0525c380c7b9aaf8c160b1e4fad24b9b2756a0d

DO NOT MERGE: Propagate revoked permissions in permission review mode

In permission review mode we represent install permissions for legacy
apps as constantly granted runtime ones since we need to keep per
user flags associated with a permission. We were not propagating
a revoked permission for a legacy app when the later is upgraded to
a modern app when permission review is enabled.

Test: CTS test passes and manual testing worked fine

Bug: 34240393
Change-Id: I32b8ffee89cc034661f8eb5a172dbe2349ea4a18
(cherry picked from commit fe0253b34ec4fdb6afceb11193f29029b524866a)

Merge "DO NOT MERGE Revert "Stop preloading EGL/GLES in Zygote"" into nyc-mr2-dev

Merge "2D Recents: Always use grid layout even in split view" into nyc-mr2-dev

2D Recents: Always use grid layout even in split view

Simplify the logic a bit, and polish the algorithm to accommodate the
various layout possibilities.

Bug: 32101881
Test: Checked layout in all orientations, with and without split view
Change-Id: I4a6dd3f7f34664908294ddaf5ea516d8e258edbf

Remove the requirement of the --checkin flag.

Bug: 34351332
Test: Manual
Change-Id: Iae8c5782f54c84fc19307468da17a067428c1b2d