Fix crash from non-default sims in QS

Test: runtest systemui
Change-Id: Icdb063589258a6b4bd94634cc9f41f788a7d2299
Fixes: 62209034

Merge "Prettify the captive portal sign-in activity" into oc-dev

Merge "Update documentation of MediaCodec regarding partial frame." into oc-dev

Merge "Fix TimePickerDialog positive button clicks." into oc-dev

Merge "SystemUI: fix notification redaction logic." into oc-dev

Merge "SystemUI: fix incorrect redaction of profile notifications." into oc-dev

Merge "Add BottomLabelLayout for battery graph" into oc-dev

Merge "Support multi-window on low-memory watches." into oc-dev

Merge "Ensure null/empty phone number does not crash international number check." into oc-dev

Merge "Fix dequeus outside of frame drawing" into oc-dev

Prettify the captive portal sign-in activity

- fuses url bar with activity header: url is now the subtitle
- url appears as text and not as editable content
- url only shows host name
- header style is the device default for Settings
- Material progress bar
- progress bar disappears at page load finished
- webview does not jump at page load finished

Bug: 62107381
Bug: 38197949
Test: tested by manually triggering the sign-in activity
Change-Id: Id75023acad04f444dbfc41de56814a09e70cc6f9

Merge "Ensure start delay is properly scaled with duration scale" into oc-dev

Update documentation of MediaCodec regarding partial frame.

Test: documentation update
Bug: 62202844
Change-Id: Ie1e269e7e22206001a145e9c7ee6f92d7e8633e1

Merge "BatteryStats: Acquire network stats without BatteryStatsImpl lock" into oc-dev

Merge "hotspot2: add/update javadoc for Passpoint APIs" into oc-dev

Fix TimePickerDialog positive button clicks.

With my TimePickerDialog change to support keyboard based input I
accidentally broke TimePickerDialog#onClick as it was no longer being
called, instead it was calling TimeSetListener directly. This CL changes
the logic back to use onClick again.

Bug: 36042834
Test: Locally tested FitBit app.
Change-Id: I47d5563c99cc46eaaf2b1d4a96483d6825fc5805

Support multi-window on low-memory watches.

On watches, multi-window is used to present essential system UI, and thus it
must be supported regardless of device memory characteristics.

Bug: 37482466
Test: Manually, on a watch
Change-Id: I7929a090b7fd46de019d237ce771c82a6d7fd3f3

Merge "Revert "Handle not having main window in createThumbnailAppAnimator."" into oc-dev

Ensure null/empty phone number does not crash international number check.

Add empty/null check for isInternationalNumber.

Test: Added unit tests.
Bug: 62162029
Change-Id: I779a1f78cbbe11adeefeca7fc83cfc1cfa431c61

Revert "Handle not having main window in createThumbnailAppAnimator."

This reverts commit 6db4d15d726b8830ad56766352a79fe0417cb9c2.

Bug:62096254
Reason for revert: Breaking the build due to test references to class

Change-Id: I6021c802d04a3a55a19f54fc94389957c8152ed9

Merge "Do not add unnecessary prefix to dumpsys activity provider [NAME] --proto" into oc-dev

Fix dequeus outside of frame drawing

Bug: 62213889
Test: Repro steps in bug
Change-Id: I8fc407b280ba82c669fc2bb03750cab139adb965

Merge "AAPT2: Update --no-version-vector list" into oc-dev

Merge "AAPT2: Allow merging of Style attributes from overlays" into oc-dev

Merge "Don't automatically autofill when manual request returned just 1 dataset." into oc-dev

Merge "Handle not having main window in createThumbnailAppAnimator." into oc-dev

hotspot2: add/update javadoc for Passpoint APIs

Add javadoc for getter/setter methods configuration classes.
Document exceptions with @throws tag.

Bug: 37907515
Test: build
Change-Id: Ied1842912f6710530bf77db4df5ef5d30a44b2a7

Merge "Added missing lock and clean up locked function that wasn't locked" into oc-dev

Merge "AOD: Scale icon padding with the drawing size" into oc-dev

Merge "Change default for group alert behavior" into oc-dev

Merge "Don't verify secondary setting on restore" into oc-dev

Merge "Track Partial wakelock time in background" into oc-dev

Merge "Fix batterystat Counter misreporting when charging" into oc-dev

Merge "Batterystats track background bad ble scan time" into oc-dev

BatteryStats: Acquire network stats without BatteryStatsImpl lock

Moves the network stats collection under a different lock to
prevent the main BatteryStatsImpl lock from being held while doing I/O.

Bug: 37645919
Bug: 38296815
Test: manual
Change-Id: I0d6b4a7b12b234939cb6eb3a32658b28f61dff4f

Merge "Don't call remoteFillService.destroy() until session is removed." into oc-dev

Merge "Always post AutofillManager operations in the UI thread." into oc-dev

Merge changes from topic 'hw_bitmap' into oc-dev

* changes:
  DO NOT MERGE Updating AnimationSpec and related internal APIs to use GraphicBuffer.
  Update code to draw into hardware bitmaps.

Do not add unnecessary prefix to dumpsys activity provider [NAME] --proto

Bug: 36904684
Test: builds, and tests succeed in the same topic branch

The test executes following
"dumpsys activity provider [launcher package name]/[provider name that implements workspace dump logging]
--proto"

In order to extract the proto binary from this stream, there cannot
be extra prefix added to each line. Hence, when "--proto" flag
is on, simply call the sub provider record's dump method.

This is following same pattern on other system service dump.
Having --proto flag also doesn't do any pretty printing.
e.g., dumpsys appwidget --proto.

Change-Id: I13712a45ce5b584123b3bd73c023df349fb86809

Add BottomLabelLayout for battery graph

This layout will automatically become vertical if there is not
enought room for children.

Bug: 37329353
Test: RunSettingsLibRoboTests
Change-Id: I0f6870c4031446e967c239d207e76837b14ab097

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

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

Ensure start delay is properly scaled with duration scale

BUG: 62126024
Test: CTS test in the same target branch
Change-Id: Ia396e69e053ccd90cdf37ac0b59c404c9952e497

Merge "Check for null owner info" into oc-dev

Change default for group alert behavior

To alert for children but not the summary.

Bug: 62027641
Test: cts
Change-Id: Ie7978514be45be6b13c2ea3bd7ff898db352a1ed

Handle not having main window in createThumbnailAppAnimator.

A recent code change associated the SurfaceControl with the main
window owner id. However, it is possible for the an AppWindowToken to
not have a main window. This case is handled later in the method.

This CL restores the original behavior of using the calling uid in
the cases no main window is present.

Change-Id: Iad69fe383c2208c1db523c8b4601a8f927f9318a
Fixes: 62096254
Test: go/wm-smoke

AAPT2: Update --no-version-vector list

Keep in sync with AAPT. Since gradle is moving to use AAPT2
by default, updates to AAPT will no longer be necessary and
this will be the authoritative list until a more programmatic
solution presents itself.

Bug: 62211148
Test: manual
Change-Id: Id9232c9fb730c50e4850887fa0f5da82574ca900

AAPT2: Allow merging of Style attributes from overlays

Previously style overlays would completely override an existing style.
To be compatible with AAPT, styles now merge with the overlay, allowing
the overlay's attributes and parent to take precedence.

Bug: 38355988
Test: make aapt2_tests
Change-Id: Id25c7240050a43e6a4a177c6e3d51e048d0cceb5

Merge "Fix for wifi detail icon disappearing" into oc-dev

DO NOT MERGE Updating AnimationSpec and related internal APIs to use GraphicBuffer.

- This reduces the copy of the hardware bitmap when it is
  parceled/unparceled.

Bug: 38507414
Bug: 62021436
Test: Launch Overview to/from app, ensure that the header bar shows
Test: go/wm-smoke

Change-Id: I85a9a59a0a3699d1642158061d10fddef34393c3
Signed-off-by: Winson Chung <winsonc@google.com>

Don't verify secondary setting on restore

It'll already be verified as the primary setting of another
managed services type, and it will have a different service
interface, so it shouldn't be overwritten with the primary setting's
service information.

Fixes: 38236571
Test: restore a device. make sure the backup has vr notification
listener permissions on (or any other pkg with both a notification
listener and a condition provider) and some other enabled
notification listener (like a launcher). make sure both
listeners are enabled post restore.

Change-Id: I2e919c2bb06695a3326269571a901ed627c8bee6

Merge "Don't crash when presented with non-roundRect clipping outline." into oc-dev

Merge "Fix header not set on initialization" into oc-dev

Merge "WifiManager: setWifiApEnabled always returns false" into oc-dev

Merge "Hide Wi-Fi activity indicator when Wi-Fi icon is hidden" into oc-dev

Merge "Fix Error Prone ArrayEquals/ArrayHashCode bugs" into oc-dev

Merge "Consistent "low storage" behavior." into oc-dev

Merge "Fix infinite loop problem in AnimationHandler" into oc-dev

Hide Wi-Fi activity indicator when Wi-Fi icon is hidden

Wi-Fi icon in the status bar can be blocked by System UI Tuner.
So the activity indicator should be hidden.

Fixes: 62211154
Test: manual - block Wi-Fi icon

Author: Akira Oshimi <akira.oshimi@sony.com>
Change-Id: Iba698d621d60f6d6c9f6d8205f91df49806f3f54

Merge "Change the thread for handling shutdown of PowerManagerService." into oc-dev

SystemUI: fix notification redaction logic.

Previously, the logic for redacting sensitive notifications is flawed
for apps that have individually overridden the system-wide redaction
setting. Sensitive notifications from such apps are still redacted even
after unlock.

This CL fixes the issue and also simplifies the redaction logic by
taking advantage of the needsRedaction() method.

Bug: 35951989
Test: manual, see bug log
Change-Id: I6b72ed97b3056a93cbe44f54c607c16445f39551

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Consistent "low storage" behavior.

When answering the question "how much space is free", use the same
logic for Settings UI and StorageManager.getAllocatableBytes().  That
is, the reported free space is usable bytes plus any cached data the
system is willing to delete automatically.

This does *not* include any reserved cache space, since we don't want
abusive apps to penalize other well-behaved apps that are storing
their data in cache locations.  Callers freeing cached data need to
now explicitly request defiance of the reserved cache space.  (Most
callers are already doing this by using FLAG_ALLOCATE_AGGRESSIVE.)

Rewrite the core logic of DeviceStorageMonitorService to understand
this new "reserved" cache space, and to be easier to understand.  It
also now handles cached data on adopted storage volumes, which had
been ignored until now.  Also fix bug where we had skipped "low"
broadcasts when the device skipped directly from/to "full" state.

Bug: 38008706
Test: cts-tradefed run commandAndExit cts-dev -m CtsJobSchedulerTestCases -t android.jobscheduler.cts.StorageConstraintTest
Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.StorageHostTest
Change-Id: Icbdcf3b52775f7ada1ceaeff2f96094c8d8052f9

Merge "Fix unnecessary relaunches when unlocking with fingerprint sensor." into oc-dev

AOD: Scale icon padding with the drawing size

Fixes an issue where the icon placement algorithm would not take into
account the drawingScale of the icon.

Change-Id: Ia6b7e7d32ce584b04035b45d2abc61e0e4590ee8
Test: Trigger AOD1, observe that icons are correctly spaced, also test transition to lockscreen and observe there's no jank
Fixes: 62205097

Don't call remoteFillService.destroy() until session is removed.

Test: CtsAutoFillServiceTestCases pass

Fixes: 62200831
Bug: 62106708

Change-Id: I4e54e28a347e800c0b8353938399c860f213edfa

Always post AutofillManager operations in the UI thread.

Test: existing CtsAutoFillServiceTestCases pass
Fixes: 62194407

Change-Id: If878b2befc03706c000b3f31b60904072b2225f4

Prune unused static libs and instant apps if space needed

We are caching unused static shared libs and instant apps
(installed and uninstalled) opportunistically. If space is
needed we delete these to free up space.

Test: manual

bug:62045000

Change-Id: Id992dee5c7c6e36b8e8b81050602dbc4eeafb0f9

Merge "Do not set the divider pos during adjust ime and minimized interaction" into oc-dev

Fix unnecessary relaunches when unlocking with fingerprint sensor.

When the device is unlocked using the fingerprint sensor in an
orientation opposite to the lockscreen orientation, the app that
will be visible is first relaunched in the current lockscreen
orientation and then later relaunched in the correct orientation.
If the keygaurd is going away then:
- Don't let keyguard affect device orientation. We want to use the
orientation of the app that will be visible.
- Allow the rotation sensor to be enabled even though draw isn't
complete so window manager can get the updated or last rotation
reading.
- Don't clear the previous proposed sensor reading to allow
window manager to use the information to update the orientation as
needed vs. falling back to the previous orientation.

Change-Id: I8369723d6a77f2c602f1ef080371fa7cd9ee094e
Fixes: 38494778
Test: Launch an app that doesn't fix orientation like clock, hold
the device in landscape, press the power button, unlock the device
using the fingerprint sensor, and verify the the app isn't
relaunched.

Merge "Turn off logspam" into oc-dev

Merge changes I4b58610f,Ib758756a into oc-dev

* changes:
  Turn off the check in the test because the assertion is off in the implementation.
  Fix typos that cause infinite recursion and CTS test failure.

Check for null owner info

Test: Cast screen, lock keyguard
Change-Id: I314244d8a5fe1d7f49afbaf5247e84d5ea4df027
Fixes: 38484035

Merge "Compare against correct user id to determine static lib clients" into oc-dev

Merge "Make NFC AIDL callbacks oneway" into oc-dev

Merge "Fix issue #62196301: Pre-O background restrictions allow starts..." into oc-dev

Merge "Fix issue with aspect ratio not applying correctly when PIP is expanded" into oc-dev

Merge "Fix NPE and wrong behavior of Typeface.isSupportedAxes" into oc-dev

Fix header not set on initialization

Header not being set was causing min priority
groups to skip expansion animation.

Bug: 38436027
Test: runtest -x frameworks/base/packages/SystemUI/tests/src/com/android/systemui/statusbar/stack/NotificationChildrenContainerTest.java
Change-Id: I34307a7e5ecfe7e30d5bf97880b17fedb0d84378

Merge "Fix QS header touch targets" into oc-dev

Merge "Show date + alarm icon when on lock screen" into oc-dev

Added missing lock and clean up locked function that wasn't locked

There was a missing lock that systemui would call to register a docked
stack listener. Since it was missing, it is assumed that two threads
went into a race condition which caused it to crash when handling begin
before finish broadcasts for the same DockedStackDividerController.
Also moved getDefaultDisplayContentLocked call that was not locked to a
locked area.

Change-Id: I3c7ef805024aaf0f5f7c4ab68fc779797114a038
Fixes: 62104210
Test: go/wm-smoke

Merge "Temporarily enable screen wakelock logging in WM" into oc-dev

Merge "Random hacks to make QS DND work" into oc-dev

Turn off logspam

For more information, see https://source.android.com/source/code-style#log-sparingly

Bug: 37252687
Test: development/tools/logblame/app_switch_test
Test: development/tools/logblame/power_toggle_test
Test: development/tools/logblame/medium_idle_test
Change-Id: Ie2f6ef33df7b6c1ea976a8180945651ac9482976

Compare against correct user id to determine static lib clients

Test: manual, static shared lib CTS tests pass

bug:36632264

Change-Id: I58b2787ff0b95c89f9e9bb9a005b39e3505aefcb

Merge "Set correct task remove mode when reparenting" into oc-dev

Merge "Fix ripple on QS Footer" into oc-dev

Merge "Fix crash from concurrent modification" into oc-dev

Don't automatically autofill when manual request returned just 1 dataset.

When the user manually requested autofill and the service returned just 1
dataset, the app was automatically filled with it.

The motivation here was that since the user explicitly asked for autofill,
it'd ok - and better for the user - to release the data to the app without
the selecting a dataset.

This assumption was ok initially, but now the API to manually request autofill
is public, so a malicious app could use to get autofill data without user
interation.

Test: CtsAutoFillServiceTestCases pass
Fixes: 62164695

Change-Id: I47c3d8c557533526572aa67a4240c0a57f54268d

Merge changes I5762a98d,I39f4e015 into oc-dev

* changes:
  ViewRootImpl: More careful draw accounting.
  SurfaceView: Avoid over-reporting DRAW_FINISHED.

Merge "Update Paint#mFontVariationSettings in set() and reset()" into oc-dev

Temporarily enable screen wakelock logging in WM

Enable DEBUG_KEEP_SCREEN_ON flag to get additional wakelock logs
to investigate b/38416971.

Bug: 38416971
Test: No change in logic
Change-Id: I83c45a26befb6a767b5d9fc30b1cde07a2432ebe

Merge "Prevent unnecesary Surface preservation." into oc-dev

Merge "AAPT2: Fix <add-resource> tag for overlays" into oc-dev

Fix NPE and wrong behavior of Typeface.isSupportedAxes

The Typeface kept by Paint object can be null if nobody calls
setTypeface. The null typeface is equivalent to the Typeface.DEFAULT.
To check whether the passed axis is usable for Typeface.DEFAULT, we
need to pass Typeface.DEFAULT to isSupportedAxes if no typeface was set
on the Paint.

At the same time this CL fixes an issue in isSupportedAxes.
If the requested axis is listed in the first element of axes list,
isSupportedAxes returned false due to wrong handling of binarySearch
result.

Bug: 62146672
Bug: 62147012
Test: am instrument -w -e class android.graphics.cts.PaintTest
android.graphics.cts/android.support.test.runner.AndroidJUnitRunner

Change-Id: I7c154adfe8a19d6ed24ad645df7c10cee7880461

Random hacks to make QS DND work

 - Don't clear out as much state on attach
 - Make sure condition updates trigger a reflecting radio check
 - Assume that null conditions are forever

Test: manual
Fixes: 37944320
Fixes: 36031650

Change-Id: Ie926d16226e7b5431551a33515ac2b847e1d7a93

Update code to draw into hardware bitmaps.

Bug: 37698012
Bug: 38510859
Test: Dock task from Recents, ensure that it enters multiwindow.
Test: go/wm-smoke

Change-Id: Ia8f60f5884bb8d620dc554b61eaa95aa0038aee3

Merge "Work on issue #62095840: NPE in JobParamteres in the background" into oc-dev