Merge "ActivityManagerService: Add an overall stack dumping timeout." into oc-dev

Merge "Don't include IME in app screenshot" into oc-dev

Merge "Store "block uninstall" flag separately from the rest of package state." into oc-dev

Merge "All kinds of little foreground services fixes." into oc-dev

Merge "Adjust forced dexopt in case of stripped APKs." into oc-dev

All kinds of little foreground services fixes.

- Better explanations of some strings.
- Fix to use the right package when an item is pressed in
  the foreground services dialog.
- Improve java docs on Service.startForeground().

Bug: 37925387
    735566 lv - Android - [Source] Context unclear or missing:
    Apps running in background
Bug: 37925888
    735566 lv - Android - [Source] Context unclear or missing:
    Tap for details on battery and data usage
Bug: 37897953
    735566 gl - Android - [Source] Context unclear or missing:
    LEFT_SIDE, RIGHT_SIDE
Bug: 37943079
    Music is dying mid playback

Test: manual
Change-Id: Ied41932c61f59335bd18dfba3fcb13af28a0bddd

Merge "SmartTextShare: Update generic labels for assist menu item." into oc-dev

Merge "RemovalClient sendRemoved should check remaining instead of fingerId" into oc-dev

Merge "Avoid unnecessary ULocale.addLikelySubtags()" into oc-dev

Merge changes Ie1686719,Ie6af7751 into oc-dev

* changes:
  Fixed a bug where the media notification image was wrong
  Improved the gradient drawing of the media notification

Merge "Audio service: only duck started players + refactor" into oc-dev

Merge "Hyphenate text in CheckedTextView" into oc-dev

Fixed a bug where the media notification image was wrong

In RTL the gradient was being applied the wrong way.

Test: switch to RTL, play music
Change-Id: Ie1686719d56436712fd0e6c99ba2723ba53d0025
Fixes: 37996998

Improved the gradient drawing of the media notification

Previously we were drawing with a color on top of the
image instead of fading it out. This could lead to
visible artefacts in the background.
We're now fading out the alpha in a similar manner instead.

Test: runtest systemui
Change-Id: Ie6af7751a734b8fa44279eba970e7ca5ba67a4c0
Fixes: 37950482

Merge "autofill service search uri is not backed up" into oc-dev

Merge "Correct scope of "disabled?" check viz backup/restore decisions" into oc-dev

Merge "Avoid reflection when mocking UserManagerService" into oc-dev

Merge "Use sim language for VZW." into oc-dev

Hyphenate text in CheckedTextView

CheckedTextView style defaults are not derived from TextView, so they
were not getting hyphenated at all. Now they are hyphenated the same
way a normal TextView is hyphenated.

Test: Manual
Bug: 37763646
Change-Id: I3d14e5a1764561cc2922eea569d124c77cb53631

Audio service: only duck started players + refactor

Refactor management of list of ducked players:
  DuckingManager has a list of DuckedApps, which reference
  the ducked players per uid.
Only consider ducking a player when it is in STARTED state.
When a player is released, remove it from the list of ducked players.

Test: play audio in GPM while having driving directions, music ducks
Bug: 37433811
Change-Id: I038a963432c0df6c9470a3a4fb80049d55e8719c

Avoid unnecessary ULocale.addLikelySubtags()

This CL gets rid of unnecessary operations from
LocaleUtils.filterByLanguage() to speed it up, especially for the case
where there is an IME that has many subtypes.

ULocale.addLikelySubtags(ULocale) is known to be slow. Given an IME
that has N IME subtypes, LocaleUtils.filterByLanguage() calls it no
less than N times even when the only one system language is selected.
This has contributed to device boot time (Bug 32343335) time and user
switching time (Bug 28750507) since Android N where IME support
started taking multi-locale into account.

With this CL, LocaleUtils.filterByLanguage() no longer calls it
for a subtype unless its language part of the locale matches one of
user-selected system locales.

The only assumption we made here is

 for any Locale objects l1 and l2
   TextUtils.equals(l1.getLanguage(), l2.getLanguage())
 and
   TextUtils.equals(ul1.getLanguage(), ul2.getLanguage())
 are equivalent, where
   ul1 = ULocale.addLikelySubtags(ULocale.forLocale(l1)) and
   ul2 = ULocale.addLikelySubtags(ULocale.forLocale(l2))

This should be reasonable assumption, at least for locales we want to
care about for IMEs. Under this assumption there is no behavior
change at all.

Test: bit FrameworksCoreTests:com.android.internal.inputmethod.LocaleUtilsTest
Bug: 37647204
Change-Id: Ic96900fcaf3db8b7046a50b3fe6ad65aceada369

Merge "Stack is visible if behind docked which is behind pinned stack (1/2)" into oc-dev

Merge "Limit btsnoop file size (3/9)" into oc-dev

Merge "Fix frozen alarm issue" into oc-dev

Merge "Do not clear recent intrusive flag if still intrusive" into oc-dev

Merge "Remove notification settings for deleted users" into oc-dev

Merge "AOD: Change DozeTriggersTest to run on prox-less devices" into oc-dev

Merge "TrustManagerService: Fix service binding leak" into oc-dev

Avoid reflection when mocking UserManagerService

Fixed a crash due to a change in the private constructor signature

Test: PackageManagerSettingsTests are passing
Bug: 37989886
Change-Id: I5d46f6018b4bdb369b2e39ad3020235e7f2c5a2e

Merge "Fix an NPE that can result from a race condition." into oc-dev

RemovalClient sendRemoved should check remaining instead of fingerId

Test: for each of the following, make sure code goes into
if (client != null && client.onRemoved(fingerId, groupId, remaining)) {
    removeClient(client);
}

1) remove single fingerprint
2) remove all fingerprint by removing password

Change-Id: I6f1ebd42c05fd57225dd4808766791ff1922b09e
Fixes: 37992170

Merge "Remove default network recommendation provider." into oc-dev

autofill service search uri is not backed up

This is expected as the settings is set by the app market of the device
each time the marked it set up.

Bug: 37684646
Test:  mmma -j40 frameworks/base/core/tests/coretests &&
       adb install -r ${ANDROID_PRODUCT_OUT}/data/app/FrameworksCoreTests/FrameworksCoreTests.apk &&
       adb shell am instrument -w -e class android.provider.SettingsBackupTest com.android.frameworks.coretests/android.support.test.runner.AndroidJUnitRunner

Change-Id: Ie9502150d0861173ab58ed7d096d3900704b8915

Do not clear recent intrusive flag if still intrusive

Test: manual, post two noisy progress notifications that update
every second. before change, they start swapping after ten seconds.
after change, no swapping.
Fixes: 36299000

Change-Id: Ia16c36660ae6d59b51b7c5b3521ad43de45aa952

Merge "Fix issue #37938835: Undismissable "Camera is running in the..." into oc-dev

Merge "Better handle MSIM DDS changed event" into oc-dev

Limit btsnoop file size (3/9)

Limit btsnoop file size by rotating between snoop files. The rotation occurrs
when a fixed number of packets have been logged and will start overwriting
the older file.

Bug: 35998031
Test: Enable snoop logs from developer options and let logs get large
Change-Id: I40d5da4c1b1c9b45908e5790d130f1c5e804c773

ActivityManagerService: Add an overall stack dumping timeout.

An ANR can result in stack dumps of a generally unbounded number
of processes. Dumping occurs on a thread that's patrolled by the
watchdog, so we should make sure that all dumps trigger within a
specified timeout.

Test: Manual; by setting the timeout to 1s.
Bug: 37177331

Change-Id: I626272dfbcc9f8b57c1ef1321fac43cdb937a885

Use sim language for VZW.

Instead of doing it 204-04 + gid, change has been made for
MCC 204. This can be revisited if there is any 204 carrier for
which sim language is not set correctly.

Test: none; reverting to earlier behavior for these mcc-mncs
Bug: 37704287
Change-Id: I30661e6467bbf5afc3ebb2ae58ddf36df7d55ce6

Merge "Workaround launching PiP task with CLEAR_TASK & NEW_TASK flag." into oc-dev

Store "block uninstall" flag separately from the rest of package state.

This allows to set "block uninstall" prior to installation and avoid the
inevitable race that happens when Device Policy app tries to force install and
then immediately block uninstall.

BUG=31043188
Test: Block com.chrome.beta in TestDPC. install, fail to uninstall through adb, unblock, uninstall

Change-Id: I5ffa2abcb003982eccfb77585c43b59532dd501d
(cherry picked from commit 1fff9dcb9d0a4d7224ff8aa0f39e82df0b30152c)

Fix frozen alarm issue

Bug 34724174
Test: Manual test with "dumpsys alarm" + putting device into doze

Change-Id: I1d9ec3ac1e78ea263b2598a8c7ab4157ec40c535

Merge "Only classify text when the toolbar is shown." into oc-dev

Merge "Developer setting to show toast for notifications without a channel" into oc-dev

Merge "ignore exception calling unlinkToDeath" into oc-dev

Only classify text when the toolbar is shown.

classifyText() is used to generate a menu item in the selection toolbar
to handle the selected text. In this cl we avoid calling classifyText()
when the selection is changing but instead call it when we are about to
show the toolbar.

Previously, we depended on invalidateActionModeAsync() to classify text
after a call to startActionMode(). Now that we've introduced
invalidateActionMode() we need to be able to tell
startSelectionActionMode() that we also want the text to be classified,
hence the introduction of an input parameter, "adjustSelection", to
startSelectionActionModeAysnc().

Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
Bug: 34966796
Change-Id: I5b9fc9e8ab443f024f8ca461df5a4bcc6485d26b

Don't include IME in app screenshot

Test: Compose message in messenger, go to recents, make sure
IME is not visible in snapshot
Fixes: 37084760

Change-Id: Ic311eb452590fa26adb1ecedb7ae5eba3275d88c

Adjust forced dexopt in case of stripped APKs.

We cannot compile from scratch for strip APK. So
if getDexOptNeeded returns NotNeeded, pretend there was
a compiler filter change, in order to pick up the vdex file.

bug: 37558732
Test: "adb shell cmd package compile" of a stripped /system apk

(cherry picked from commit 96d12a93aeffb8f6db80a7b415cb7d4eb2c4198c)

Change-Id: I3b699a18799ea035ff78c60578c08f8803a17e36

Merge "Handle volume keys to apps in Vr mode" into oc-dev

Merge "selectBackupTransportAsync should report failure when backup is not available" into oc-dev

Merge "Reset touch location after touch slop is exceeded in recents" into oc-dev

Merge "Allow TIMEZONE_CHANGED to launch apps" into oc-dev

Fix an NPE that can result from a race condition.

NetworkInfo and WifiInfo are set via broadcast receiver so while in a
true stable state network info should not be null if wifi info is, this
can occur in rare instances due to a race condition. It is convention in
AccessPoint.java to always null check network info.

Bug: b/37951301
Test: runtest --path
frameworks/base/packages/SettingsLib/tests/integ/src/com/android/settingslib/wifi/WifiTrackerTest.java

Change-Id: Ia1b357c84b961a555ec24a5b930ade35ddf8dbba

Merge "Rename CompatibilityDisplay to Vr2dDisplay" into oc-dev

Rename CompatibilityDisplay to Vr2dDisplay

CompatibilityDisplay used for displaying 2D activities in Vr world. This
is confusing in more than one way. To clarify what it really means,
rename it to Vr2dDisplay.

Changes in this CL -

- /s/CompatibilityDisplay/Vr2dDisplay
- /s/Compatibility/Vr2d - in CompatibilityDisplay context.

Bug: 37851944
Test: cts/hostsidetests/services/activityandwindowmanager/util/run-test
CtsServicesHostTestCases android.server.cts.ActivityManagerDisplayTests
passes on Marlin.

Change-Id: I0a442302fa9d0f48bbd0c01a7766b94369e04386
Signed-off-by: Karthik Ravi Shankar <karthikrs@google.com>

Merge "Added TaskSnapshotCacheTest back to presubmit" into oc-dev

Merge "Always start a manual request, even if view was already autofilled." into oc-dev

Merge "Add persistent state for Bluetooth high quality audio support" into oc-dev

Allow TIMEZONE_CHANGED to launch apps

Bug 37866305
Test: manual

Change-Id: If15df909e99e910c012f2a7b33ae8600c0f97d8d

Fix issue #37938835: Undismissable "Camera is running in the...

...background" notification even after force stopping camera

Ensure we have a message scheduled to dismiss the next app every
time we go through the app list.  Also don't show the notification
for an app when it is in the foreground.

Test: manual

Change-Id: Idd93c8d9f8252b880c8b4bf069c71ad71a6b27c9

Merge "Revert "Don't add other's child as header or footer"" into oc-dev

Merge "Fix potential race in DexManager" into oc-dev

Merge "Don't copy NoCopySpans for assist and autofill" into oc-dev

Merge "Reduce the timeout for waiting for the network state update." into oc-dev

Merge "Avoid reading HW bitmap pixels twice" into oc-dev

Reset touch location after touch slop is exceeded in recents

There was a jump when a user touch moves over the slop value. To fix
this the location is updated to new y position so that the move
animation of the card after has no visual jump.

Change-Id: Ia9b1d04cc0b2983c2e61114ebbf3bdeb154404e9
Fixes: 35720341
Test: manual - enter recents and slowly drag a card vertically and
notice no jump with the fix, moving after the slop has a consistent
motion

Merge "Revert "O is API 26"" into oc-dev

Stack is visible if behind docked which is behind pinned stack (1/2)

Handles the case if pip is shown with splitscreen when the stack order
is pinned, docked, current stack, etc (top to bottom). Added a condition
to check to make sure that this order of stacks allows the current stack
to be visible. Also added a condition to not hide docked stack when pip
appears (this only occurs with command line or cts tests); added TODO
for refactor.

Change-Id: I53bd55014c08c60f360b95ed7100ef49778f891b
Fixes: 37294521
Test: run-test CtsServicesHostTestCases
android.server.cts.ActivityManagerPinnedStackTests#
testPinnedStackWithDockedStack

Merge "Grant ephemeral app access during service bind" into oc-dev

Merge "Accessibility shortcut on Android TV." into oc-dev

Remove default network recommendation provider.

BUG: 37686309
Test: gts-tradefed run gts -m GtsGmscoreHostTestCases -t com.google.android.gts.netrec.NetRecHostTest

Change-Id: Ia67790172418b7feba97acdb4a6df04506358f87

ignore exception calling unlinkToDeath

Change-Id: If3fdb2ca46f8118d9d2859fc932b4614bb62220e
Fixes: 37893511
Test: Manual

Bluetooth: OBEX: Don't close transport when obex is disconnected

Precondition: PTS 7.0

Usecase:
Execute test case TC_TG_CA_BI_02_C for AVRCP 1.6

Expectation:
Test case should pass

Observation:
Test case fails

Root cause:
Disconnection of OBEX from PTS leads to disconnection of transport
(initiated by DUT) leading to test case failure. Obex server was
unnecessary assuming that transport would be disconnected so was
informing registered users that transport has been disconnected,
which was leading to issues on new obex connect on same transport
channel from remote obex client

Fix:
Remove check to break out of loop after obex disconnection, as that is
already handled in case when read would return -1 in case transport
is closed.

Test: TC_TG_CA_BI_02_C can be passed after the patch.

Bug: 35590000
Change-Id: Ic1682ce77c07cf2e43da09dc95fff2214338f758
(cherry picked from commit 850758fb63ded57acf218341812ead95f7159ad6)

Merge "Removed deprecated ViewStructure.setUrl() method." into oc-dev

Better handle MSIM DDS changed event

On DSDS MSIM devices, when Default Data Subscription event
is received, the ImsService may choose to switch which
Slot ID is active.

This change better handles that switch and removes a
crash that was happening due to
ImsServiceProxy#getFeatureStatus getting called
circularly.

Bug: 37361882
Test: Run Telephony unit tests
Merged-In: I55cac364221c728ad1c77be783c358e8d98ec2b4
Merged-In: I68cd04f288f2d7006a2c55cba3584fd74117a1b8
Change-Id: Iaa29bba25f82e699c96b951b3b110568163e12d8

Merge changes from topic 'allow_none_scorer' into oc-dev

* changes:
  Revert permission check on setActiveScorer.
  Allow the active scorer to be set to NULL.

Merge "Add network disabled reason starting index" into oc-dev

Merge "Add compatibility matrices to VintfObject.report()" into oc-dev

Merge "Make all FrameBuilderTests run on RT" into oc-dev

Merge "Use timeout for *dexopt* wake lock" into oc-dev

Revert "O is API 26"

This reverts commit 9bcef8e2da32ba959b61a0b400adfe9fde1a2a10.

Reason for revert:  b/37355569

Bug: 37480230
Bug: 37355569
Change-Id: I39b4a7fd9bda77df0746c7bc32b87f0c2b722f2b

Merge "Fix Array Index out of bounds error." into oc-dev

AOD: Change DozeTriggersTest to run on prox-less devices

Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/doze/DozeTriggersTest.java
Change-Id: I4077d4fcd93f76f08bdd1bd6192cae746f64cc76
Fixes: 37760104

Merge "Notification: Remove actions from public view" into oc-dev

Merge "Prevent NPE in startActivityUnchecked" into oc-dev

Merge "Send AutofillValue of Views that originated a forced request." into oc-dev

Correct scope of "disabled?" check viz backup/restore decisions

*Really* don't back up or restore data for disabled apps.

Bug 37851910
Test: manual

Change-Id: I4b9bdcabb386789b03633d9d26775e54bf3c3d03

Merge "AOD: Pass through pulse reason to DozeUi" into oc-dev

Merge "Don't mix "current dataset" tokens for different backup transports" into oc-dev

selectBackupTransportAsync should report failure when backup is not available

Merged-in: I393308857ea6082dca167e3e417b946615a83c20
Fix: 37304539
Test: runtest -x frameworks/base/services/tests/servicestests/src/com/android/server/backup/TrampolineTest.java
Test: setup device owner, then set up profile.
      Ensure tapping Settings->Google->Work->Security -> Verify app
      is working

Change-Id: I88c3e3163605d62123c88377ae0843084e354212

Merge "Fix Wakelock underlock in GnssLocationProvider." into oc-dev

Merge "Fix RttManager documation unit issue." into oc-dev

Revert "Don't add other's child as header or footer"

This reverts commit 358d0d329785a225449aa9c8aea7fcc2c7bec741.

This CL resulted in b/37933333 regression, so it'll have to re-land with a target API check.

Change-Id: I1c2e4de43a15c1851d98f6aff432585ada80fb5b

Always start a manual request, even if view was already autofilled.

Fixes: 36034784
Test: CtsAutoFillServiceTestCases pass
Test: testAutofillManuallyAgainAfterAutomaticallyAutofilledBefore()
Test: testAutofillManuallyAgainAfterManuallyAutofilledBefore()

Change-Id: Ided8c42367b71766772bd364445510848db2adce

Merge "Fix QS visibility logs missing some info" into oc-dev

Workaround launching PiP task with CLEAR_TASK & NEW_TASK flag.

- When launching an activity with CLEAR_TASK and NEW_TASK, the result code
  of the start is START_SUCCESS, but we still need to notify SystemUI to
  expand the PiP.  However, because the PiP transition now waits for the
  first draw, this can cause severe jank and delay if the original activity
  is a trampoline activity. As a workaround, we immediately move the task
  to the fullscreen stack when clearing and restarting the task to ensure
  that the new task shows without delay.

Bug: 37501224
Test: Open YT in PIP, launch sub-shortcut from Home

Change-Id: I16bebf19b082f30695e99da1d93bc4adf5e9df0c

Merge "TextClassifier: Append  to url intents that need one." into oc-dev

TrustManagerService: Fix service binding leak

Fixes an issue where TrustAgentWrapper was created too early.
If we later decide to not actually start that agent, it is already
bound to, and then we just drop the TrustAgentWrapper class, never
unbinding.

In addition, this can also create a rebind loop where we are constantly
trying to rebind to the trust agent.

Bug: 37853858
Test: force-lock, observe that trust agents are properly unbound and not constantly trying to rebind
Change-Id: I4d383f9cb499874803a7f024915baf85b3682631

Handle volume keys to apps in Vr mode

When a device is in Vr mode -
 - Volume key event must be passed to the audio subsystem to reflect
 changes when the configuration is enabled.
 - When the configuration is disabled, neither the apps not the audio
 subsystem should be able to receive the volume key events

Bug: 37625887
Bug: 36280728
Test:
- Verified that the app doesn't receive volume events
- Verified that the system volume bar is also not triggered

Change-Id: I245f7797873f764d0b017879723a6dbff1bb4d36
Signed-off-by: Karthik Ravi Shankar <karthikrs@google.com>

Fix potential race in DexManager

The internal cache of DexManager holding information about the code
locations may race if an install happens at the same time with a dex
load.

Test: runtest -x services/tests/servicestests/src/com/android/server/pm/dex/DexManagerTests.java
Bug: 37922353

Change-Id: Ia28fb54d6859ec185aaaf8e0ed68981004b173e5