Fix NPE in TaskSnapshotController

Test: Pray
Change-Id: Ie65f072b35fbf1a9a65caf5be48dabbb7abb6be8
Fixes: 36592724

Merge "AAPT2: expose return values through JNI." into oc-dev

Merge changes from topic 'bt5-cherry-pickery-2' into oc-dev

* changes:
  Remove enable from PeriodicAdvertisingPariameters (1/2)
  Bluetooth 5 fix default AdvertisingSetParameters values
  Bluetooth: fix comment wording

Merge "Keep a removed API to not break GCore presubmit" into oc-dev

Merge "Deprecated ACTION_PRECISE_DATA_CONNECTION_STATE_CHANGED" into oc-dev

Keep a removed API to not break GCore presubmit

Change-Id: Ica38b7aad2a5a8792917d0da1d497f373b118077

Merge "Add getSelfManagedPhoneAccounts() TelecomManager API." into oc-dev

Merge "Make sure system UI context has the same LoadedApk as system context" into oc-dev

Merge "Revert "Revert "Handle case when snapshot dimensions don't match""" into oc-dev

Merge "add debug logs" into oc-dev

Merge "Expose UserHandle.getIdentifier() to CTS." into oc-dev

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

Merge "Update fontchain_lint to test new emoji font" into oc-dev

Merge "Fix screen pinning in seascape" into oc-dev

Merge "Hide ColorSpace.Renderer" into oc-dev

Merge "ImageReader/Writer: add usage flag support" into oc-dev

Remove enable from PeriodicAdvertisingPariameters (1/2)

Instead of setting enable to true, one can just pass null
PeriodicAdvertisingParameters and achieve same result when starting the
set.
Passing the "enable" when updating the parameters make no sense, and
might be confusing.
Experience with "timeout" field, which was a part of AdvertiseSettings
show that merging fields that go into different HCI commands can cause
problems during processing, so keep enable as separate field.

Test: manual
Bug: 30622771
Change-Id: Ida02c59eb8433537179b4d22202fe745f8b4bb3e
(cherry picked from commit e6c453d828fec3df82708d2b0c3bb00bd0a0dbf8)

Bluetooth 5 fix default AdvertisingSetParameters values

Having advertiser be scannable and connectable by default is a bad
choice for new advertising set.

Bug: 30622771
Test: sl4a Bt5ScanTest
Change-Id: I4fc270e78ca4e62d3077c5cd28aa59b0518d2e77
(cherry picked from commit 010cc95583d6b93bdcc9cf7959f8470d109f8a3d)

Bluetooth: fix comment wording

Test: manual
Bug: 30622771
Change-Id: I5a589c98553f35248b0d95d332e9f35774075b24
(cherry picked from commit e7f89b07e6fa3a99877af3298c2a80db23a591d7)

Merge "Update string description and char limits" into oc-dev

Expose UserHandle.getIdentifier() to CTS.

This allows CTS to pass user IDs returned by APIs as UserHandle to various
ADB commands.

Test: Exposing as TestApi only; m -j
Change-Id: Iedba6d83b717baacf9e7cf97f1d32f93c191a5ca

Merge "Fix race condition in removeForegroundServiceFlag" into oc-dev

Hide ColorSpace.Renderer

Bug: 37073273
Test: Compile, make sure CTS does not rely on this API
Change-Id: Ia5d2638676be44464a173f904a21158c554b9ebf

Fix screen pinning in seascape

Test: visual
Change-Id: I7792d0f5f98e83f0b2fd57a831e82efa07533c2b
Fixes: 34072199

Merge "Don't ensure config for activity that is about to be invisible" into oc-dev

AAPT2: expose return values through JNI.

Test: Tests for JNI in the studio-master-dev branch.
Change-Id: I709595aabf31c9c2fea659c7af20a5c50d1e3d34

Merge "Remove .(ll)ndk suffix from (ll)ndk_library" into oc-dev

Merge "Move API for disabling an autofill service to the manager" into oc-dev

Import translations. DO NOT MERGE

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

Merge "Revert "ViewRootImpl: Fix child lifetime."" into oc-dev

Revert "ViewRootImpl: Fix child lifetime."

This reverts commit ba29dbc3b38d9efb743d6fa7f9f8c4bb75c38b28.

The original commit causes apps such as phone and clock to not wake the
device.

Fixes: 37197491
Change-Id: I1b45cafbcbf12991d8e1e716e4d1751cd077c556

Merge "ignore RemoteException from pm if shutdown is already on-going" into oc-dev

Revert "Revert "Handle case when snapshot dimensions don't match""

This reverts commit ba53d8ae410976709e1413b74173a791e8dead15.

Also fixes that we always had a size mismatch.

Test: TaskSnapshotSurfaceTest
Test: Open app in different orientation than snapshot, make sure
looks ok.

Bug: 36991071
Change-Id: If572b68fd72cec7679984fdff0be5905caba69f4
Fixes: 36703868

Merge "Ensure the system boots with a halfway sane time/date" into oc-dev

Move API for disabling an autofill service to the manager

Test: the topic has a CTS test

bug:37170723

Change-Id: I2be18968ea8df0fd3fb070fe2efc3e811b0e5a76

Merge "Cache mEphemeralAppsDisabled state" into oc-dev

Merge "OverlayManagerService: Make broadcasts/updates explicit" into oc-dev

ImageReader/Writer: add usage flag support

Also add an ImageWriter ctor to take additional arugment (format)

Test: ImageReader and Writer CTS tests
Bug: 32766711
Change-Id: I99e3862dd5b9a85c9df7879c14c84b68a35718ec

Make sure system UI context has the same LoadedApk as system context

When the system Application object is created, it should be shared by
both system and system UI context.

Bug: 37111478
Test: - Open settings
Test: - adb shell am crash com.android.settings
Test: - with crash dialog open, press volume keys
Test: - observe no crash
Change-Id: I2b4656680d25fe61fee69c01ee10522aac4e2942

Merge "Fix READ_PHONE_NUMBERS label" into oc-dev

Merge "Fix double-detach DualTimer bug" into oc-dev

Deprecated ACTION_PRECISE_DATA_CONNECTION_STATE_CHANGED

This broadcast intent will no longer work for the apps
running in the background. Apps should use Connectivity
Manager to listen for network changes.

Test: Telephony sanity tests
bug: 37156634
Change-Id: I5297cc5a624857d3e11a6d5fdf9369ab99ce88d4

Merge "AAPT2: Share split functionality between link and optimize" into oc-dev

Merge "Fix crash when measuring storage." into oc-dev

Update string description and char limits

Change-Id: Idba5977e614094c3a96b1101f1234f26433300c8
Fix: 36680364
Test: builds

add debug logs

Test: TreeHugger
Change-Id: I4b12acb43aa4af7a5587842fe5e2528fee8f4fed

Fix double-detach DualTimer bug

DualTimer attempted to detach its subTimer twice when reset(true) was
called, once explicitly and once via a call to the main timer. This
fixes that problem by getting rid of the explicit detach in reset.

Bug: 37208694
Test: runtest -x
frameworks/base/core/tests/coretests/src/com/android/internal/os/BatteryStatsTests.java
and manually looked for "Removed unknown observer" error in logcat.

Change-Id: Ic5ff7d799d46236a74ab0825e108bef40bac0360

ignore RemoteException from pm if shutdown is already on-going

- In new shutdown flow, adb is guaranteed to be alive while system server
  is alive.
- power manager can be killed before system server is killed.
- This leads into RemoteException for shutdown request to power manager.
- If shutdown is already on-going (=sys.powerctl set), the RemoteException
  should be ignored.

bug: 37096155
Test: adb shell svc power reboot|shutdown and check output to adb
Change-Id: Ia90a900d55123bd7c1bf57c90c49d35332ebfdf5

Update fontchain_lint to test new emoji font

The new emoji font has been update to Unicode 10/Emoji 5, and has a
new mechanism for flag fallbacks. It also shares glyphs between
gendered emoji and non-genders ones.

Test: make fontchain_lint passes
Bug: 34688862
Bug: 33276970
Bug: 34738377
Change-Id: I4d43b8c777deb93be1e6a784eaa109e2e7febd7d

Add getSelfManagedPhoneAccounts() TelecomManager API.

This is consistent with the existing getCallCapablePhoneAccounts API
which is used to get the phone accounts capable of placing phone calls.

Test: CTS
Bug: 34159263
Merged-In: I1396c63a6882947cd66721528e53b6964a9091e0
Change-Id: I1396c63a6882947cd66721528e53b6964a9091e0
(cherry picked from commit 8931707058c2680878aacc5e6b4150a9aedc0f4d)

Merge "send broadcast to the right users" into oc-dev

Merge "Do not set outActivity if value is null." into oc-dev

Remove .(ll)ndk suffix from (ll)ndk_library

Soong handles these automatically now.

Bug: 33241851
Test: Android-aosp_arm.mk is the same before/after
Test: build.ninja is the same before/after
Test: build-aosp_arm.ninja is the same before/after
Change-Id: Ic10796e75bb886cc3f92c234714ab3e6c85112df

Cache mEphemeralAppsDisabled state

Init mEphemeralAppsDisabled in systemReady when settings provider
is available.

Registered content observers for ENABLE_EPHEMERAL_FEATURE and
WEB_ACTION_ENABLED.

Test: manual triggering enable_ephemeral_feature=1/0
Fixes: 36869938
Fixes: 36895304
Change-Id: I0051c45bb209d86041b967a409f8a95967df21dc

Ensure the system boots with a halfway sane time/date

Just in case the kernel or vendor glue don't initialize the system time
sensibly, we now make sure the current time at boot is at least up to the
build time of the system disk image.

Bug 36736983
Test: manual

Change-Id: I46f776300ca4f5ad01396fc17c83fc833e1713f9

Merge "NetworkPolicyManagerService: fix deadlock" into oc-dev

Merge "Fix activity move between displays" into oc-dev

send broadcast to the right users

Change-Id: Ibda2d922cd58d6ca1c1a4ecdccaa25e5c71c6320
Fixes: 37192695
Test: manual; create restricted profile and see that users haven't been removed

AAPT2: Share split functionality between link and optimize

Generating splits should be possible to do from the optimize command.
This means that a lot of infrastructure around split APKs can be
shared by both the optimize and link phase.

Bug: 35925830
Change-Id: Ia88b9e4bff300a56353b2f7a4a2547c8eb43a299
Test: manual

Merge "Just broadcast intent to bring up MTP/PTP as soon as switching function" into oc-dev

Merge "Make visible only accounts for which authenticator supports contacts operations for callers with READ_CONTACTS permission." into oc-dev

Do not set outActivity if value is null.

Recent changes passed down the nullable array to a method where it
was not being properly checked.

Change-Id: Ibf940f9225e4c9901869188aa55a04d0908015c3
Fixes: 37197666
Test: manual via bug repro steps.

Fix race condition in removeForegroundServiceFlag

Also fix flake in our tests.

Test: runtest systemui-notification
Fixes: 36550043
Fixes: 36551790
Change-Id: I910e9233a17ef325bcb173c50c00d9982dc72224

Fix READ_PHONE_NUMBERS label

Change-Id: If4a254092168d9f7b133a58b59dd946f64cefdae
Fixes:37207265
Test: builds

Don't ensure config for activity that is about to be invisible

We were relaunching an activity in the docked stack that is been
dismissed because we call ensure configuration on it due to the
resize of it been moved to the fullscreen stack. This isn't needed
as the activity is about to become invisible since it is been moved
to a stack that isn't visible.
We now ensuring the configuration if:
- The stack the activity is in shouldn't be visible.
- The activity is currently stopped or stopping.

Change-Id: I6adac6b11a0db4b100e623e4074e169660206567
Fixes: 37118176
Test: Activity in docked stack isn't relaunched when dismissed.

Merge "Add an API to enable disable safebrowsing in WebView" into oc-dev

Merge "Add ICompanionDeviceManager.getAssociation overload with explicit userId" into oc-dev

Merge "Tethering: support Local-only Hotspot mode for downstreams" into oc-dev

Merge "Update decor LayoutParams when window is already added." into oc-dev

Merge "AutoSize TextView - fix measurements" into oc-dev

Merge "VPN: cancel always-on notification with new id" into oc-dev

Tethering: support Local-only Hotspot mode for downstreams

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - "runtest frameworks-net" passes
Bug: 31466854

Change-Id: Ia50e28c8ce0af8cdd7ac63217d921aff213668e7

Merge "Fix mock of setApplicationRestrictions()" into oc-dev

Merge "DngCreator: Populate baseline exposure tag" into oc-dev

AutoSize TextView - fix measurements

The BoringLayout measurements for auto-size are
incorrect. Removed those and kept the StaticLayout
measurements.

Bug: 36940118
Test: cts-tradefed run cts-dev -m CtsWidgetTestCases -t\
      android.widget.cts.TextViewTest

Change-Id: I772ade08ed26efac05ca56cb7df8cfec0327633b

NetworkPolicyManagerService: fix deadlock

Callers of addNetworkPolicy() were not taking locks in the correct order
inside NetworkPolicyManagerService:
  - addNetworkPolicy() is an internal method that calls
    setNetworkPolicies which takes both mUidRulesFirstLock and
    mNetworkPoliciesSecondLock in order.
  - both callers of addNetworkPolicy, mWifiStateReceiver and
    mConnReceiver via ensureActiveMobilePolicy, were taking
    mNetworkPoliciesSecondLock before calling addNetworkPolicy.
    - this causes the order of locking to be reversed, which can cause a
    deadlock when another concurrent codepath in
    NetworkPolicyManagerService tries to take both locks in the correct
    order.

This patch fixes this issue by wrapping both problematic codepaths into
addNetworkPolicy() with a lock on mUidRulesFirstLock.

Test: build, flashed, NetworkPolicyManagerServiceTest passes
Bug: 36972283
Change-Id: If7888c11aef8b628e1b013224075c4c75eae0022

Merge "Fix counting problems in StopwatchTimer." into oc-dev

VPN: cancel always-on notification with new id

Previously we used 0 for the id of the "Always-on VPN disconnected"
notification, as did many other system services. In a recent CL, an
effort was made to assign a dedicated id to each of these system
notifications. Unfortunately, this CL only changed the id in the call to
NotificationManager#notify(), but not NotificationManager#cancel(). This
causes the notification to persist even when always-on VPN is turned off
by the user.

This CL changes the id in the cancel() call, thus fixing the issue.

Bug: 37175930
Test: manually enable and disable always-on and observe the notification
Change-Id: I39c7325c78479afe3dc9e1764d54406bd721b067

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 "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 "Import translations. DO NOT MERGE" into oc-dev

Merge "Restore session on new window after app kill" into oc-dev

Merge "Start service from libscheduleservicehidl." into oc-dev

Import translations. DO NOT MERGE

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

Merge "Use feature instead of hidden config for multi-display" into oc-dev

Merge "FingerprintSettings for non-owner should respond to fingerprint touches" into oc-dev

Add an API to enable disable safebrowsing in WebView

WebView safebrowsing can be opted in using a manifest value. However,
we also need to control individual WebViews.

Bug:37158813
Test: See change I71e813bccc2fab73d100384661128c7311dd396c

Change-Id: I647dc304787d6406691b5cbadf1c9a4f13ac5604

Fix activity move between displays

1. ActivityConfigCallback might not have been registered
because DecorView was not yet attached to window and ViewRootImpl
was not available. In this CL the callback is set as soon as a
DecorView is attached to window.
2. When private display was removed from system, its stacks were
moved to bottom in AM but moved to top in WM.
3. When reparenting stack visibility of activities should be updated
before reparenting in WM, because otherwise WM will be resizing
windows that should no longer visible and reporting it to clients.

Bug: 34164473
Test: android.server.cts.ActivityManagerDisplayTests
Test: #testOnMovedToDisplayCallback
Test: #testContentDestroyOnDisplayRemoved
Change-Id: I6ccc27d873d0d60d7650659fb25cbfcaaeb0fd07

Merge "Enable profiling for system server for userdebug|eng" into oc-dev

Enable profiling for system server for userdebug|eng

Only for these two configurations since selinux permissions only
allow system_server to access profiles in these cases.

Motivation is to allow collecting profiles for improving RAM and
performance for preopt.

Setting the property dalvik.vm.profilesystemserver to true is required
to enable the collection of profiles for system server.

Test: device boots

Bug: 34927277
Bug: 36457259

Change-Id: Icbba6dafe376fd9c6c3d0c7feba7f64009f7c2c3

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Merge "Make a11y node info parceling more robust" into lmp-mr1-dev am: 88698afd01 am: 7cb5e1dfd1 am: d7552dc819 am: e3a1b81c8f am: 879bdbeb37 am: f580a2c257 am: 68a93d2be2 am: c7db1bf43a am: 593df9d186 am: 92e6a1d941
am: fbc531077f

Change-Id: I391a1599854997edc52e704ab75a976d7ba567b3

Restore session on new window after app kill

Also:
- Give the session an integer ID as the activityToken is not stable over
  restarts of the activity
- Verify that session is only accessed by one UID
- stabilize AccessibilityViewIds over activity lifecycle at least for
  the IDs we can do that. This required to split the ID namespace in
  "per-app" and "per-activity" views. Only the later ones can be
  restored.
- Do not end session when app is killed (as it can be restarted)

Bug: 35484143
Fixes: 36392498
Test: cts-tradefed run cts-dev -m CtsAutoFillServiceTestCases --test=android.autofillservice.cts.SessionLifecycleTest
      cts-tradefed run cts-dev -m CtsAutoFillServiceTestCases
Change-Id: I229acc1b3ce35fb57262da7d7466b5d4328b49d4

Import translations. DO NOT MERGE

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

Merge "Make a11y node info parceling more robust" into oc-dev