Fixed memory leak with the inflater

Because min priority children could be removed from
their parents after the removal, a new inflation task
would be started, leading to the view being instantly
readded again. This lead to memory leaks.

It also fixes a bug where the inflation would not inflate
enough views that could lead to crashes / wrong layouts.

Finally there was a indexing error when handling removal
of group summaries.

Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/NotificationInflaterTest.java
Change-Id: Iac242946bd30060967ee7877560d40e63f39f996
Fixes: 62067645

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

Merge "Couple autofill fixes." into oc-dev

Merge "Allow users to quickly focus the minute field in TimePicker." into oc-dev

Merge "Fix recent apps in system UI for Skia pipeline" into oc-dev

Merge "Introduce result ranges for activity start return results." into oc-dev

Merge "OMS: Setup state for users on boot and when added" into oc-dev

Merge "Add new reason code for IMS Call ended during conference merge process" into oc-dev

Merge "Revert "Respect PopupWindow.setOverlapAnchor when above anchor"" into oc-dev

Merge "Add missing break statements in FingerprintManager" into oc-dev

Merge "Updating expand icon asset." into oc-dev

Merge "Fix broken PackageParserTest" into oc-dev

Merge "AAPT2: Allow <uses-permission-sdk-23> in manifest" into oc-dev

Merge "TextView: Invalidate selection action mode on "select all"." into oc-dev

Merge "Fixed an issue where the media notification wouldn't have contrast" into oc-dev

Fix recent apps in system UI for Skia pipeline

Enable HW Bitmaps for Skia pipeline just enough to make
recent apps list working by adding support for BitmapShader.
Drawing HW bitmaps in a canvas is also supported.

Test: recent apps work, HWUI unit tests pass, CTS tests pass.
bug: 38136140
Change-Id: Ibd06c859c86dc213310d5ce5272497e1882d0cc6
Merged-In: Ibd06c859c86dc213310d5ce5272497e1882d0cc6

Merge "strong fp lockout after 20 failed attempts" into oc-dev

Merge "Do not perform backup/restore of instant apps" into oc-dev

TextView: Invalidate selection action mode on "select all".

Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
bit CtsWidgetTestCases:android.widget.cts.TextViewTest
Change-Id: I0b5ebb8d6f1af1a9938151f758a2feedb14fcb9f
Fixes: 38244876

Import translations. DO NOT MERGE

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

Merge "Add command to retrieve the instant app resolver" into oc-dev

Merge "Properly invalidate text classification cache." into oc-dev

Merge "Added redundant focusable attribute read in TextView" into oc-dev

Introduce result ranges for activity start return results.

The return result from starting activity actually represents two
pieces of information. First, it conveys whether the activity
started. Secondly, it conveys whether there was a fatal error if
the activity did not start. Many parts of the code assume that a
value greater than or equal to the defined success means that the
activity successfully started. This is not the case as there are a
number of results greater than success where the activity does not
start.

This change addresses the issue by introducing three distinct
result ranges. The first represents results where the activity
could not start due to a fatal error. The second represents results
where the activity did not start due to a non-fatal error. The last
range represents successful activity starts. Two convenience methods
have been added to ActivityManager to return whether the result
represents a fatal error and whether the result was successful.

Change-Id: Ifaf844c353641a28b03b3c2d7b6be053fd9b8b44
Fixes: 38021882
Test: cts-tradefed run cts-dev --module DevicePolicyManager --test com.android.cts.devicepolicy.DeviceOwnerTest#testLockTask_deviceOwnerUser

Merge "Update ShortcutManager/Service javadoc/impl around sizing" into oc-dev

Merge "TextClassifier: Ensure url scheme is in lowercase" into oc-dev

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

Merge "Work on issue #38242094: Activity manager oom adj computation seems broken" into oc-dev

OMS: Setup state for users on boot and when added

Some users never get switched to (managed profile/work profile) so the
overlay state for a user would never be setup (but they could still show
UI and apps).

This change ensures that user state is setup after an OTA to android O,
and whenever a user is added.

Bug: 37899201
Test: manual (add user via Device Admin sample: vendor/google/tools/DeviceAdminSample)
Change-Id: If214e26e39b18c2861794baf5c608a47d536e5ff

Couple autofill fixes.

- Fixed PFLAG3_NOTIFY_AUTOFILL_ENTER_ON_LAYOUT value - it was overlapping with
PFLAG3_ASSIST_BLOCKED.
- Added missing permission on methods called by Shell command.
- Improved debugging.

Bug: 38173625

Test: android.autofillservice.cts.MultipleFragmentLoginTest#loginOnTwoFragments

Change-Id: Ie7880c97d38298e06134709c5c2e9e38fa1bacbb

Adding AOSP copyrights header

Test: N/A

Change-Id: Iabec646a1f8087480f899c5b4996122a9f4bd13a

AAPT2: Allow <uses-permission-sdk-23> in manifest

Bug: 38269610
Test: build tools/aapt2/integration-tests/AppOne
Change-Id: I1d0b87aa36f6bf82aab146339acf8848166b218d

Merge "Updating AppWidgetServiceImplTest as per Mockito changes" into oc-dev

Updating expand icon asset.

Bug: 38237934
Test: Launch activity in PIP, tap to show expand icon
Change-Id: Icae85ff27accd1e2143836541587e3bedc17570b

Add missing break statements in FingerprintManager

Fixes: 38268065

Test: manual

Change-Id: I64ac92b2c5682bcaa494f43ff5defdf9915c5a65

Merge "Batterystats BLE results counter takes in count" into oc-dev

Merge "Fixed exceptions during test tearDown" into oc-dev

Merge "Setting Session commited broadcast when an existing apk is enabled for a different user" into oc-dev

Do not perform backup/restore of instant apps

This keeps the install flow fast, and also happens to mean that
there is feature parity in this regard between pre-O and O+
instant app handling.

Fix #38263351
Test: manual

Change-Id: I8eb08c12d800f5288583e792621920c9876efdfc

Merge "SystemUI: add NETWORK_SETTINGS permission" into oc-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 38120923
Change-Id: I3ebe901771f444c2986b15a52f4d16bd342fac37

Merge "Merge "Add PID info into ANR crash log header" am: 2fe972c822 am: 5d57502c89 am: 5dbe26f0f1 am: 4ae323995d" into oc-dev

Merge "Fix JobSet getJobsByUser" into oc-dev

Work on issue #38242094: Activity manager oom adj computation seems broken

This shouldn't change any behavior, but improve the reason we
report for each process's oom_adj / proc state level.  There are
two significant things going on here:

1. We now consider a bump up in proc state to be just as significant
as a change in oom_adj, and accordingly update the reason when this
happens.  Given how many proc stats we now have mapped to some of
the single oom_adj levels, this matches your expectation to have the
reason be why it is at that proc state and not just some random
other thing at that level.

2. There is special handling of an app at the top state connecting
to another app, deciding the actual state to apply at the end.  This
was not at that point updating the reason, so anything the top app
is connected to would get the top state but myseriously have some other
reason, looking very broken.  We now propagate the reason over.

Test: manual

Change-Id: Idecbe206d73e7c4cbd989ef6faf3b1679e06c088

Allow users to quickly focus the minute field in TimePicker.

Set imeOptions to be actionNext for the hour field of TimePicker to
allow users to jump to the next field using the keyboard.

Bug: 36863243
Test: Manual, keyboard now shows next button instead of enter that moves
      focus to the next field.

Change-Id: Ia9d6137d14579ba5b99fbb714a78423cc75b5fa0

Merge "Set notification TAG and style in TV PIP notification" into oc-dev

Merge "Avoid activity leak via Companion callback" into oc-dev

Merge "Removed deprecated APIs." into oc-dev

Merge "Update Bitmap#prepareToDraw doc" into oc-dev

strong fp lockout after 20 failed attempts

Fingerprint currently locks out for 30s after every 5 failed attempts.

This change makes it so that in addition to the above,
PIN/pattern/password will be required after 20 failed attempts.

Fixes: 35954316

Test: manual
Change-Id: I3aeb0e5b5b4d5011555f60fbe6cc2c1ce702a670

Merge "Do not hold WM lock while closing animation transaction" into oc-dev

Merge "Only force reset when adb is not enabled (i.e. mtp is)" into oc-dev

Merge "Do not notifyNoFullUi if no view is focused" into oc-dev

Merge "Remove forward matches before checking for default" into oc-dev

Merge "Not specify return val of View#onSaveInstanceState" into oc-dev

Merge "Default grant mic, storage, camera to SMS app" into oc-dev

Merge "Prevent multiple copies of android.hidl.manager-V1.0-java" into oc-dev

Fix JobSet getJobsByUser

Retrieve jobs from SparseArray with index using valueAt rather than get
(which uses keys not indices)

Fixes: 38198221
Test: manual

Change-Id: I636f5e0ef43456619700e27944eb6c2d65021643
Signed-off-by: Michael Wachenschwanz <mwachens@google.com>

Update ShortcutManager/Service javadoc/impl around sizing

to account for AdaptiveIconDrawable type of shortcuts

Test: builds for now.. If Makoto is happy with this change, will add tests

b/37743074

Change-Id: Ieecd95d621db57f858fac449d3bcedd173c8607c

Merge "Fix racy clone" into oc-dev

Merge "Add Save and Restore of BluetoothOn setting" into oc-dev

Merge "Fixing the colorization of legacy media notifications" into oc-dev

Merge "Don't flash colors when hinting" into oc-dev

Merge "Fixing flakey tests" into oc-dev

Merge "Don't set keyguard state to showing when service disabled" into oc-dev

Merge "Add time spent by uid per cpu frequency to batterystats." into oc-dev

Add Save and Restore of BluetoothOn setting

This change will automatically save the Bluetooth On setting when
the user chooses to backup the phone settings into the cloud. This
setting is restored by the Setup Wizard (SUW) when configuring the
phone and this change will enable or disable the Bluetooth based
on this restored setting.

Bug: 35657817
Test: Manual test with Sailfish
Change-Id: Ie4518593af63f96f8c363f98941ca5260a3ec4bb
(cherry picked from commit 767f05feea67e642a76bd3e2e7633a8f5273f077)

Fixed exceptions during test tearDown

- Acquire WM lock when tearing down test. Prevent the handler thread
from modifying state at the same time.
- Don't set surface boundaries on window states without a surface
control.
- Avoid exception if process can read screen-edge-insets for Pip.

Change-Id: If4a2438032866eaec84ed79c30c1d716e0f89758
Fixes: 38113905
Test: this is it!

Revert "Respect PopupWindow.setOverlapAnchor when above anchor"

This reverts commit 07084924be4ce0dc67f150d77b886f21fcb01aa7.

Bug: 37951656
Test: android.widget.cts.PopupWindowTest
Change-Id: I6c950d19bbf5c778f799a652595550a2733aae08

Merge "More QS theme fixes for contrast" into oc-dev

Removed deprecated APIs.

Test: CtsAutoFillServiceTestCases pass
Test: manual verification with existing service

Bug: 37563972

Change-Id: Ibea2810f9230f06e59d2d81e4ba853de54987956

Default grant mic, storage, camera to SMS app

Test: CTS tests updated in same topic

bug:36713956

Change-Id: I496d8f8dac86594956dcdeb5afafa5b210e28de4

Merge "Make sure fd is always initialized" into oc-dev

Fixed an issue where the media notification wouldn't have contrast

Because we were relying on the output to go in the right direction
already, this could be wrong. We're now only following the given
lightness direction if it is even possible to satisfy contrast.

Test: runtest -x core/tests/coretests/src/android/app/NotificationTest.java
Change-Id: I06d934a6b5c328ebbf0cf707030b0d707ccb5ab4
Fixes: 38182819

Added redundant focusable attribute read in TextView

Some apps were relying on TextView favoring focusable over
focusableInTouchMode when they were explicitly set to
opposite values in XML (usually because they start with an
EditText and only set focusable=false). This behavior is
undefined (and is, in-fact opposite to every other View).
In order to keep backwards-compatibility, this restores
the old behavior.

Bug: 36497135
Bug: 37916052
Test: Tested those apps and they behave like they used to.
      TextView CTS still passes.

Change-Id: I65bd1b343a6dfb087f41c9fc8af4b5c1e4c71493

Setting Session commited broadcast when an existing apk is enabled for
a different user

Test: Manual tests and CTS
Bug: 38204385
Change-Id: Ic3bddbc2e6070c11b11f0685077db7737f2b8849

Updating AppWidgetServiceImplTest as per Mockito changes

Bug: 38251013
Test: adb shell am instrument -e class com.android.server.appwidget.AppWidgetServiceImplTest \
 -w com.android.frameworks.servicestests/android.support.test.runner.AndroidJUnitRunner

Change-Id: I847335e1c5f9b720b2eecbac1b89b95d9b695e47

Merge "Remove lots of log spew" into oc-dev

Merge "Prevent premature display of starting window." into oc-dev

Merge "Safeguard against uncommon scenarios when pruning" into oc-dev

Batterystats BLE results counter takes in count

There would likely be too many BLE scan result calls for batterystats to
be healthy. Therefore, the caller can supply a count by which to increase the
counter, instead of doing it one result at a time.

Bug: 37720787
Test: runtest -x frameworks/base/core/tests/coretests/src/com/android/internal/os/BatteryStatsTests.java
Change-Id: Ib8b2e8d250036be24c6c3dac477bc8d34fe498a6

SystemUI: add NETWORK_SETTINGS permission

Add the NETWORK_SETTINGS permission to the sysui components so we can
properly check for allowed callers in WifiService.

Bug: 36760523
Test: make
Change-Id: Ia054ffad7a9508e0d80f7b3a444691373ad1924f

Remove lots of log spew

Change-Id: Ia8dbc32c72585c14f9e1a2c5364ab2663bbf97ea
Fixes: 33351736
Test: Manual; start the system and enjoy how little PackageManager spams

Merge "Re-Re-enable SettingsBackupTest" into oc-dev

Prevent premature display of starting window.

- When starting an activity, don't tell window manager to make the
windows visible is the keygaurd visibility for the activity is false.
Instead let the activity windows be made visible once the keygaurd
visibility transitions to true.
- Don't send duplicate visibility change request to window manger if we
are not changing state.

Fixes: 34545029
Test: Open a hangouts notification message on the lock-screen
and make sure the starting window doesn't flash.

Change-Id: I745e985766a1af97203e1d22b6443dabdd0c0363

More QS theme fixes for contrast

Test: visual
Merged-In: I4494e92ebf79bbc20c30a18f5ac13228801a3e54
Change-Id: Ifb03403042561ec23e67159eb51387a460d4f90b
Fixes: 37443237

Properly invalidate text classification cache.

Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
Change-Id: I02b77fc0edf53d13b1623631f4521487cd0bf081
Fixes: 38247676

Merge "Show misc channel with pre-upgrade fields" into oc-dev

TextClassifier: Ensure url scheme is in lowercase

 packageManager.resolveActivity(intentWithUppercaseScheme, 0) returns null.
 This means we don't find an activity to handle the browser intent.
 Create a browser intent with url's scheme in lowercase instead.

Test: bit FrameworksCoreTests:android.view.textclassifier.TextClassificationManagerTest
Fixes: 36504599
Change-Id: Idb0c5b73d54cf77d5d6fb4ee0af4478ec11e8b82

Do not hold WM lock while closing animation transaction

Animation transactions can be blocking, leading to total
window manager starvation. Fix this by not holding lock.

Test: Double tap recents button, make sure it's always responsive
Change-Id: I8e09e04f243d2bfc09fb68097846a42e76c7cab5
Fixes: 38192114

Merge "TRON: Count smart selection events." into oc-dev

Merge "Disable font test temporarily" into oc-dev

Merge "Fix all issues with SystemUI visibility and transitions" into oc-dev

Fix broken PackageParserTest

The PackageParserTest has been broken since ag/2084396 was merged.

Test: runtest --path services/tests/servicestests/src/com/android/server/pm/PackageParserTest.java
Bug: 38250904
Change-Id: Ief35d9e1227f9e2f43343d52e47fe02ee126f0c4

Merge changes from topic 'onarevert' into oc-dev

* changes:
  Create a NotificationChannel for ONA.
  Revert "Delete old WifiNotificationController strings."

Fix racy clone

BUG: 38184803
Test: cts AnimatorSet tests
Change-Id: I308ae9cc014f13a6aa928d8d82e339f476f4b3de

Add time spent by uid per cpu frequency to batterystats.

Currently we have total user and system cpu time spent by the uid
in the batterystats. This change will add times spent by the uid
at each different cpu frequency to the batterystats dump which will
help in better understanding the battery usage.

Bug: 34133340
Test: manual
Change-Id: I567dc75875eb58543c0e71572d0b440e87dbb4b9

Merge "Allow Recents to rotate freely." into oc-dev

Merge "System/root/shell need to explicitly ask for static libs to get them" into oc-dev

Merge "Reduce impact of RenderThread in system_server" into oc-dev