Merge "Add metrics for status bar" into oc-mr1-dev

Merge "Update demo mode clock to 8:00 for O." into oc-mr1-dev

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

Merge "Fix a bug where image apps were double-counted." into oc-mr1-dev

Merge "Fix issue #65359137: Apps with target API 26 can bypass background..." into oc-mr1-dev

Merge "Use raw string when setting wifi preference title." into oc-mr1-dev

Merge "Import translations. DO NOT MERGE" into oc-dr1-dev
am: b1d8caa355  -s ours

Change-Id: Ifd66f13f6b1384eae3f3697b7ac275c442386e0c

Merge "Import translations. DO NOT MERGE am: 2c1d066618  -s ours" into oc-mr1-dev

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

Import translations. DO NOT MERGE
am: 2c1d066618  -s ours

Change-Id: I70ac747da20112bcabac751ab990e82da87bfcb1

Add metrics for status bar

Test: runtest systemui
Fixes: 65289824
Change-Id: Ife5ce98862fc6a92740eaf80cdc0b74cb01dd04e

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

Merge "Retry crashed bound foreground service with some delay" into oc-mr1-dev

Merge "Close idle connections to release RAM" into oc-mr1-dev

Merge "Allow Instant Apps to access send_action_app_error" into oc-mr1-dev

Merge "Fix shift in share sheet" into oc-mr1-dev

Fix issue #65359137: Apps with target API 26 can bypass background...

...limit introduced in Oreo by swiping away from the task list

There was an edge case where, when restarting a service, if the
process wasn't already running we would bring it up directly in
the idle state and never go through the mechanism of force stopping
services once an app goes idle.

The fix for now is pretty targeted, detecting when we have a new
uid that is immediately idle, and in that case doing the same
thing we do when an existing uid goes idle (stopping any services
in it).  This means that in this flow you will get a glitch of
the service starting and then immediately stopping when idle is
applied at that point, but we do end up ultimately in the correct
final state.

Test: manual
Bug: 65359137

Change-Id: I64c29975ad75e7d7390512bf59673d423df16cf2

Merge "Make settings themed apps have light nav" into oc-mr1-dev

Merge "Use correct thread to remove starting window" into oc-mr1-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import
Exempt-From-Owner-Approval: translation import

Bug: 64712476
Change-Id: I1f5a8fdd408de1a74689fe6e1de0422529d797d8

Import translations. DO NOT MERGE

Auto-generated-cl: translation import
Exempt-From-Owner-Approval: translation import

Bug: 64712476
Change-Id: I8f4884b5af544c9846580af19a9b23badbed357c

Make settings themed apps have light nav

This is needed to help with differential aging.

The nav needs to be white with a 1dp divider on it, so add support
for the divider and add hidden attribute to set the nav buttons
inverted.

Test: Open settings
Bug: 63630024
Change-Id: Iec6046baeb86fb554df73346df8652fee8b2736a

Merge "Keyguard: Don't show when booting to cryptkeeper" into oc-mr1-dev

Use correct thread to remove starting window

Test: Open app, reopen app
Test: go/wm-smoke

Bug: 63784898
Bug: 64954955
Fixes: 65256018
Change-Id: I58eb90796a002cc6962efd686cef73fb36040567

Merge "Fix resetPasswordWithToken before user unlock" into oc-mr1-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import
Exempt-From-Owner-Approval: translation import

Bug: 64712476
Change-Id: Ib170094788c00758e4fb41bdcbf781ac196b0e34

Import translations. DO NOT MERGE

Auto-generated-cl: translation import
Exempt-From-Owner-Approval: translation import

Bug: 64712476
Change-Id: I409427c1b5e9b8b3e29947002e8bc6d5985bec77

Merge "Add PACKAGE_USAGE_STATS permission to shell." into oc-mr1-dev

Merge "disable carrier actions on redirection by default" into oc-mr1-dev

Merge "Lighter Quick Settings" into oc-mr1-dev

Merge "Add dump of USB descriptors to the dumpsys for UsbHostManager service." into oc-mr1-dev

Use raw string when setting wifi preference title.

This is necessary for talkback to correctly recognize content
description on the whole viewgroup.

Merged-In: Ic792102b269b9a8db09c632f7e70d8773048c840
Change-Id: I555f94afcd6f4c681bb16dd23d94b2ae0fd4c116
Fixes: 65202862
Test: robotests

Merge "Add EVENT_ROAMING_SETTING_CHANGE in DctConstants." into oc-mr1-dev

Fix a bug where image apps were double-counted.

The filter for photos apps includes them as well as the one for "other
apps". This removes them out of other apps.

Change-Id: I56058080b6492c054ea3171addd5ab343769bc9a
Fixes: 65384691
Test: SettingsLib integ test

Retry crashed bound foreground service with some delay

After quick crashes, back-off for 30 minutes and try again.
Keep backing-off up to 10 times and then give up.

Bug: 63075467
Test: runtest -x frameworks/base/tests/ServiceCrashTest

Change-Id: I3819aefac5fd48b49a70b1765e07696f2ad61328

Lighter Quick Settings

Now using material grey 900 for QS, volume and power menus.
Feels more polished than just a solid black.

Test: Visual, on multiple devices
Change-Id: Ife05454d9fa0cf5f587ed0e58f3b28447de14f7b
Fixes: 65369406

Merge "audio: Fix for IPlayerShell object leak." into oc-mr1-dev

Merge "Fix issue #65298216: DeviceIdleController needs to watch..." into oc-mr1-dev

Merge "Nat464Xlat: correct racefree teardown" into oc-mr1-dev

Merge "Allow translucent RenderNode->Bitmap captures" into oc-mr1-dev

Merge "Always use SSID average for SpeedLabel." into oc-mr1-dev

Merge "resolve stage dir before validation" into oc-mr1-dev

disable carrier actions on redirection by default

1. the assumption of "carrier will redirect users to their captive
portal page when users dont have data plan" might not be true for all
carriers. We have seem issues like carriers random redirect users when
they still have balance left.
2. turn this off by default and turn it on if the carrier is
behaving correctly and agree with it.

Bug: 65113738
Test: Manual
Change-Id: I6bc2d8ba45958eaf09e09e3ea2e57c79e0ac461e

Merge "DO NOT MERGE: Dynamic doze check should be required before adjusting sensor rate. am: 1f3da1c8b9  -s ours am: c7088d8bf8  -s ours am: ef6e002b30 am: 44bdde7bd1" into oc-mr1-dev

Merge "Double tap to expand PiP." into oc-mr1-dev

DO NOT MERGE: Dynamic doze check should be required before adjusting sensor rate. am: 1f3da1c8b9  -s ours am: c7088d8bf8  -s ours am: ef6e002b30
am: 44bdde7bd1

Change-Id: Ie9ed360a7a1ab763310bcf743804bd0f3300f9a3

DO NOT MERGE: Dynamic doze check should be required before adjusting sensor rate. am: 1f3da1c8b9  -s ours am: c7088d8bf8  -s ours
am: ef6e002b30

Change-Id: Ie9aabc371ae2bad03bfced07acce9502d6c897ee

DO NOT MERGE: Dynamic doze check should be required before adjusting sensor rate. am: 1f3da1c8b9  -s ours
am: c7088d8bf8  -s ours

Change-Id: I6cbd05dcb09b013e17f5c26c1fa24a656c7b8316

Merge "Prevent INTERACT_ACROSS_USERS exception in DefaultDialerManager." into oc-mr1-dev

Merge "MediaBrowserService: Clean up ConnectionRecord when browser is dead" into oc-mr1-dev

Merge "Remove config_tether_apndata for Telstra." into oc-dr1-dev
am: e424d123ab

Change-Id: I58c3c189ef03bf4d054e69fae1c45852425f70d7

Add dump of USB descriptors to the dumpsys for UsbHostManager service.

Cleaned up text formatting.

Bug: 65158441
Change-Id: Ie6531d42bb0d78f2b8af382329a973e629f23c5f
Test: Manual - connected USB headset, ran adb shell dumpsys usb. Examined output.

Merge "Remove config_tether_apndata for Telstra." into oc-dr1-dev

DO NOT MERGE: Dynamic doze check should be required before adjusting sensor rate.
am: 1f3da1c8b9  -s ours

Change-Id: Ib94052397392808af53de5c6f265a0cd29950b4e

Merge "Fix A2dpProfileTest#getLabelDeviceConnectedWithHighQualityCodec" into oc-mr1-dev

Merge "Fix clipping on 3g icon" into oc-mr1-dev

audio: Fix for IPlayerShell object leak.

IPlayerShell object is created for each MediaPlayer object and
this will register to linkToDeath, which will have global
reference. IPlayerShell release method is unregistering to Death
monitor. Here IPlayerShell object is not released at all, when
application releases mediaplayer object. As linkToDeath will have
global reference for each IPlayerShell object, GC will not be able
to clear this object.

Fix is to call IPlayerShell release when application releases
mediaplayer object.

authored-by: Divya Narayanan Poojary <dnaray@codeaurora.org>
Test: manual
Bug: 65374542

Change-Id: Icbd7632d462d44efdbb851c1c8a662bdef0e551c

Merge "AAPT2: Make sure to fail when XML linking fails" into oc-mr1-dev

Merge "Don't crash from null ServiceState in cdma" into oc-mr1-dev

Allow translucent RenderNode->Bitmap captures

Set translucent to true when doing createHardwareBitmapFromRenderNode.
This ensures that the buffer is initialized to transparent which
is a safer default for this path. Marginally less performant in some
cases, but as this is a rare capture safety is preferred for and
performance can be revisted if it turns out to be an issue.

Bug: 65164016
Test: builds
Change-Id: I6467c0ccf46e5e2a87c31cc4855ced8275236ca9

Always use SSID average for SpeedLabel.

When showing the SpeedLabel for the connected network, continue to use
the SSID based average rather than switching to the curve for the known
AP. This addresses changes in SpeedLabel from when the user selects a
network vs when it appears in the connected section.

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

Change-Id: I06edc1ca4b1b5c8e8615f48c000544ea22a6bbbd

Merge "FalsingManager: Allow disabling the HIC via device config flag" into oc-mr1-dev

resolve stage dir before validation

We rely on the stage directory being correctly resolved
in order for validation to occur. This forcably happens
when commiting [after validation], reading or writing
to the session. For normal installs, this isn't an issue
since the session will be written to, thus, resolving
the stage directory.

Instead of waiting until the commit, resolve the stage
directory just prior to package validation.

Change-Id: I667b9d27749405d24e3a3c6666e9591ce3e883df
Fixes: 65369970
Test: bit FrameworksCoreTests:android.content.pm.PackageManagerTests\#testInstallNonexistentFile

Fix issue #65298216: DeviceIdleController needs to watch...

...interactivity, not display state

Now it does.

Test: manual
Bug: 65298216
Merged-In: Iec15bff87c6eef43f60b706fa71954c983a4a1e8
Change-Id: Iec15bff87c6eef43f60b706fa71954c983a4a1e8

Double tap to expand PiP.

- Also fix issue with dismiss scrim not clearing when quickly swiping in
  the dismiss area and back, due to an incorrect check (menu state NONE
  != menu is not visible).

Bug: 62078042
Test: com.android.systemui.pip.phone.PipTouchStateTest
Test: Double tap PiP to expand it

Change-Id: I96d9ba169feadf0ea8315152fb3a5b2cfb69a2f0

DO NOT MERGE: Dynamic doze check should be required before adjusting sensor rate.

Some devices have catastrophic firmware issues in switching the sensor rate;
for these devices, the sensor rate switch should be skipped entirely.

Change-Id: I04fc3d27c99f95b33f86f012a8d398f817f7713a

Merge "Do not consider overlays when finding task to reuse." into oc-mr1-dev

Merge "Add internal implementation for notifyPackageUse" into oc-mr1-dev

Merge "Delete old TEXT_SELECTION_MENU_ITEM_ASSIST logs." into oc-mr1-dev

Merge "Move to using a profile for boot image" into oc-mr1-dev

Merge "AOD Policy: Fix pausing timeout to use minutes instead of seconds" into oc-mr1-dev

Don't crash from null ServiceState in cdma

Test: runtest systemui
Change-Id: Ib9813af4896d90878df970ef798f3f09e6522f3a
Fixes: 65224543

Fix clipping on 3g icon

Viewport was a _little_ too small.

Test: visual; adb shell am broadcast -a "com.android.systemui.demo" -e
command network -e mobile show -e datatype 3g
Fixes: 64725694

Change-Id: I7e09bb8bb2a93ce6dc6670c175284d3927b007d7

FalsingManager: Allow disabling the HIC via device config flag

Bug: 65126931
Test: Verify HIC is still enabled on devices where it was not disabled
Change-Id: I30ac9e156dde693c34039d8c21e404a474dd86eb

Fix resetPasswordWithToken before user unlock

1. Fix system server crash when resetPasswordWithToken is called before use
   unlock, due to DPMS enforces user is unlocked when calculating password
   sufficiency.
2. Propogate new password metric from LockSettingsService to DPMS after a
   password reset with token, and fix a bug where stale quality was used.

Bug: 64923343
Bug: 64928518
Bug: 65286643
Test: cts-tradefed run cts-dev -m CtsDevicePolicyManagerTestCases -t com.android.cts.devicepolicy.ManagedProfileTest#testResetPasswordWithTokenBeforeUnlock
Test: cts-tradefed run cts-dev -m CtsDevicePolicyManagerTestCases -t com.android.cts.devicepolicy.MixedManagedProfileOwnerTest#testResetPasswordWithToken
Test: runtest frameworks-services -p com.android.server.locksettings
Test: cts-tradefed run cts-dev -m CtsDevicePolicyManagerTestCases -t com.android.cts.devicepolicy.DeviceAdminHostSideTestApi24#testRunDeviceOwnerPasswordTest
Test: runtest frameworks-core -c android.app.admin.PasswordMetricsTest
Test: runtest frameworks-services -c com.android.server.devicepolicy.DevicePolicyManagerTest
Change-Id: Ibb3736547b3b36da4a8a67af711e08a38427aa56

Delete old TEXT_SELECTION_MENU_ITEM_ASSIST logs.

This field is now being used for new smart selection logging.
See: Icc9d5b542140bdb01b2ad4211048bd83eae2ce83

Bug: 64914512
Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
Test: bit CtsWidgetTestCases:android.widget.cts.TextViewTest
Test: bit CtsAccessibilityServiceTestCases:android.accessibilityservice.cts.AccessibilityTextTraversalTest
Change-Id: If1ce8ca038b089500cd7adb9a146330abe909407

Nat464Xlat: correct racefree teardown

This patch relays the NetworkBaseObserver notifications about nat
464xlat stacked interfaces onto the ConnectivityService handler.

This allows to process interface up and down notifications in the
same thread context and eliminates several races:

  - NPE risk due to race between fixupLinkProperties called on
    ConnectivityService thread and interfaceRemoved called on
    NetworkManagementService thread.
  - stale LinkProperties pointer reads in both NetworkBaseObserver
    callbacks not called on ConnectivityService handler.
  - removes the race between stop() and interfaceRemoved().
  - removes superfluous LinkProperties notifications when stop() is
    called before the stacked interface goes up.

The teardown procedure logic common to stop() and interfaceRemoved() is
put into enterStoppedState() and enterIdleState().

This allows to distinguish and correctly handle the following teardown
scenarios:
 - an IPv4 appears -> ConnectivityService calls Nat464Xlat#stop()
                   -> Nat464Xlat calls stopClatd
                   -> clatd stops
                   -> if the stacked interface was up, it is removed
                   -> Nat464Xlat#interfaceRemoved() is triggered and
                      a LinkProperties update is sent.

 - network disconnects -> ConnectivityService calls Nat464Xlat#stop()
                       -> Nat464Xlat calls stopClatd
                       -> clatd stops
                       -> if the stacked interface was up, it is removed
                       -> Nat464Xlat#interfaceRemoved() is triggered and
                          a LinkProperties update is sent.

 - clatd crashes or exit -> Nat464Xlat#interfaceRemoved() is triggered
                         -> Nat464Xlat unregisters itself as a network
                            observer
                         -> ConnectivityService is updated about the
                            stacked interface missing, and restarts
                            Nat464Xlat if needed.

Note that the first two scenarios have two cases: stop() can be called
before the notification for the stacked interface going up (STARTED), or
after (RUNNING). In the first case, Nat464Xlat must unregister
immediately as a network observer to avoid leaks.

This patch also:
  - removes/simplifies comments related to the threading model which
    are no obsolete.
  - extract clatd management logic from ConnectivityService into
    NetworkAgentInfo
  - add new unit tests where there was none before.

Bug: 62918393
Bug: 62997041
Bug: 64571917
Bug: 65225023
Test: runtest frameworks-net
Merged-In: I27221a8a60fd9760b567ed322cc79228df877e56
Merged-In: I8f07dfbe5ea8259ff9f5793503f534945e67ad74
Merged-In: I8612db5e5050690db8cf41dd04944b4c22da340c
Merged-In: Icb2dc8229b5ea45e319233b588f2dbe39ea40d4c
Merged-In: Ibafea69224e832a6316c17dbb9b2d62a233088ac

(cherry picked from commit ef502887ec58886e9347afb841aa06cb0d13acea)

Change-Id: I9d075048873b0e1c5ed45b5674ada3fb303c2bfb

Merge "TextView text selection interaction logging." into oc-mr1-dev

AOD Policy: Fix pausing timeout to use minutes instead of seconds

Bug: 65010918
Change-Id: I2a5f3a2109a38d7093fd0ea84fbb1026eb6c1215
Test: On AOD cover prox for 10s. Verify screen turns off.

Merge "ConnectivityService: improve wakelock logging" into oc-mr1-dev

Do not consider overlays when finding task to reuse.

An overlay should not be considered the top activity in a task when
considering a task when launching an activity. Doing so will alter
the behavior of launch modes, such as singleTop and singleTask. In
these cases, the developer has chosen such mode with the expectation
that their task will not have activities from other tasks placed on
top, which is the case for features such as locking a work profile.

This changelist addresses the issue by not considering an overlay to
be the top activity when finding a task based on ActivityRecord.

Fixes: 64839155
Test: bit FrameworksServicesTests:com.android.server.am.ActivityStackTests#testFindTaskWithOverlay
Change-Id: I2684baf6929e5af321404e2eef597f456ff87ee8

ConnectivityService: improve wakelock logging

This patch adds the following wakelock related counters to connectivity
service dumps included in bug reports:
 - total number of wakelok acquisitions and releases
 - total cumulative wakelock duration
 - longest time the lock was held

Bug: 65085354
Test: runtest frameworks-net, also manually dumped connectivity service
      and check new logging

Change-Id: I8f67750c2eea73abf3d44f7f6df484427a8ea3f9

Fix A2dpProfileTest#getLabelDeviceConnectedWithHighQualityCodec

Fixes: 63899732
Test: make RunSettingsLibRoboTests -j7
Change-Id: I60559a04a1c234acaca2919a67e8bbe126d2bb3a

TextView text selection interaction logging.

TODO: Log typing over selection

Bug: 64914512
Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
Test: bit CtsWidgetTestCases:android.widget.cts.TextViewTest
Test: bit CtsAccessibilityServiceTestCases:android.accessibilityservice.cts.AccessibilityTextTraversalTest

Merged-In: Icc9d5b542140bdb01b2ad4211048bd83eae2ce83
Change-Id: Icc9d5b542140bdb01b2ad4211048bd83eae2ce83

Merge "AOD: Keep brightness sensor registered while covered" into oc-mr1-dev

Merge "Set flags on the correct Intent, please" into oc-mr1-dev

Record even more stats even more often

Like kale, one can never have enough stats.  =)

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
Bug: 29337859
Bug: 32163131
Change-Id: Ieb47c3beed50f21c2c858fe57438afd48cfdc662

Merge "Add a photos app filter and a metric for using it." into oc-mr1-dev

Merge "Low ram devices now use 60% reduction for task snap shots" into oc-mr1-dev

Merge "Fix issue #65259946: APR: runtime restart: at com.android.server.am.ActivityManagerService" into oc-mr1-dev

Merge "Gracefully handle integer overflows." into oc-mr1-dev

Merge "Fix issue with PiP menu activity disallowing touches" into oc-mr1-dev

Merge "Bluetooth: Add AVRCP 1.3 as a developer option for AVRCP version (1/2)" into oc-mr1-dev

Bluetooth: Add AVRCP 1.3 as a developer option for AVRCP version (1/2)

This is requied for the 2012 Mazda 3 carkit as the carkit refuses to send
AVRCP commands other than passthrough commands when the version is anything
else. AVRCP 1.3 is compatible with most carkits on the market and can be
used to get most carkits working at the cost of losing many features.

Bug: 37943083
Test: Set AVRCP 1.3 in developer options and see that SDP and the AVRCP
      capabilites have updated to reflect this.
      TestTracker: 105915/3975

Change-Id: I637b695e5cd8baa5f054ca0dda81372080a96269

Merge "Add test for first draw blend behavior in HWUI pipeline" into oc-mr1-dev

Merge "Fix NPE in ActivityView on surface destruction" into oc-mr1-dev

Add PACKAGE_USAGE_STATS permission to shell.

This is needed for calling AM.registerUidObserver.

Fixes: 64400666
Test: cts-tradefed run singleCommand cts-dev -m CtsAppTestCases -t \
      android.app.cts.ActivityManagerProcessStateTest
Change-Id: I4f500d0d8d516b6b8961ea2f8c083add3ae949a9

Merge "use simOperator name in the notification" into oc-mr1-dev

Fix issue #65259946: APR: runtime restart: at com.android.server.am.ActivityManagerService

Catch security exception as well.

Bug: 65259946
Test: manual
Change-Id: Icffccb7d28ef2cf01955173f7dc5adbb81ed0caa

Move to using a profile for boot image

Updated the system server profile, moved to using a profile for the
boot image.

Use cases for obtaining profiles:
Automated CUJ tests
Automated app launches
Manual usage of dialer, navigation, playstore, and play music.

Storage savings:
Boot art+vdex+oat size: 78305556->61539326 (-17MB)

New system server profile has more methods (1.5MB -> 4MB oat) since it
has more coverage from use cases.

Ram savings (sample averages for CUJ test):
Average PSS of 3 runs (~100 samples total):
.Boot_vdex: 9359K -> 9902K
.oat_mmap: 21857K -> 13372K
.Heap: 29779K -> 29483K
.Boot_art: 15736K -> 18096K
.LinearAlloc: 9112K -> 8965K
Total PSS: 85846K -> 79819K (-6MB)

Average pages read from flash (512MB device) for CUJ tests (3 samples):
1508MB -> 1183MB

TODO: Investigate why .art goes up, maybe we are missing some classes
in the profiles.

Bug: 37966211
Bug: 63178181

Test: make and flash

Change-Id: I2031c52eab7b868c59d4c5a46cad50bfbc6cd714