Don't get application info for the system

Bug: 28107178
Change-Id: Ic1a75a7d6d6e80e0df8f3017fb8737cf18f4dd3a

Merge changes Ic2c84de7,I379aef92 into nyc-dev

* changes:
  Remove duplicated code in NetworkStateTrackerHandler.
  Improve Tethering and NativeDaemonConnector logging.

Merge "DND related restrictions" into nyc-dev

Merge "Keyboard shortcuts: UI polish 1" into nyc-dev

Keyboard shortcuts: UI polish 1

* title color for system groups is now: "material_deep_teal_500"

* background behind shortcut keys:
** Now has 2dp rounded corners
** changed color from "material_grey_200" to "material_grey_100"

* the text items now have a minimum width equal to their height.
This means that now the text items with one character are always
the same (square) size as the icon items. Makes the UI look much
cleaner thus easier to read

* the line item now has a minimum height of 48dp and the content
is vertically centered

* minor variable renaming for increased readability

Bug: 28075364
Change-Id: Id7090b607b9c604c55513e7c393ed1084a1c8df0

Merge changes If7388e15,Ica73e67a,Ia4d4362e into nyc-dev

* changes:
  Assume package failed to compile unless proven otherwise
  Refactor return values of performDexOpt
  Update packages in post-boot background job

Assume package failed to compile unless proven otherwise

BackgroundDexOptService keeps a list of packages which failed to
compile so that they are not revisited. If compilation takes so long
that the background job is killed, the offending package is not
recorded.

This patch records the package before dexopt is called and removes it
from the list if dexopt succeeds.

Bug: 28082762
Change-Id: If7388e159b999287b60f19dc99cf4dde61ec64c8

Refactor return values of performDexOpt

PackageDexOptimizer.performDexOpt would return DEX_OPT_PERFORMED if
dexopt succeeded on the package and DEX_OPT_SKIPPED otherwise, even
if dexopt failed. This patch fixes that and cleans up the code.

PackageManagerService.performDexOpt* would return true only if
PackageDexOptimizer.performDexOpt returned DEX_OPT_PERFORMED.
Consequently, it would return false when dexopt was not needed. This
patch refactors the code to return true unless PackageDexOptimizer
returns DEX_OPT_FAILED and documents the behaviour.

Bug: 28082762
Change-Id: Ica73e67ab02025ef5619746bb8c465c96b72846b

Update packages in post-boot background job

Adds a post-boot job which scans all optimizable packages and updates
those whose OAT files are out of date. This is meant to offset the
fact that on OTA we only update the most used packages.

Bug: 27901338
Change-Id: Ia4d4362ecead1ca63d08d62c6814dad4b810f7cc

Merge "Keyboard shortcuts: sort application items" into nyc-dev

Merge "Fix inefficient CursorAnchorInfo#hashCode()." into nyc-dev

Merge "Keyboard Shortcuts Helper: Find device with KeyCharacterMap" into nyc-dev

Merge "Expose the Keyboard Shortcuts Helper in Activity" into nyc-dev

Remove duplicated code in NetworkStateTrackerHandler.

Also use MessageUtils to decode message names.

Change-Id: Ic2c84de73fbf3eb10cc9634a083e5d3b3ffb807a

Improve Tethering and NativeDaemonConnector logging.

1. Deduplicate the Tethering message numbers, and use MessageUtils
   to convert them to strings.
2. Add a warning to NativeDaemonConnector when an unsolicited
   event is more than 500ms late or takes more than 500ms to
   process.

Bug: 27857665
Change-Id: I379aef9257027d1ccf30906e79c6389ef1f95420

Fix inefficient CursorAnchorInfo#hashCode().

It turns out that the current CursorAnchorInfo#equals() is quite
inefficient because our CursorAnchorInfo#hashCode() tries to use almost
all the fields.  Even worse, as Matrix#hashCode() is hard-coded to 44,
we get the same hashCode() when comparing two CursorAnchorInfo objects
that are different only in transformation Matrix after such a complex
hash calculation.

In the real world scenarios, most likely calculation hash code only from
Matrix and composing text would be good enough for our use case, because
the former can cover UI scrolling scenario and the latter can cover the
text typing scenario.  More complex hash calculation is probably
inefficient.

With this CL, CursorAnchorInfo#hashCode() is pre-calculated only from
those two fields, and carefully reorder comparisons in
CursorAnchorInfo#equals() to improve the likelihood of returning false
with fewer comparisons.

Bug: 28105733
Change-Id: Id896adeab5ffe87ceddb2c2762d6d91475e28ec4

Merge "Fix crash when deleting multiple files." into nyc-dev

Merge "Use inode numbers for CE storage." into nyc-dev

Merge "Relax SparseMappingTable Slog.wtf()." into nyc-dev

Relax SparseMappingTable Slog.wtf().

When it's triggered, it happens with such frequency that it can DoS
the system server.

Bug: 28104329

Change-Id: I5c58e5f5bf4d88af2cb6215bcfddf35704e22eaa

Fix crash when deleting multiple files.

When deleting files, MtpDocumentsProvider clears LoadingTask in
DocumentsLoader to update directory contents list. Previously it can
clear ongoing task, and it skips calling Mapper#stopAddingDocuments.
Since Mapper#startAddingDocuments and Mapper#stopAddingDocuments must be
called 1 to 1, it causes precondition check failure at the next call of
Mapper#startAddingDocuments.

Change-Id: I23e2b117da826297e45404be4db4cc29f96e5510
Fix: 28076320

Merge "Shift+Meta+Space should reverse-rotate subtypes." into nyc-dev

Use inode numbers for CE storage.

Certain operations, such as clearing/destroying app data, or just
counting on-disk size, require us to know the CE storage directory
of a particular app.  To facilitate these operations, offer a method
to get the inode of a CE directory, and accept that inode number
for later operations.  Collect and store the inode number in
PackageUserState for future use when that user's CE storage is
still locked.  This design means it's safe to clear/destroy app
data in both CE/DE storage at the same time.

Move most installd-related methods to a uniform calling convention
that accepts a single parent PackageParser.Package, and internally
fans out to handle all "leaf" packages under that parent.

In previous releases, we started installing apps using a new
directory-based layout, where all app code, unpacked native libraries,
and optimized code is bundled together.  So now we only have a single
path to measure for code size.  This fixes several outstanding bugs
that were causing sizes to be miscounted for apps supporting multiple
architectures.

Fix a subtle bug in PackageSettings that would cause "notLaunched"
to be parsed incorrectly.

Bug: 27828915, 27197819
Change-Id: Ia582cf3550553292bde4bb4313367111332913ec

Merge "Fixed a bug where clear all wasn't translated" into nyc-dev

Merge "Fixed a bug where the wrong group was HUNd" into nyc-dev

Shift+Meta+Space should reverse-rotate subtypes.

This is a follow up CL to my previous CL [1], which added a new key
binding Meta+Space to rotate enabled IME subtypes.  With this CL,
Shift+Meta+Space starts reverse-rotating enabled IME subtypes as
originally planed.

 [1]: I4005692215edfcf8bed3e86b1e07000148f986f5
      ae61f7118a92e097e854c840d5726c0920f5db0e

Bug: 25753404
Bug: 28103839
Change-Id: I3694edd80be6dfe18b90360e24ae4d451b331928

Merge "Fix minimal size for tasks in right-hand pane" into nyc-dev

Merge "Fix opening archives in Downloads." into nyc-dev

Merge "For Auto, display battery status as that of a connected device." into nyc-dev

Merge "Revert "Keep read/write external storage as built-in permissions"" into nyc-dev

Revert "Keep read/write external storage as built-in permissions"

This reverts commit 4086750e30d04715d5be3ca6eaa75a4a1fa41d56.

Change-Id: I5f0d8e2920e523b4806da1cca3dc73b1c0354015

Merge "More freezing of apps when doing surgery." into nyc-dev

Merge "Revert "Customizes input extract edit view for Wear"" into nyc-dev

More freezing of apps when doing surgery.

We're still hearing rare reports of apps running while the system
is trying to do surgery on app code/data.  To fix this once and for
all, start guarding all PackageManager critical sections by freezing
and then killing the app before doing surgery.

This is done by introducing a new PackageFreezer class which can be
used in try-with-resources blocks.  It also handles child packages
uniformly, and it uses CloseGuard to defensively un-freeze packages
if a caller leaks without closing.

The set of frozen packages is now maintained outside of PackageSetting
to support newly installed packages.  Add docs for the various locks
and method syntax conventions, including the new "LIF" syntax which
indicates the caller is responsible for freezing the package being
worked on.

Bug: 27698554
Change-Id: I64c4c48123060ccb4d4c50c2fbf3ef223c01e659

Revert "Customizes input extract edit view for Wear"

This reverts commit 880602eb640f1f0f7f77f8d17c4ad34aca8b751d [1],
which was committed with a wrong author email address and lacked license
notice in some files.  To avoid confusion, this we decided to
temporarily revert that CL so that we can commit it with proper license
notice and author address again.

 [1]: I16226ce393f2d15065d08e66a36d008eb1a0c8a1

Bug: 22512982
Bug: 28098677
Change-Id: I3dd3c7bf0ee9634fc4f3bf433bf5023675873e46

Revert "Keep read/write external storage as built-in permissions"

This reverts commit 4086750e30d04715d5be3ca6eaa75a4a1fa41d56.

Add missing method called from native code.

Without this method, boot would fail.

Bug: 28093387
Change-Id: I304d48bbf3ea24ce6af3b5a71c60b8ed06be38e1

Merge "Keep read/write external storage as built-in permissions" into nyc-dev

Keep read/write external storage as built-in permissions

These are permissions that were mapped to gids but we need
to keep them listed event though they are no longer mapped
to gis until an upgrade from L to the current version is to
be supported. These permissions are built-in and in L were
not stored in packages.xml as a result if they are not defined
in the platform.xml while parsing packages.xml we would
ignore these permissions being granted to apps and not
propagate the granted state. From N we are storing the built-in
permissions in packages.xml as the saved storage is negligible
(one tag with the permission) compared to the fragility as one
can remove a built-in permission which no longer needs to be
mapped to gids and break grant propagation.

bug:27185272

Change-Id: I440f6ceb7bc4710dece1a2fadabc995b18fc2a83

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

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

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

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

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

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

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Merge "More APF debuggability." into nyc-dev

Merge "Disable snap targets that make apps <220dp" into nyc-dev

Disable snap targets that make apps <220dp

- Make minimal task size 220dp.
- Disable upper and lower targets if they result in less
than 220dp task size.
- If even the middle target doesn't allow 220dp task size,
disable entering split screen altogether.

Bug: 26451260
Change-Id: I06e358c9b3da0172c5def75cdadf975f87f9fa57

Merge "ExifInterface: inform the developers to close their FD/streams" into nyc-dev

Merge "Only force drawing status bar background if target >= N" into nyc-dev

Only force drawing status bar background if target >= N

Since forcing it all the time has the potential of breaking
compatibility with apps, we don't want to do this.

Instead, we only force it if the app targets >= N.

We communicate this to window manager with
PRIVATE_FLAG_FORCE_DRAW_STATUS_BAR_BACKGROUND.

We introduced this for 2-up split-screen. If we have an app
that doesn't draw the status bar background by itself, we
just force the whole bar to be black.

The same applies for windows that used translucent status
bar - we also force the whole bar to be black

Bug: 27285627
Change-Id: I7f1ceaa364f8a4e851935f77aa5e8d913bf11791

Merge "Excluded certain APNs (e.g. IMS) from mobile data usage." into nyc-dev

Merge "Fix for activity manager crash due to null task" into nyc-dev

Merge "SoundTriggerHelper re-design." into nyc-dev

Fixed a bug where clear all wasn't translated

Change-Id: Ib58611f79ac71ec01f8027cda74a62b71824c03c
Fixes: 27597203

Fixed a bug where the wrong group was HUNd

Fixes: 22466201
Fixes: 27925006
Change-Id: Iaaf91b63c0bb218a75b449554f17ed3828fdfff8

Fix a few issues with non-resizable info activity

- If stack doesn't have focus, we execute TASK_TO_FRONT transition.
Set the correct animation for this transition type.
- Make sure to execute app transition when we are finishing an
activity that isn't resumed.
- Correctly set mAnimatingExit for the case if the activity is
already paused.

Bug: 27327287
Bug: 27154882
Change-Id: I253938727ba0eea76ebadba242315bd2d305d0b4

Update visuals of force resizable

Bug: 27327287
Change-Id: I32d30a8044f683e2f6287035250126999f623e15

Do not retry taking screenshots

We are holding the activity manager lock during a sleep() :-(, which
is a really really bad idea, and leads to delays in certain cases.

Bug: 28026841
Change-Id: I0855350ee429907e4597e5813c7e4fefd889319d

Fix BackdropFrameRenderer leak

Make sure to stop the thread when the window gets detached. When dismissing
the docked/fullscreen stack with the divider, we stop the activity while
we are still in resizing mode.

Bug: 28054032
Change-Id: I2d5d0ffaa9bc47e4d5252414b9a045beaebb7a69

Merge "Improved javadoc for onLocalVoiceInteractionStopped" into nyc-dev

Use writeStringList for selection parceling.

Bug: 27208679
Change-Id: Id0506e49254e0b3fcddb7d3f92717da0a95f2f6d

Merge "Fixed the sign error in GnssClock" into nyc-dev

Merge "Camera2: update API per API council review" into nyc-dev

Camera2: update API per API council review

Per API concil recommendation, made below API change:
* change method name from createCaptureSessionByOutputConfiguration to
createCaptureSessionByOutputConfigurations
* Change method name from
createReprocessableCaptureSessionWithConfigurations to
createReprocessableCaptureSessionByConfigurations

Bug: 27950067
Change-Id: I8346d384210556a40bc2544c3660cc4819f304a1

Merge "Consolidate user actions metrics into one histogram." into nyc-dev

Merge "Reduce unnecessary accessibility cache clearing." into nyc-dev

Merge "audio service: fix volume burst on user switch" into nyc-dev

Merge "Update the documentation on createCircularReveal" into nyc-dev

Merge "Improve content descriptions for volume dialog icons." into nyc-dev

Consolidate user actions metrics into one histogram.

Bug: 27301081
Change-Id: Ib2ac46dd268e492c576bc082dc349c4a6826897f

DND related restrictions

- Apps without dnd access cannot call adjuststeamvolume if that will
change the ringer mode
- DND muted streams cannot be unmuted when DND total silence is enabled.

Bug: 27624414
Bug: 25395278
Change-Id: Id10988c42fc6cb6407aa2abcf66cc5c384fe533a

Improve content descriptions for volume dialog icons.

Bug: 27971063
Change-Id: I9282083f68a5cd607fe0cb2893829675559acb6d

Merge "This assert is triggering when it shouldn't be. Disable it for now." into nyc-dev

For Auto, display battery status as that of a connected device.

For Android Auto Embedded, we would like to display the battery status
of a device that is connected via Bluetooth and not the battery status
of the device itself (which would not make sense in Auto).

To accomplish this, introduce a new CarBatteryController that only
monitors the status of battery via Bluetooth. Note that AAE is not
explicitly handling the battery icon that appears in the quick settings
because this will be hidden for Auto.

This CarBatteryController implements a new BatteryController interface.
What used to be the BatteryController has been moved to the
BatteryControllerImpl class.

Bug: 28002775
Change-Id: I2285bcbd3d207cdcc1ac5a98ec3685d4fff0f0d9

Fixed the sign error in GnssClock

Bug: 28068514
Change-Id: I71b68acb70d4139f94f3829bd4a280d926e251de

Merge "Add granular control over disabled effects." into nyc-dev

Merge "Customizes input extract edit view for Wear" into nyc-dev

Customizes input extract edit view for Wear

Provides an alternate presentation of the extract edit area
of an input method window, designed for sub 250dp width screens.

An icon is used on a round material style button in place of a text
action, providing more horizontal space for the editing area.

BUG: 22512982
Change-Id: I16226ce393f2d15065d08e66a36d008eb1a0c8a1
(cherry picked from commit fd71a6a5d99bad5d3ab32edc328fb5b026b66215)

Merge "Fix NPE for MATCH_FACTORY_ONLY" into nyc-dev

Merge "Visual updates to battery screen" into nyc-dev

Fix NPE for MATCH_FACTORY_ONLY

Bug: 28087606
Change-Id: Ib86c5189aa9a9ca87504625dd19bdeab56fd4967

audio service: fix volume burst on user switch

Change the convention on use of default device volume with
audio policy manager: Now setting the default device volume on a stream does not
reset all specific device volumes. The default volume is just used by audio policy
manager if no specific device volume is present for a given selected device.

Bug: 27557733

Change-Id: I1edd9530ccafc615d9353b9b06489c688e21f719

Merge "Fix early session termination in FalsingManager" into nyc-dev

Merge "Fix HUN Scrim" into nyc-dev

Merge "Fix invalid stack position when docked on right" into nyc-dev

SoundTriggerHelper re-design.

This CL redesigns and cleans up a lot of the core logic in STH.

Major changes include:
   - Adds Tron logging based counters.
   - Common startRecognition(), stopRecognition() and updateRecognition()
     routines that work on both Keyphrase and Generic sound models.
   - Common streamlined logic for when recognition is aborted, requested and
     paused due to phone call/power save etc, for both keyphrase and generic
     models.
   - Special handling of the singleton nature of keyphrase models in
     startRecognition().
   - Moves all Keyphrase model state to the common ModelData structure.
   - Adds Recognition pause/resume() for generic sound models so that they react
     to power save, call state, recognition aborted due to service unavailable
     etc.
Bug: 27972641

Change-Id: I96a7f567d2a4973facec556892a163ca74176bbf

Fix minimal size for tasks in right-hand pane

Minimal size adjustment was not applying correctly to right-hand pane.
Also when task with minimal size set was moved to docked stack on the
right, first configuration was calculated only for its visible part.

Bug: 27621228
Change-Id: I36bc5cdfe08056ee1aea8b0cc08fd28e87e578cc

Merge "Allow the ranker to autobundle notifications." into nyc-dev

Reduce unnecessary accessibility cache clearing.

Tracking if accessibility focus is being cleared because it is being
set to another view in the same window. In this case, leave
accessibility focus on the window.

This change greatly reduces the amount of cache re-indexing.
Previously we flushed the cache every time accessibility focus moved.

Bug: 28077283
Change-Id: If80899d36e7f58b22635f844bdd4ea37a55b875e

Fix invalid stack position when docked on right

If task is dragged from fullscreen recents to right to perform
dock-to-right, it's stack is initially positioned incorrectly.
This CL shifts position in correct direction.

Bug: 27621228
Change-Id: I14df76a036aa0f6f7b500e4e8b5ce04fbc03de9a

Merge "Allow mode of TileService to be changed" into nyc-dev

Merge "Moving HelpUtils to SettingsLib" into nyc-dev

Merge "Fix issue #26879170: Adjust doze maintenance windows..." into nyc-dev