Merge "Fix NPE which could occur when showing notification guts" into oc-mr1-dev

Merge "Refinement of auto-dimming nav" into oc-mr1-dev

Merge "Add low-ram check for PiP support even if feature is enabled." into oc-mr1-dev

Merge "Minor fixes on WebView.onProvideAutofillVirtualStructure() javadoc." into oc-mr1-dev

Refinement of auto-dimming nav

 - Make dimming only happen for interaction with nav
 - Extend time to dim
 - Shorten time to start dimming by same amount.

Test: visual
Change-Id: Ieb5b31bacdde9ce1952601ad65cff2d54386dd0f
Fixes: 65694001

Merge "Fix consequent activity launch on virtual displays" into oc-mr1-dev

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

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

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

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

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

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

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

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

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

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

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

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

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

Merge "Do not stop paused activity when sleeping." into oc-mr1-dev

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: Ided02ec91e7a2f5aacd8e5c28201672d14fceea7

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: I6dab1e6544868c0ac2e0e4029a969a216f4c9b4d

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: Ic2ce0ef69af1bfd9f4b8980b3079b9756f93763c

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: I431251ef3dc0930b820a108edc6b76f876639035

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: Ic586e92feca951793fd3cb9a401093e57e05b015

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: I81c38a635544d1672a6cbd1dc9528d95ccdbd894

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: I3978b88537a89176a6926eab1c21e09ab169a2cc

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: Ib786e99af653b130f2547c8ed1e223f0ddbcf3c8

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: Ie25a5101373e7c24af943b998a27f6f02da62a34

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: I8d182fda9abf6a8c4f5bb4f62de78e61a5c3d627

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: Ifdd0077f7516e9266a9f38ff474bc8f71cc7591b

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: I61400c4699a3694bc91bc6672e7dc62a28545bcc

Import translations. DO NOT MERGE

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

Bug: 64712476
Change-Id: Ie76a52fa6f6c717d78db5f9c338b1fde5cc6b4ad

Merge "Fixed size of autofill service icon on Save affordance." into oc-mr1-dev

Add low-ram check for PiP support even if feature is enabled.

Bug: 65602088
Test: Can not enter PiP on low-ram devices
Change-Id: I526cf73788fb4c7298433bee5cb54711820f042b

Fix NPE which could occur when showing notification guts

SystemUI keeps shown heads-up notification for minimum display time
even if the notification is canceled right after it's posted.

This is intentional behavior but causes inconsistency between
mEntry and mRankingMap in NotificationData.
That inconsistency could cause NPE when showing notification guts.

To avoid this, update an Entry only when the corresponding Ranking
is available.

Fixes: 65567562
Test: manual - long press a missed call notification
               immediately after receiving an incoming call
Test: runtest -x packages/SystemUI/tests/src/com/android/systemui/statusbar/NotificationDataTest.java

Change-Id: I6dc70d8e57efa7af0f23862a00d0e18cf39dfebb

Minor fixes on WebView.onProvideAutofillVirtualStructure() javadoc.

Test: mmm -j108 frameworks/base/:doc-comment-check-docs
Bug: 65751159

Change-Id: Ie60b1f32e4d9acc675c6e25b7d3e606049b6a3c3

Merge "Color extraction should not block switchUser" into oc-mr1-dev

Fixed size of autofill service icon on Save affordance.

Test: manual verification
Test: cts-tradefed run commandAndExit cts-dev -m CtsAutoFillServiceTestCases

Bug: 65739885

Change-Id: Ic9efd7bbb853c28b2d869a609d21eff4e6ef6d2d

Do not stop paused activity when sleeping.

This changelist removes logic added to stop a top level paused
activity when trying to sleep. Doing so delays sleep and leads to
double sleep requests on the client side as it will also try to
stop the activity on sleep.

Bug: 65457314
Test: manual
Change-Id: I252e2b968a537cf2d696972d6a65e43661875943

Merge "Increased the text size for colorized notifications" into oc-mr1-dev

Merge "New metrics fields for selection logs" into oc-mr1-dev

Merge "getInstalledDialerApplications should not count ForwardIntentToParent" into oc-mr1-dev

Increased the text size for colorized notifications

Increased the text sizes a bit to make them more prominent.

Test: add colorized notifications of all styles
Change-Id: Ie22053edcb0297e7c3ee0c43b29cdd0e7b6a4516
Fixes:  35925915

Merge "Fix a no-op uninstall being treated as a failure" into oc-mr1-dev

Merge "Don't take invisible screenshots" into oc-mr1-dev

Merge "Make sure that updated wallpaper id is immediate" into oc-mr1-dev

Merge changes from topic "embms-cherry-pick-3" into oc-mr1-dev

* changes:
  API council suggested edits, part 3
  Apply suggested edits to the file-download API
  Make changes to MBMS Streaming APIs per recommendations

getInstalledDialerApplications should not count ForwardIntentToParent

Issues:
1. Work profile call is logged into work profile side even there
is no work profile dialer.
2. In Settings, "Android" is shown as the default work profile dialer.

Cause:
When are querying installed dialers in work profile,
getInstalledDialerApplications return android/ForwardIntentToParent
because we forward calls to personal side if work dialer
is absent. But apparently, ForwardIntentToParent is not an installed
dialer in work profile. So Settings shows "Android" as default work
dialer and Telecom wrongly think that we have work dialer and insert
calllog in to work profile side.

Solution:
Ignore cross profile resolver by checking targetUserId.

Test: Manual
1. Check Settings, no longer see "Android" is set as work dialer.
2. Make a phone call using work contacts, observed that the call
   is inserted into persoonal side.
3. Make a phone call using personal side, observed that the call
   is inserted into personal side.

BUG: 65672386

Change-Id: I13d4234ad1684f714e1b4d7704f2413d903a0179

Merge changes Id8bdb14e,I573a6735,Ia804777f,Ia68122cb,Ia1997800, ... into oc-mr1-dev

* changes:
  AAPT2: Bump to version 2.19
  AAPT2: Fix issue with resource deduping
  AAPT: Dump badging should pickup strings from the right package
  AAPT2: Change the daemon mode to be line based
  AAPT2: better error handling for daemon mode
  AAPT2: Fix regression in Manifest.java permissions
  AAPT2: Define intermediate compiled XML proto
  AAPT2: Add a daemon mode
  AAPT2: Fix mkdirs implementation
  AAPT2: Change XmlDom to exclude Namespace as a node
  AAPT2: Fix windows unicode path issues
  AAPT2: Fix typo in Config parsing
  AAPT2: Change proto format to reduce usage of StringPool
  AAPT2: Add navigation type
  AAPT2: Document Format.proto

New metrics fields for selection logs

Changes that affect logging:
- Switched to new category/actions/fields that are used exclusively by
  selection logging.
- Action types are now logged in the tron type (the information that was
  previously in the field could be derived from other fields).
- Entity types are now logged in the tron sub type.
- Delta time from previous event is now zero for first event (was time
  since epoch).
- The smart select model version is now written for all events after
  smart select has run. (i.e. better support for analysis based on
  terminal events alone)
- Restoring the logging removed in ag/2901629.

Internal only changes:
- Made debug statement conditional on DEBUG_LOG_ENABLED.
- Reduced logging tag length to fit into 23char limit.

Bug: 64914512
Test: Manually tested that logs are correct. Will follow up with automated tests.
Merged-In: If12daa2f206fb42be53587decbe2c4abfd689f41
Change-Id: Iec0f6cf1f9f2efdc8518133cf8551cd96b095ad8

Fix a no-op uninstall being treated as a failure

Treat a stageUninstall() doing nothing as a success, not a
failure. This prevents the system retrying the uninstall
later.

Unit tests run with:

make -j30 FrameworksServicesTests
adb install -r -g \
  "out/target/product/marlin/data/app/FrameworksServicesTests/FrameworksServicesTests.apk"
adb shell am instrument -e package com.android.server.timezone -w \
  com.android.frameworks.servicestests \
  "com.android.frameworks.servicestests/android.support.test.runner.AndroidJUnitRunner"

Test: See above
Test: Manual testing, adb dumpsys timezone
Test: PTS: run pts -m PtsTimeZoneTestCases
Bug: 65657176
Merged-in: Ifd205da90e848552711ac3f4207cd22ad2c2747b
Change-Id: Ifd205da90e848552711ac3f4207cd22ad2c2747b
(cherry picked from commit 8e27c9226b82f41279ee6c108d2b06bb6f1cef5e)

Don't take invisible screenshots

This fixes a bug where we took a snapshot of a task while the
surface wasn't visible yet. The transition was set-up so the win
animator was already shown but translucency was set to 0, leading
that the snapshot is a black screen.

We fixed this by also checking whether the alpha is non-zero.

Fixes a bug where launching a Hangouts notification from the
lockscreen causes a black flicker. When we launch the activity,
the Hangouts activity is in the mClosingApps list, because it's
immediately hidden from the lockscreen, which means we try to take
a snapshot. Shortly after that we reopen the activity because
Keyguard is going away, so we try to reuse a snapshot.

Test: go/wm-smoke
Test: Open hangouts, close hangouts with back button, go to
Keyguard, receive message, launch notification

Change-Id: Iffc1db1a738800c90a2421a5f90174d647eb4440
Fixes: 65695379

Merge "Finalize package intents used to trigger tz update" into oc-mr1-dev

AAPT2: Bump to version 2.19

Test: manual
Bug: 65645766
Change-Id: Id8bdb14e498bd79ff642f9ad56ef05ec7a5825af
Merged-In: Id8bdb14e498bd79ff642f9ad56ef05ec7a5825af

Merge "Fix conflicting JavaDoc of TYPE_TEXT_FLAG_NO_SUGGESTIONS" into oc-mr1-dev

Merge "Fix missing null checking" into oc-mr1-dev

Merge "Throw if BitmapFactory.Options.inBitmap is HARDWARE" into oc-mr1-dev

Merge "Frameworks: Fix NPE for agent attach" into oc-mr1-dev

Merge "AudioPlaybackConfiguration: prevent race condition on mIPlayerShell" into oc-mr1-dev

Merge "docs: updated link on RemoteInput page" into oc-dev am: 6087b990df
am: 91fb9b2a16

Change-Id: Ib5ff83e4ce6c5a172db8f483387ebb6a8a97ae55

Merge "Do not animate icons when leaving AoD" into oc-mr1-dev

Merge "docs: updated link on RemoteInput page" into oc-dev
am: 6087b990df

Change-Id: Idc629243c3794565cdfebe7bd737a78d97010295

API council suggested edits, part 3

* MbmsException no longer thrown from the common methods, and the error
codes are now containined in MbmsError.
* In order to avoid ANRs while processing download-done broadcasts,
don't perform arbitrary moves when a download is done. Instead, move
into a staging directory and let the app handle it.

Bug: 30981736
Test: testapps
Change-Id: I9416f28b4f24f89af1da6f56a93ea0f0e4ea3878

Merge "docs: updated link on RemoteInput page" into oc-dev

Apply suggested edits to the file-download API

* Rename Manager -> Session
* Add Handler for callbacks in download flow
* Separate out registering a state callback into another method
* Rename dispose -> close, implement AutoClosable
* No more thrown MbmsExceptions
* Add sanity check for temp file root
* Remove the getNames method, replace with getNameForLocale

Bug: 30981736
Test: testapps
Change-Id: I1a2054d79c934bc4929464de4a644b6960db47e9

Fix conflicting JavaDoc of TYPE_TEXT_FLAG_NO_SUGGESTIONS

TYPE_TEXT_FLAG_NO_SUGGESTIONS is just a hint and does not mean
IME should never show a UI to display suggestions.

This CL makes that point clear in JavaDoc.

Test: checkbuild
Bug: 35875399
Bug: 38139781
Bug: 38184682
Fixes: 65693181
Change-Id: Id0c3b6bc05689a5f1c8b52637664f59d45850a60

Merge "Avoid saving position before minimizing when in transition" into oc-mr1-dev

Merge "Fixes quick switch animation entering recents for low ram devices" into oc-mr1-dev

Fix missing null checking

Bug: 65666724
Test: build

Change-Id: Ie3fa891da012cbb3f313a33e0e5759f743887480

Merge "Added SafetyNet logging for UsbDebugging security bug." into oc-mr1-dev

Merge "Show big text on AoD" into oc-mr1-dev

Merge "Use correct date pattern for displaying year in DatePicker" into oc-mr1-dev

Merge "Change battery icon to the side of bt icon" into oc-mr1-dev

Color extraction should not block switchUser

switchUser is called from the main thread. This would block it
until color extraction ends, possibly causing a timeout in
ActivityManager.

We're now offloading color extraction to another thread.

Test: Change wallpapers, switch users.
Change-Id: I570c3ce5a562b56106d614c8febc12134e151afc
Fixes: 65146279

Frameworks: Fix NPE for agent attach

Fix missing null check.

Bug: 62445317
Test: m
Test: adb shell am start --attach-agent dummy.so
Test: m cts && cts-tradefed run commandAndExit cts-dev --module CtsJvmtiAttachingHostTestCases
Change-Id: Ia16c2973b24f81839d20502339978e75a23a93bc

Throw if BitmapFactory.Options.inBitmap is HARDWARE

Test: BitmapRegionDecoderTest#testHardwareBitmapIn
bug:34415265
Change-Id: I58061504dd42ba918f89e1e5daf668f7601ca8d3

Make changes to MBMS Streaming APIs per recommendations

* Rename MbmsStreamingManager -> MbmsStreamingSession
* MbmsStreamingSession now implements AutoClosable instead of using
dispose()
* No longer throw checked MbmsExceptions from any methods. All such
errors will be reported via the async callback.
* null arguments no longer accepted for Handlers
* Guarantee that no further callbacks will get enqueued after closing.

Test: run testapps
Bug: 30981736
Merged-In: I6db045053f1baa4cd4de3718865fcfe525e8a4d6
Change-Id: I85e98850759c6c7fe7b883ffffa5b1f4e362770d

Merge "also log user-initiated notification collapse events" into oc-mr1-dev

Merge "Do not fail session.commit on current thread" into oc-mr1-dev

Merge "Clear calling identity" into oc-mr1-dev

Merge "Display volume dialog for longer..." into oc-mr1-dev

Added SafetyNet logging for UsbDebugging security bug.

Bug: 62187985
Test: n/a
Change-Id: I69244381b2172357c5adc8785fc73d1cfd3cc17d

Merge "DO NOT MERGE Backporting potential usb tapjacking precaution." into oc-dev am: d740b2d09a  -s ours
am: fe5ce91c5f  -s ours

Change-Id: I28bb498ea60d1a42d4b557f0b58c7134650bb3dd

Merge "DO NOT MERGE Backporting potential usb tapjacking precaution." into oc-dev
am: d740b2d09a  -s ours

Change-Id: I952391f1801cd678240b145971a733c755f62078

Merge "DO NOT MERGE Backporting potential usb tapjacking precaution." into oc-dev

Merge "Unhide EMBMS download API" into oc-mr1-dev

Make sure that updated wallpaper id is immediate

As soon as setBitmap() or equivalent returns, the new wallpaper id needs
to be immediately observable.  This was being subverted by inconsistent
and racy "initialize from persisted state" handling in the set-wallpaper
case: a load triggered by rebinding the static image display service
could in some cases wind up overriding the new state calculated while
new wallpaper imagery was being processed.

The fix is to clarify the semantics of when load-from-persisted happens:
it is now done *only* when the user is first spun up (or at boot, for
the system user), and a firm guarantee provided about the up-front
availability of the associated bookkeeping.  That, in turn, means not
having to futz with lazy init when some client wants to read the current
wallpaper imagery, and eliminating that gets rid of the races.

And in a strictly-cosmetic fix, corrected the descriptive text for one
of the permission enforcement calls.  Copypasta strikes again!

Bug: 65016846
Test: cts-tradefed run cts-dev -m CtsAppTestCases -t android.app.cts.WallpaperManagerTest\#setBitmapTest
Change-Id: I73da48a58cca1849f073b8aea72019916dc2272b

Merge "Show mobile signal strength in QS edit panel" into oc-mr1-dev

AAPT2: Fix issue with resource deduping

Resource deduping relies on the definitions of
ConfigDescription.ConflictsWith, ConfigDescription.IsCompatibleWith,
and ConfigDescription.Dominates.

ConflictsWith is supposed to ignore range-based qualifiers, like
version, density, smallest width, screen size, etc.

This was not the case for screen size, and was assumed that the
choice of screen size is mutually exclusive.

This CL fixes the assumption and includes screen size (small, normal, large, xlarge)
as a qualifier that does not conflict.

Bug: 64397629
Bug: 65645766
Test: make aapt2_tests
Change-Id: I573a6735fedd7721a10ba32902bc5d27ef99b88e
Merged-In: I573a6735fedd7721a10ba32902bc5d27ef99b88e

AAPT: Dump badging should pickup strings from the right package

A bug in the dump badging command caused strings to be looked up in the
first loaded package only.

Bug: 64948230
Bug: 65645766
Test: none
Change-Id: Ia804777fe3f963004a5c053129ef6e0c94b7f6a3
Merged-In: Ia804777fe3f963004a5c053129ef6e0c94b7f6a3

AAPT2: Change the daemon mode to be line based

Accept a set of arguments separated by newlines.
This avoids path separator conflicts with the argument
format for passing splits.

Test: manual
Bug: 65645766
Change-Id: Ia68122cb77b7dde2292a0fd953e79f02996ac01c
Merged-In: Ia68122cb77b7dde2292a0fd953e79f02996ac01c

AAPT2: better error handling for daemon mode

Test: manual
Bug: 65645766
Change-Id: Ia1997800ad94f386dbbb4195011395ecee1ba91f
Merged-In: Ia1997800ad94f386dbbb4195011395ecee1ba91f

AAPT2: Fix regression in Manifest.java permissions

Permissions defined with the same leaf name emit the same
string symbol, which causes collisions. AAPT would override
the symbol with the last one seen.

Do the same thing as AAPT, but emit a warning.

Bug: 64472942
Bug: 65645766
Test: make aapt2_tests
Change-Id: I17b9dc7e8d8bd80db98869394c93695cb453bebd
Merged-In: I17b9dc7e8d8bd80db98869394c93695cb453bebd

AAPT2: Define intermediate compiled XML proto

This proto format is meant to encapsulate more information
that is specific to Android and allows for easier validation
and manipulation across tools.

Test: make aapt2_tests
Bug: 65645766
Change-Id: I13bc34a460671fc0a36246be0d287a3d37d244d6
Merged-In: I13bc34a460671fc0a36246be0d287a3d37d244d6

AAPT2: Add a daemon mode

Gradle and other build tools that need to frequently interact with aapt2
can use the daemon mode to issue regular CLI commands without paying
the process start cost for each invocation.

Test: manual
Bug: 65645766
Change-Id: I543858ed46496278bad643f0dae688c924ec6c3f
Merged-In: I543858ed46496278bad643f0dae688c924ec6c3f

AAPT2: Fix mkdirs implementation

Once switched to using std::string, the mkdirs implementation was
trying to create an empty string when an absolute file path on
linux or macOS was used.

Bug: 62336414
Bug: 65645766
Test: manual
Change-Id: I52f3050b410a923ca48f353b0983667c16d00ee8
Merged-In: I52f3050b410a923ca48f353b0983667c16d00ee8

AAPT2: Change XmlDom to exclude Namespace as a node

In preparation for exporting an XML proto format for UAM to consume,
this change brings the XML DOM API more in line with other APIs that
do not make the Namespace a separate node.

Treating Namespace declarations as just properties of an Element
node makes the implementation of algorithms much simpler, as
the constraints that Namespace nodes have only one child
are now built in and traversing to find Element nodes
is much simpler.

Also made a bunch of quality of life improvements, like formatting and
comment style.

Test: make aapt2_tests
Bug: 65645766
Change-Id: Ib97ff1c4252b7907e2cc1f13a448dc4ca3b809a4
Merged-In: Ib97ff1c4252b7907e2cc1f13a448dc4ca3b809a4

AAPT2: Fix windows unicode path issues

Mingw64 was being difficult, so instead of defining a wmain entrypoint,
the command line parameters are parsed manually using built-in Windows
methods that support Unicode. The results are converted to UTF8 and
handled just like the rest of the linux/mac version of the code.

This also removes dependencies on std::istream in favour of a
FileInputStream which calls the appropriate unicode version of
open to read a file.

No speed regressions found on Linux or MacOS.

Bug: 62336414
Bug: 63830502
Bug: 65645766
Test: manual
Change-Id: I597da51e33729ed1b98bf246e7e773337fd3fee8
Merged-In: I597da51e33729ed1b98bf246e7e773337fd3fee8

AAPT2: Fix typo in Config parsing

The 'any' syntax of a configuration is rarely used so this has not been
an issue in the field.

Test: none
Bug: 65645766
Change-Id: Icb9cb2d5ad061e3ea1c43dc1cd7ca2f222965cdc
Merged-In: Icb9cb2d5ad061e3ea1c43dc1cd7ca2f222965cdc