Merge "Fix data disabled icon from not showing in status bar"

Merge "Update fill UI"

Fix data disabled icon from not showing in status bar

The NetworkController seemed to think the user wasn't set up since
the DeviceProvisionedController didn't send an initial callback
on add.

Also add some extra listening to data disabled state to avoid race
conditions there.

Change-Id: I17f759894a283da08eb0da7ceb5181355c8f4d55
Fixes: 35708363
Fixes: 35632846
Test: runtest systemui

Merge "Revert "Don't trigger auto-fill request if mode is manual""

Merge "Fix an unintentional switch-case fall-through"

Merge "Add metric for new storage screen."

Merge "Ensure we use the fullscreen stack bounds when expanding PiP."

Merge "Have VirtualDisplayDevice handle Binder death directly."

Merge "Temporarily whitelist apps known to hit the wtf case"

Merge "Disallow parallel backups"

Merge "Serialize restore operations"

Merge "Correct SurfaceControl matrix parameter names."

Update fill UI

1. Abstracted the fill/save view and window management
   in dedicated classs

2. Avoided the need of a second window to detect outside
   touches

3. Simplified the fill-ui window management

4. Moved the UI in its own package to ease mmigration to
   sys UI.

5. Removed hard-coded colors from the layout

6. Make sure the save UI cannot grow as wide as the screen
   as this would not look good on tablets

7. Update the save UI to look closer to mocks

Test: CTS tests pass
Bug: 35708258

Change-Id: Ia74a5aad6f16bba0047a9e8e61958c77af0d358d

Merge "Add isBundled argument to LoadedApk.makePaths call"

Merge "Add autofill service to BridgeContext"

Merge changes from topic 'skia_merge_1933880'

* changes:
  Fix HWUI tests to use android-only SkPaint flags
  Update framework to use new method for disabling DF path rendering.

Revert "Don't trigger auto-fill request if mode is manual"

This reverts commit 576a847e03f213693f4e7f2f0361f52217f9e1cf.

Fixes: 35701500
Change-Id: I972c2d0e56d365d1fad76a66e74bfad227bf26c4

Merge "Expose isStateSaved() in FragmentManager."

Merge "Added VSYNC and INTENDED_VSYNC timestamps to FrameMetrics API."

Merge "BitmapFactory: Remove legacy premultiply hacks"

Merge "Added sanitization for CompoundButton and RadioGroup."

Merge "Display longer date string in certificate information"

Have VirtualDisplayDevice handle Binder death directly.

Previously, the VirtualDisplayDevice delegated its cleanup to the
parent VirtualDisplayAdapter. The adapter looked up the device
from an AppToken mapping. It is possible for this mapping to not
include the token, leading to the device not being destroyed.

This changelist has the VirtualDisplayDevice take care of calling
destroy on itself. It still informs the adapter of this event for
bookkeeping/cleanup purposes.

Fixes: 26221019
Test: Verified through adb shell dumpsys activity activities that the
      virtual displays were removed upon app kill after the
      MediaProjection for these displays ended.

Change-Id: I500fdc87618632184f5d3c9821ccf1979f460595

Add autofill service to BridgeContext

Test: layoutlib tests
Change-Id: I52b360a64518165eefee4250b1c32e9f5f87845d

Merge "Update DisplayAdjustments in Display from Resources."

Fix HWUI tests to use android-only SkPaint flags

Test: compile only
Change-Id: Ia3a5710d4ae13d68e5b2655a8b135dae8ffeb513

Merge "Deprecate TestSuiteBuilder and related classes." am: a26000b8f5 am: fd61eb1e39
am: 9dcdc8f187

Change-Id: Icbc80cc054244f2660814bde4a9dc681b501024f

Merge "Fix build failure when using javac toolchain" am: c098bea921 am: 5da6f4fbd1
am: df8bb0ec52

Change-Id: If59986dd60ca90836adecdf0cb52f3570bf81f9d

Merge "Deprecate TestSuiteBuilder and related classes." am: a26000b8f5
am: fd61eb1e39

Change-Id: I23f137865997ca9ce4aa229c9329d8ea9fbcb6ee

Merge "Fix build failure when using javac toolchain" am: c098bea921
am: 5da6f4fbd1

Change-Id: I65623ab75b0cc7b828fe67544074549fc21206ca

Merge "Deprecate TestSuiteBuilder and related classes."
am: a26000b8f5

Change-Id: Ia9d491bd72f2a9f08838e298aaf4de5de28c7d4f

Merge "Fix build failure when using javac toolchain"
am: c098bea921

Change-Id: I56e965336c537295efc4c0c778f5c236f955bf10

Merge "Deprecate TestSuiteBuilder and related classes."

Merge "Fix build failure when using javac toolchain"

Update framework to use new method for disabling DF path rendering.

Test: compile only
Change-Id: Ifa690a0af01cb6b07c4fd303302cb6a557e8dfde

Add isBundled argument to LoadedApk.makePaths call

There is a disconnect between the way webview created
classloader and the way makePaths decides if paths are
intended for bundled app.

This change moves decision making out of makePaths method
which allows WebViewZygote to pass correct argument and
have makePath omit java.library.path for libPaths

Bug: http://b/35426785
Test: manual
Change-Id: Iab5a18c0091d0193dafa750498eb00f378411ba0

Merge "Revert "OMS: integrate OverlayManagerService into framework""

Revert "OMS: integrate OverlayManagerService into framework"

Bug: 31052947
Bug: 35697944

This reverts commit 21a3d1ad686dee97b9cf0ed80389ee2ab0d48013.

Change-Id: I2d86931020301524c26cf8c8e80d557c97fdd6c3

Merge "IP connectivity metrics: cleanup obsolete code"

Merge "Added auto-fill support for Spinner."

Merge "Don't trigger auto-fill request if mode is manual"

Merge "Fix toggling of AM/PM of TimePicker using hardware keyboard."

Fix an unintentional switch-case fall-through

This is a follow up CL to my previous CL [1], which introduced a
classical fall through bug due to a missing return statement.

 [1]: Ic614f112f960382280acd8891b3af56d47679f08
      ce82eb2d865e5ee1b69705ca7a0aedd0e92d0d4e

Test: Manually made sure that calling
      InputConnection#performPrivateCommand() no longer causes
      IllegalStateException("Already recycled.").
Bug: 35079353
Fixes: 35681994
Change-Id: I9aa3c0497385e053718d829aad49495771b22b59

Merge "Optimize IMMS.MyPackageMonitor more"

Merge "Remove abstract from AccessibilityService methods"

Merge "VideoView: option for audio focus, support for AudioAttributes"

Merge "Don't copy ringtones when profile sync goes off"

Merge "renaming telephony internal hidden intents" am: 09c663a84c am: 979e4690ec
am: 201320492c

Change-Id: I4e37359a98fb8ecdff63b45a54f08b61e7be25ad

resolve merge conflicts of ac9b2c4fab42 to stage-aosp-master am: 4fb8988854
am: 609cfc916b  -s ours

Change-Id: Ib912a18c3bea70d268cc6fa8e3de8bbb9fd414b0

Merge "OMS: integrate OverlayManagerService into framework"

Merge "resolve merge conflicts of f17c1d488840 to master"

Merge "Reconcile apps in 2 phases"

Merge "Add queryStatsForPackage() API."

Merge "Start locking down /data/media access."

Merge "Slowly deprecate hidden getPackageSizeInfo() API."

Merge "Fix concurrent modification crash in onAlarm"

Added sanitization for CompoundButton and RadioGroup.

Bug: 33269702
Bug: 33550221
Test: CtsAutoFillServiceTestCases (with new tests) pass

Change-Id: Ie2c8d2784227371588aa02973b8ef3ac1a6950aa

Merge "Add missing sound to important channels."

Don't copy ringtones when profile sync goes off

Experimentally, it makes more sense to more people to have the parent
setting as an overlay not a concrete thing.

Test: make cts -j30 && cts-tradefed run cts --module CtsDevicePolicyManagerTestCases --test 'com.android.cts.devicepolicy.ManagedProfileTest#testRingtoneSyncAutoDisableRingtone' </dev/null 2>&1
Bug: 34730524
Change-Id: I5f804713def9e54921b90e4f5cea742ba8aaa685

Merge "Add "Use open Wi-Fi automatically" setting."

Merge "context_hub HAL uses "default" service name"

Merge "renaming telephony internal hidden intents" am: 09c663a84c
am: 979e4690ec

Change-Id: I8cce065cc9ce6a87fa0ce9753a5e9170da77da59

Merge "renaming telephony internal hidden intents"
am: 09c663a84c

Change-Id: Ic20ae9e6816d629c979859a7d18f1827bafcc034

Optimize IMMS.MyPackageMonitor more

This CL is a follow up CL to my previous CL [1] that aimed to reduce
false positives in InputMethodManagerService.MyPackageMonitor when
deciding if InputMethodManagerService (IMMS) needs to rebuild the list
of enabled IMEs or not.

Currently IMMS.MyPackageMonitor#onSomePackagesChanged() gets called
back to trigger IMMS#buildInputMethodListLocked() when either the
following rule A or B is fulfiled.

 A. Intent with one of the following actions for any package name:
    - ACTION_PACKAGE_ADDED
    - ACTION_PACKAGE_REMOVED
    - ACTION_EXTERNAL_APPLICATIONS_AVAILABLE
    - ACTION_EXTERNAL_APPLICATIONS_UNAVAILABLE
    - ACTION_PACKAGES_SUSPENDED
    - ACTION_PACKAGES_UNSUSPENDED
 B. ACTION_PACKAGE_CHANGED with a package that is included in the
    known IME package list, which can be obtained from PackageManager.

The previous CL [1] addressed Bug 28181208 by introducing the rule B,
but we can actually apply the same optimization for A, except for one
false negative case where an appearing package that is not in the
known IME package list actually contains one or more
InputMethodService.

In short, we can reduce false positives by replacing the above two
rules with the following two rules.

 A. Intent with one of the following actions for any package name that
    is in the known IME package list:
    - ACTION_PACKAGE_ADDED
    - ACTION_PACKAGE_CHANGED
    - ACTION_PACKAGE_REMOVED
    - ACTION_EXTERNAL_APPLICATIONS_AVAILABLE
    - ACTION_EXTERNAL_APPLICATIONS_UNAVAILABLE
    - ACTION_PACKAGES_SUSPENDED
    - ACTION_PACKAGES_UNSUSPENDED
 B. Intent with one of the following actions for any package that
    implements at least one InputMethodServivce.
    - ACTION_PACKAGE_ADDED
    - ACTION_EXTERNAL_APPLICATIONS_AVAILABLE

Basically in the rule A PackageManager gives us the list of relevant
package names that might contain IMEs regardless enabled/disabled
state, and such a list works well to filter out irrelevant
notifications except for one case where a new package is adding one
or new IMEs that we did not know. This is why we also need the rule B.

Even though the rule B requires a secondary query to PackageManager,
it can be done outside of the state lock of IMMS.

 [1]: I7b69c349318ce06a48d03a4468cf2c45bfb73dc2
      c4e4491735ad5614ec4592fae98f05c455f5944d

Test: Manually verified as follows.
       1. tapas ShortcutDemo && make -j
       2. Copy ShortcutDemo.apk to the current directory.
       3. adb shell dumpsys input_method | grep mMethodMapUpdateCount=
           to check the "mMethodMapUpdateCount".
       4. adb install -r ShortcutDemo.apk
       5. adb shell dumpsys input_method | grep mMethodMapUpdateCount=
           to make sure "mMethodMapUpdateCount" remains unchanged.
       6. adb install -r ShortcutDemo.apk
       7. adb shell dumpsys input_method | grep mMethodMapUpdateCount=
           to make sure "mMethodMapUpdateCount" remains unchanged.
       8. adb uninstall com.example.android.pm.shortcutdemo
       9. adb shell dumpsys input_method | grep mMethodMapUpdateCount=
           to make sure "mMethodMapUpdateCount" remains unchanged.
Test: Manually verified as follows.
       1. tapas SoftKeyboard && make -j
       2. Copy SoftKeyboard.apk to the current directory.
       3. adb root
       4. adb install -r SoftKeyboard.apk
       5. adb shell dumpsys input_method
           Make sure that
             com.example.android.softkeyboard/.SoftKeyboard
           is recognized by IMMS.
       6. adb shell pm disable com.example.android.softkeyboard/.SoftKeyboard
       7. adb shell dumpsys input_method
           Make sure that
             com.example.android.softkeyboard/.SoftKeyboard
           is no longer recognized by IMMS.
       8. adb shell pm enable com.example.android.softkeyboard/.SoftKeyboard
       9. adb shell dumpsys input_method
           Make sure that
             com.example.android.softkeyboard/.SoftKeyboard
           is recognized by IMMS again.
Fixes: 35361128
Change-Id: I063688297156188f68fe0b55a46d72f2e811dc88

Merge "renaming telephony internal hidden intents"

Fix concurrent modification crash in onAlarm

Change-Id: Idfd094f3c9ea59356440d6851ccd5abda36ca6ba
Fixes: 35640585
Test: manual; after boot, opened the power menu to force
system_server to spin up a ThreadedRenderer instance
(and thus register itself as a callback on GraphicsStatsService).
Then manually set the date forward by a day to trigger onAlarm
and verified the system didn't reboot/crash. A systrace capture
verified that the alarm fired and package:android (system_server)
had a log rotation event.

Fix toggling of AM/PM of TimePicker using hardware keyboard.

Set AM/PM mode before setting hour in TextInputTimePickerView. This is
needed so we know exactly which it is when we are using 12 hour clock.
Not setting it early was causing TextInputTimePickerView to update TimePicker
with the hour that was not offset correctly based on AM/PM state.

Test: existing tests caught this - TimePickerTest#testKeyboardInputModeClockAmPm
      verified locally that this test now passes.
Bug: 35262830

Change-Id: I7e912320cb5a72c5d9d4e80dc0b42deb651f2636

resolve merge conflicts of ac9b2c4fab42 to stage-aosp-master
am: 4fb8988854

Change-Id: Ic679f452ad245cbe58ea8069c34c15e50ee9e5c2

Add metric for new storage screen.

Bug: 34715777
Test: None here -- used in topic.

Change-Id: Icffecd5abba197d544a1f9bd382d5c09b0bbb388

Merge "Fix build."

Added auto-fill support for Spinner.

Spinner is a special kind of list-value field because its auto-fillable
values are not currently present in the assist structure: this change
adds a getAutoFillOptions() to fill that void, and implements the
auto-fill APIs on Spinner.

It also fixes RadioGroup.getAutoFillValue() - it should return the index
of the child, not its resource id.

Bug: 33550221
Test: CtsAutoFillServiceTestCases (with new tests) pass
Change-Id: I625e7dd705fade56fda490fcd3c4446c0c245ee1

resolve merge conflicts of ac9b2c4fab42 to stage-aosp-master

Change-Id: I4a6094c41ab267f89a15c5e8caa25e6b683419d0

Merge "Fix a memory leak in AutoFillManager"

Merge "TelephonyManager#setAllowedCarriers: comment update"

Merge "Fixed missing auto-fill save callbacks on TextView."

Reconcile apps in 2 phases

During boot app data folders are reconciled in 2 phases:
 - in the constructor only core apps are reconciled. prepareAppData
   for remaining apps is deferred and run on a separate thread (phase 2)
 - Phase 2 must finish before third-party apps can start

Also moved GC to final stages of system server init. GC alone takes ~200 ms.

Overall boot time improvement: ~1 second

Before:
02-17 18:33:33 D/BaseBootTest: successive-boot :
28835.0,29638.0,30205.0,29793.0,29752.0,28228.0,30125.0,28983.0,28487.0,28865.0,
02-17 18:33:33 D/BaseBootTest: successive-boot_avg : 29291.1
02-17 18:33:33 D/BaseBootTest:
SystemServerTiming_StartPackageManagerService :
3150.0,3615.0,3515.0,3495.0,3814.0,3158.0,3746.0,3274.0,3222.0,3607.0,
02-17 18:33:33 D/BaseBootTest:
SystemServerTiming_StartPackageManagerService_avg : 3459.6
02-17 18:33:33 D/BaseBootTest: SystemServerTiming_StartServices :
8244.0,8863.0,9035.0,9832.0,8998.0,8096.0,8719.0,8209.0,8279.0,8754.0,
02-17 18:33:33 D/BaseBootTest: SystemServerTiming_StartServices_avg :
8702.9

After:
02-17 17:59:51 D/BaseBootTest: successive-boot :
27711.0,27607.0,28408.0,28968.0,28397.0,28063.0,27885.0,28483.0,27917.0,29317.0,
02-17 17:59:51 D/BaseBootTest: successive-boot_avg : 28275.6
02-17 17:59:51 D/BaseBootTest:
SystemServerTiming_StartPackageManagerService :
2467.0,2489.0,2369.0,2548.0,2647.0,2523.0,2497.0,2553.0,2482.0,2657.0,
02-17 17:59:51 D/BaseBootTest:
SystemServerTiming_StartPackageManagerService_avg : 2523.2
02-17 17:59:51 D/BaseBootTest: SystemServerTiming_StartServices :
7686.0,7538.0,7598.0,7869.0,7884.0,7950.0,7971.0,8370.0,7696.0,7885.0,
02-17 17:59:51 D/BaseBootTest: SystemServerTiming_StartServices_avg :
7844.7

Test: manual
Bug: 28750609
Change-Id: I3543ef577af1365394775318e40907584ddbe950

Merge "Add a flag to enable/disable the cache quota calc."

resolve merge conflicts of f17c1d488840 to master

OCL: http://ag/1932572

Test: refactoring CL, with unit tests

Change-Id: I5da27ee70afb8bf6e5fe9f6f3b545cd26d5c8d20

Slowly deprecate hidden getPackageSizeInfo() API.

It was never a public API, and we now have a support and CTS tested
public API over at StorageStatsManager that apps should be using.

Test: builds, boots
Bug: 34643504
Change-Id: Iad3f2c000dd9d7c1b1e70d0bf2d281826737322e

Fix build.

Test: make -j48 tests
Change-Id: I51287ddae3c7c689a44ca45edbae838e90b88e98

Merge "Add UI prompt for MMI code failure on 3gpp"

Temporarily whitelist apps known to hit the wtf case

Bug: 34973771
Test: Manual tests with hangouts

Change-Id: If456649cb90164562cd71503119625454590d60d

Add queryStatsForPackage() API.

This gives developers a way to collect package-level stats, even if
it means we have to use manual calculation for sharedUserId apps.

Also round size of storage devices to nice power-of-two values so we
do a better job of matching retail packaging.

Test: builds, boots
Bug: 35294241
Change-Id: I24946c443bb9dc4b0411a8149a0656702ac1fd24

Start locking down /data/media access.

The new sdcardfs filesystem requires that we have stricter access
controls around /data/media style locations.  Start by taking away
the "media_rw" GID from apps requesting the WRITE_MEDIA_STORAGE
permission.

Common use-cases like music playback appear to continue working fine,
as clients should only be attempting to use /data/media paths after
calling maybeTranslateEmulatedPathToInternal().

Test: builds, boots, music playback works
Bug: 35447080
Change-Id: Iba9f3ef41d3277c75497f675a1fe6d3406cf4542

Replace WearWifi/Cell services with WearConnectivityService am: 5555bccc70
am: abfd5bccb9

Change-Id: If051c765d3ea81cce2bedc2dfa02d64744088054

Merge "Revert "Ensure network connectivity on app start.""

Replace WearWifi/Cell services with WearConnectivityService
am: 5555bccc70

Change-Id: I5d8c97b8d5c394aeae99b7ddbe0dfdd5706a4293

BitmapFactory: Remove legacy premultiply hacks

Skia now has a mode that will perform color correction,
but still do the premultiply on gamma encoded bytes.

Test: Flashed device, ran cts.

Change-Id: Ib1d1ea1074b5e6d1a49ecfc45e25552643dc04bc

Add "Use open Wi-Fi automatically" setting.

- Add constant in NetworkScoreManager for the meta-data key required
for NetworkRecommendationProviders to specify which package provides
this feature.
- Add Setting to specify which package is enabled for providing this
feature.

Bug: 34773276
Test: make
Change-Id: I3f8209c21b8b219c242650f97ba407b5985a5250

Revert "Ensure network connectivity on app start."

This reverts commit 27e693158398059d00ff20e60640cb105e255625.

Change-Id: I4c41ce7c1839d3b91931bc8a8f117f32bf6cd381

Merge "Revert "Notify NPMS when uid becomes active or idle.""

Merge "Revert "Set default wait-for-network timeout to 0.""

Merge "AAPT2: Rename strip phase to optimize"

Merge "Don't show the percentage twice when charging."

Revert "Notify NPMS when uid becomes active or idle."

This reverts commit ac0a61bad30e5a2ff45a85aaa6cf910e08911a2b.

Change-Id: I87d239cef51fc00567054644efceeca1ad2b463d

Revert "Set default wait-for-network timeout to 0."

This reverts commit d4f081cca4aa2b3b0fd1065839aedc1587dac94a.

Change-Id: Id55a08d00119c048c3d738b9f6f2cc5626df9bbf

Merge "AAPT2: Shared library support"

Remove abstract from AccessibilityService methods

Two methods on AccessibilityService were declared abstract, but
there doesn't seem to be a good reason for them to be abstract, as
it's possible to write an accessibility service that doesn't do
anything useful with the methods. Removing the abstract qualifier
to reduce boilerplate for service developers.

Bug: 31705365
Test: Trivial change, so I just verified that it builds.
Change-Id: I2af39848c8ab343d562361fdc1c3f8ca3ecb5f00

Merge "Updated some internal apps to use TYPE_APPLICATION_OVERLAY"

Merge "Ensure that we tint PiP actions to be consistent."