Merge ""Temporarily" removed progress check." into nyc-dev

"Temporarily" removed progress check.

The code that asserts the right progress is displayed was based on the
percentage text displayed in the system notification, but such text is
gone.

Since UIObject doesn't easily expose the underlying ProgressBar, such
checks were temporarily disabled.

BUG: 28114251
Change-Id: Idb64fe97cf84f5f73e08e293b8fd0384bc8b70d6

Merge "Clear calling identity when enabling WebView fallback logic." into nyc-dev

Merge "ContextHubManager: Avoid bad NanoApp objects" into nyc-dev

Merge "Add systrace for resolveIntent()" into nyc-dev

Merge "Don't allow userID changes on upgrade" into nyc-dev

Merge "Add UserHandle to ShortcutInfo, and simplify LauncherApps APIs." into nyc-dev

Merge "Added getProfileIds method returning array of userIds" into nyc-dev

Merge "Support multiple Enrollment APKs" into nyc-dev

Merge "Use old-style pickers on sub-small screens" into nyc-dev

Merge "don't copy importance forward" into nyc-dev

Merge "Merge "Show volume safety warning for all users" am: 0a51955 am: 6cd66de" into nyc-dev

Merge "Fix QS accessibility issues" into nyc-dev

Merge "Dialog theme overlay should reset window fixed width and height" into nyc-dev

Merge "Reinstate much-reduced ConnectivityService logging." into nyc-dev

Merge "Make additional IME Subtype ID persistent." into nyc-dev

Fix QS accessibility issues

 - Add label to quick settings expand indicator
 - Add battery tile content description

Change-Id: I112c3ba23906a4afe8068acf916e710ec9d40917
Fixes: 28073200

ContextHubManager: Avoid bad NanoApp objects

We provide a preferred constructor for the user which will
always generate a serializable NanoApp object.  While we
still support the other path, we throw a specific exception
when trying to serialize a bad NanoApp object to ease
debugging.

Change-Id: I7ee610f0fce2f0dd489526e4819f66035281024b

Merge "Ensure we don't crash the system server when disabling/enabling package." into nyc-dev

Merge "Guard against crash in RTL." into nyc-dev

Merge "Retry compiling a previously failing package after update" into nyc-dev

Merge "Fix weird broken state from lock screen -> QS edit" into nyc-dev

Guard against crash in RTL.

Change-Id: Id692360cc66600a06a4353a8029440be422cc79f
Fixes: 27927374

Merge "Show volume safety warning for all users" am: 0a51955
am: 6cd66de

* commit '6cd66defe33ef84580aff5ab067cd5308d258c01':
  Show volume safety warning for all users

Change-Id: Ic417e1591a1a7b7876b5a5e4f75153eb1992fe66
(cherry picked from commit 683bda7cb228c0576fbeebcca6df3cd3c9a8ca6c)

Bug: 24884692

Fix weird broken state from lock screen -> QS edit

Change-Id: I852d28e0bbe682b85178163c362ed8d1b00a1eb7
Fixes: 28075240

Retry compiling a previously failing package after update

BackgroundDexOptimizer (BDOS) keeps a list of packages it attempted
to compile but dexopt failed. Once such a package is put on the list,
it is never removed from it until reboot.

This CL modifies BDOS to accept notificiations from
PackageManagerService on recently installed packages and removes
their names from the list.

Bug: 28082762
Change-Id: I0c3d3f657bd3ff5f0a61dfeef1d8174ed7be0b74

Dialog theme overlay should reset window fixed width and height

Bug: 28071435
Change-Id: Ib05fdd5294ebfb6448a992c8c7fe8106eacd8bfa

don't copy importance forward

copyRankingInformation should only copy side notes to be used by rankers.

Bug: 26919636
Change-Id: I6d0d87d4dc93ced65180ce58a4dcde2cac60e8b6

Merge "Revert "Throw security exception for non-parent aware APIs"" into nyc-dev

Merge changes Ic2c84de7,I379aef92 into nyc-dev

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

Reinstate much-reduced ConnectivityService logging.

Change-Id: I9d1292f8c036be2a854fdb537f4ccb957d211966

Use old-style pickers on sub-small screens

Bug: 28085161
Change-Id: I70eea06a1dc177484ed47f1ac59945e3d24a4973

Merge "DND related restrictions" into nyc-dev

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

Revert "Throw security exception for non-parent aware APIs"

This reverts commit bd7ae782ed815a1765b15852caf6137ee6bae47e.

Bug: 28107853

Change-Id: Ic286d9a181e96b8eb9826dab9ed297e0e4dfa6de

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

Make additional IME Subtype ID persistent.

It turns out that IME subtypes specified to
InputMethodManager#setAdditionalInputMethodSubtypes() are stored in the
presistent storate without subtype IDs.  As a result, when the system is
rebooted, the system would no longer consider those additional subtypes
as enabled due to subtype ID mismatch, until the IME re-adds those
additional subtypes again with the original subtype IDs.

Bug: 28104337
Change-Id: I1445213e0b83d76631a839b974ec1ab9b28ad7d2

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

Ensure we don't crash the system server when disabling/enabling package.

We shouldn't crash when trying to disable/enable a non-existent fallback
package.

Bug: 27918467
Change-Id: Ia94425ac197b17430900bb7a9e7b2d2bf65ab9b3

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