Merge "Update process state after unbinding a service." into oc-dev

Merge "Resolve styles by font table if not specified." into oc-dev

Merge "Revert "Treat apps that use channels as O apps"" into oc-dev

Merge "Bluetooth: Move Bluetooth battery stats tracking (1/2)" into oc-dev

Merge "Fixing regression in what is considered a restart attempt" into oc-dev

Merge "TextClassifier: Remove debug logs." into oc-dev

Merge "Add wifi throttling to backup blacklist." into oc-dev

Revert "Treat apps that use channels as O apps"

This reverts commit 07a08b2ae92cc26b5e7ec00064e6ecea79d3d9c3.
Bug: 38016678
Change-Id: I0bf2f6da9e145bfe1f79449cac38ad34f5956969

Merge "Make status bar disable() debugging more concise." into oc-dev

Resolve styles by font table if not specified.

android:fontStyle and android:fontWeight is an optional attribute.
If they are missing, we are using non-italic 400 weight, but this is
not good idea. We should resolve these style value from font metadata.

Bug: 37988154
Test: am instrument -w -e class android.content.res.cts.ResourcesTest
      android.content.cts/android.support.test.runner.AndroidJUnitRunner

Change-Id: I80a436cd3632d8c58fa3d8bdaedac73b95f61a8c

Merge "Prevent NPE in ActivityStack#shouldbeVisible." into oc-dev

Merge "Fix settings test for show_notification_channel_warnings" into oc-dev

Merge "WifiManagerTest: test for startLOHS return code" into oc-dev

Merge "ImageWriter: Attach non-opaque Images as well." into oc-dev

Fixing regression in what is considered a restart attempt

- CTS tests can start activities directly in the pinned stack with the
  CLEAR_TASK and NEW_TASK flags, so this check should only consider it
  a restart attempt if mReusedTask is set, as in
  setTaskFromIntentActivity(), otherwise the newly started activity
  will be triggered back into fullscreen.

Bug: 37501224
Test: android.server.cts.ActivityManagerPinnedStackTests
Change-Id: Ia37f9d77bfb85882e50271300234bfc6c2969f9a

Merge "Ensure that we hide the menu without resizing when expanding." into oc-dev

Merge "Revert "Create auxiliary info for local instant apps"" into oc-dev

Revert "Create auxiliary info for local instant apps"

This reverts commit e15e7cb592a923eea029c12aa9fc144a3fbaddc0.

Change-Id: I116904fd8c2b28df7e03bc5ec2414ed67761b380

Merge "Add TRON logging for bindApplication" into oc-dev

Merge "Audio service: always unduck focus winner" into oc-dev

Merge "Fix bounds calculation" into oc-dev

Merge "Don't set the failure extra on split install" into oc-dev

Merge "Create auxiliary info for local instant apps" into oc-dev

Prevent NPE in ActivityStack#shouldbeVisible.

If there is a visibleBehind activity, we check to see if the top
activity in the top stack is not fullscreen. However, it is possible
for the top stack to be empty. This can happen if the previously top
activity is being re-installed.

This changelist addresses the issue by checking if the top activity is
null before referencing it.

Change-Id: I6de904bed1c7035ed1e112c9cacc1b6c2bac4e8f
Fixes: 32180256
Test: bit FrameworksServicesTests:com.android.server.am.ActivityStackTests#testShouldBeVisibleWithVisibleBehindActivity

Merge "Finish noHistory activities immediately after resumed activity shown." into oc-dev

Add TRON logging for bindApplication

Such that we know how much startup latency is the system to blame
and how much of it is the client process to blame.

Test: Open dead app, inspect logs
Change-Id: Ib8d359c2d59ef6f90893928f2b1d4e2ac2590d74
Fixes: 37327717

Fix bounds calculation

Don't include the wallpaper window when calculating the frame
in multi-window.

Test: Go multi-window, go recents.
Change-Id: I8b4fb48767d6bdbda055d9c631811ad5a1dcaee6
Fixes: 36240988

Merge "Import translations. DO NOT MERGE" into oc-dev

Merge "Import translations. DO NOT MERGE" into oc-dev

Merge "Import translations. DO NOT MERGE" into oc-dev

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

Import translations. DO NOT MERGE

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

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

Import translations. DO NOT MERGE

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

Merge "Hyphenate text in CheckedTextView" into oc-dev

Audio service: always unduck focus winner

When a FocusRequester gains focus, always unduck it.
This fixes a bug where a player could remain ducked even after
  regaining focus. The issue was that the unducking was only
  done in the focus gain if the loss had been
  AUDIOFOCUS_LOSS_TRANSIENT_CAN_DUCK, which is the only one
  that could have caused a duck. But between the moment
  the app lost audio focus with LOSS_TRANSIENT_CAN_DUCK and
  the moment it regains focus, it could have lost the focus
  once more with LOSS_TRANSIENT. And in this case, it would
  not have been unducked.
The fix consists in always unducking an app that gains focus. If
  it wasn't ducked, that operation is a no-op.

Test: see bug
Bug: 37433811
Change-Id: I95a5bfcc2d5063986bd62841c1893bbbd00b45bf

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

Import translations. DO NOT MERGE

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

Bluetooth: Move Bluetooth battery stats tracking (1/2)

Move Bluetooth battery stats tracking to AppScanStats for a more
unified tracking system.

Bug: 37720787
Test: cts-tradefed run cts-dev -m CtsIncidentHostTestCases -t com.android.server.cts.BatteryStatsValidationTest#testBleScans
      Perform BLE scan and check battery stats
Change-Id: Ie6c682374e6e258c291d3a11eede649c7747ef40

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

Ensure that we hide the menu without resizing when expanding.

- This should be done when expanding/dismissing from the menu activity,
  otherwise the menu state change can cause a stack-resize animation that
  clobbers the expand or dismiss animation.

Bug: 37650245
Test: Launch Duo in PIP, end call from notification tray
Change-Id: Ie880715ae7be28a4fe2dd0d294e369b62e776c35

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

Update process state after unbinding a service.

Bug: 37334267
Test: manual
Change-Id: I5304ea26a1e742523b7b22fa0c7e1ab7d1ece9b3

Merge "Fix frozen alarm issue" into oc-dev

Add wifi throttling to backup blacklist.

Test: Ran SettingsBackupTest test.

Bug: 37684646

Change-Id: I3cd2e30fc6736f36b8569400941a3128e06d2f48

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

Fix settings test for show_notification_channel_warnings

Test: runtest --path
core/tests/coretests/src/android/provider/SettingsBackupTesst.jav
Bug: 37684646

Change-Id: I77ff42107979b22ca2d4b81a406fc69a5b16417f

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

ImageWriter: Attach non-opaque Images as well.

With all the work under the hood of ImageReader/Writer in the last few
releases, this is much more straightforward to enable.

Only trick is to ensure that the Image detached from the ImageReader
correctly clears out its planes, if present.

Bug: 19962027
Test: Camera CTS still passes on angler, sailfish, ryu
Change-Id: Ic8a1f3e2f45cf15971d76e308a5af25eb2fd66f1

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