Fix when vr app launches 2d activity

When a 2D app is launched from a VR app, since the VR app was on
DEFAULT_DISPLAY, the 2D app is also moved to DEFAULT_DISPLAY. This needs
to be addressed by moving any non-VR 2D app launch in VR mode to the
virtual display.

Bug: 37111307
Test: android.server.cts.ActivityManagerDisplayTests
Test: #testVrActivityLaunch
Test: #testVrActivityReLaunch
Test: #testPostVrLaunch

Change-Id: I83bffe94bb89415a6b3c026dd7077e70fe36fee0
Signed-off-by: Karthik Ravi Shankar <karthikrs@google.com>

Merge "FingerprintService: check current user correctly" into oc-dev

Merge "Special-case English for DateTime and Time key listeners" into oc-dev

Merge "Revert "Do not make strong reference to Resources."" into oc-dev

Merge "Add a carrier option to hide the option of "Enhanced 4G LTE Mode"" into oc-dev

Revert "Do not make strong reference to Resources."

This reverts commit 2d2315a66d81d29896e6a0ffedbfbbed3981b9ed.

Reason for revert: possible cause for b/37156733

Change-Id: I02d468449f0632d1281ab5ad2a381e7e36a5b0ea

Merge "Color management, the missing pieces" into oc-dev

Merge "Don't delete new notification channels not present in backup" into oc-dev

Merge "Remove WebView-fallback code for handling the non-functional stub." into oc-dev

Merge "Fix theming on brightness dialog" into oc-dev

Merge "Revert "Ignore a broken test until it can be fixed."" into oc-dev

Merge "Make sure to synchronize on references to plugin dependencies" into oc-dev

Merge "Synchronize on sysui dependency references" into oc-dev

Merge "AOD: More VisD updates" into oc-dev

Merge changes If96c0146,I0d68ad60 into oc-dev

* changes:
  Improved accessibility behavior of the notification shelf
  Fixed a bug where the indeterminate progressbar janky

Merge "remove deprecated API sendDialerCode" into oc-dev

Merge "Check component before notifying activity is visible." into oc-dev

Color management, the missing pieces

Implement missing color management pieces for bitmaps:

- Bitmap.createBitmap(Bitmap src, ...) now creates a bitmap
  in the same color space as the source bitmap
- Bitmap.createScaledBitmap() now creates a bitmap in the
  same color space as the source bitmap
- Bitmap.createBitmap(..., ColorSpace colorSpace) to create
  bitmaps in a specific color space
- Fix copy from A8 to F16
- Copying bitmaps in F16 or with a color space does not work,
  it's currently a limitation in Skia

Bug: 36905374
Test: BitmapColorSpaceTest
Change-Id: I0092fe4432511db50daa3a9393389a9db05e0c2a

Merge "Removed unused variables in PerformBackupTask#finalizeBackup()" into oc-dev

Merge "Expose package_verifier_user_consent to Instant Apps" into oc-dev

Merge "Fix format for VR virtual display backing buffers" into oc-dev

Merge "Change requestWait API according to request" into oc-dev

Remove WebView-fallback code for handling the non-functional stub.

We have already removed the non-functional WebView stub, so we can now
remove the logic ensuring that a primary device user could use any
valid WebView package even when a secondary user did not have any
WebView package to choose from (since all device users should always
have a valid package to use as WebView implementation).

Bug: 34730873
Test: run WebViewUpdateServiceTest tests.
Test: Create new user, disable Chrome for that user, ensure WebView
loads for that user and for the primary user.
Change-Id: Ia77cc2efd9e40a157fb5b68a988425ef15508033

Merge "Add NETWORK_STACK permission" into oc-dev

Merge "Omit using ctor of PreferenceViewHolder in tests." into oc-dev

Removed unused variables in PerformBackupTask#finalizeBackup()

The unused variables got left behind after changes in ag/1841867.

Bug: 35994670
Test: it builds
Change-Id: Ic28719dafb83542d7e4c73cb48508d6792d10174

Merge "Do not make strong reference to Resources." into oc-dev

Expose package_verifier_user_consent to Instant Apps

Test: manual
Bug: 37119785
Change-Id: I0a6deb96f1267f3f80bcf017d898fcf365515353

Add NETWORK_STACK permission

Add a new permission that only allows other system services to call
networking related methods.  This permission is intended to prevent
other apps from calling these methods that might leave networking
services in intermediate states.

Bug: 36538042
Test: build, run wifi unit tests, boots, wifi connects
Change-Id: Icba6ad5280e9a8863bd1c706ca34b7add8de5294
(cherry picked from commit fa84050a7e51e2a6643ea242b43f16cd6e3114dd)

Do not make strong reference to Resources.

This CL removes the strong reference added for mapping display ids
and Resources to Displays. Instead, the key pair is now the display
id and ResourcesKey, and the mapping is pruned when key is
invalidated.

Change-Id: I60d767b52de7bbf769f6761f5a3301dd7aff6ddf
Fixes: 36625868
Test: make -j32 cts; cts-tradefed; run cts --module CtsAppTestCases --test android.app.cts.DisplayTest#testRotation

Merge "NetworkPolicyManagerService: turn off rule debugging" into oc-dev

Merge "Return inet addr for discoverable printers" into oc-dev

Merge "Who wants...to live...forever? The "Save for Autofill?" dialog does!" into oc-dev

Merge "Log excessive remote callbacks" into oc-dev

Merge "Check for a null packageInfo before notifying about app install" into oc-dev

Merge "Revise Accessibility button chooser intent/permissions" into oc-dev

Merge "Add UPDATE_TZDATA to config updater intents" into oc-dev

Merge "Tuner plugin settings updates" into oc-dev

Merge "Added onMovedToDisplay method to WebView" into oc-dev

Change requestWait API according to request

Test: USB Device CTSVerifier test
Change-Id: Ib55bed248d7b37c6f5f4a7be7ec2d90a7ac396c7
Fixes: 36555805

Who wants...to live...forever? The "Save for Autofill?" dialog does!

Fixes: 37083017
Test: manual verification
Test: CtsAutoFillServiceTestCases pass

Change-Id: Id177fb59810956e4b43c1d357de415d64ab9d89a

Merge "Add metrics for nav buttons through KeyButtonView" into oc-dev

Merge "NotiViewWrapper: Fix test that breaks on master" into oc-dev

Merge "Reconcile secondary dex paths when compiling with adb shell cmd..." into oc-dev

Merge "Pass SElinux info to installd dexopt" into oc-dev

Merge "Add system config to specify Multi-Display support" into oc-dev

Merge "Fix crash when using list of enabled a11y services" into oc-dev

Merge "Fix injected settings titles not matching a locale change." into oc-dev

NetworkPolicyManagerService: turn off rule debugging

Left over from Change-Id Ic75d4f7a8853e6be20e51262c4b59805ec35093a.

Test: no functional changes
Bug: 37084734
Change-Id: Ifb10b4fe146bb6bea6260d8cd723a8e7f5b853a9

Log excessive remote callbacks

Bug: 36778087
Test: N/A
Change-Id: Ifb02fe09e3c0869f7f6c741f886421064e5c1b8a

Merge "SurfaceControl: Crash due to invalid JNI cast" into oc-dev

Revert "Ignore a broken test until it can be fixed."

This reverts commit 1ee48434efc8129c604b20eae90605b0098f674e.

Change-Id: Ie51d2931a6bafb187902c6ac845066ca70937f2c

Merge "Set android.display thread to top-app cpusets and schedtune group" into oc-dev

Added onMovedToDisplay method to WebView

Delegate the action to WebViewProvider, by default it is no-op.

Bug: 36006397
Test: There is no implementation yet to test.
Change-Id: Ib5101d3669a92ae81cfb34cc5db607c374712a3d

Merge "Add intent to intialize programs on APP installed" into oc-dev

Merge "Don't start a partition when session is pending FillResponse authentication." into oc-dev

NotiViewWrapper: Fix test that breaks on master

oc-dev allows constructing views with a null context, but
that doesn't work any longer on master.

Bug: 36430936
Change-Id: If069f81d54e3e3b9145ca5187b87888cd6173c0f
Merged-In: If069f81d54e3e3b9145ca5187b87888cd6173c0f

AOD: More VisD updates

- Change alpha for clock on AOD2
- Increase header bottom margin on ambient notification

Fixes: 37089014
Bug: 30876804
Test: visual inspection
Change-Id: If7ba1a51fdce3167129e8d8845bcb4a09b22a78d

Merge "Preserve app window when it requests a relaunch" into oc-dev

Merge "Add lower case charging string." into oc-dev

Merge "Removed deprecated onFillRequest() method" into oc-dev

Add metrics for nav buttons through KeyButtonView

Doesn't cover home-long-press or overview, but those appear to be
covered by other events.

Test: runtest systemui
Merged-In: I590b52765aaef3c2bf573ce8ab980aa992b1d5b0
Change-Id: I630356245e223a3fb8d4d2fe11be5d498227c0e3
Fixes: 25931698

Merge "Use hwservicemanager getTransport." into oc-dev

Merge "Add showChargingMethod in BatteryInfo" into oc-dev

Merge "Add tests for verifying network availability on activity start." into oc-dev

Merge "Fix testWaitForNetworkStateUpdate." into oc-dev

Special-case English for DateTime and Time key listeners

Because English time patterns use uppercase letters by default (and a
comma to separate date and time when both are represented), we were
concluding they need internationalized input. Although they
literally do, let's keep the world simple and assume they don't need
internationalized input.

Compared to Nougat, we will now accept uppercase AM and PM and comma
for English if the IME allows them, we just continue to not signal
that an internationalized layout is needed.

Test: CTS tests pass
Bug: https://code.google.com/p/android/issues/detail?id=2626
Bug: https://code.google.com/p/android/issues/detail?id=82993
Bug: 8319249
Bug: 33276673
Bug: 34394455
Bug: 37079150
Change-Id: I82bfde323ba49ae1a27ff5c2e729063b7d81dcc8

Fix injected settings titles not matching a locale change.

The injected tiles code currently treats the string metadata value
for the title key in the Activity's manifest as the title itself.
When the language on the device is changed the title remains in the
previous language until a reboot.

This CL will make injected tiles attempt to treat the string metadata
value as a resource variable to get the integer res id and then
properly load the localized title using the res id. To be backwards
compatible, if the res id is 0, we treat the string metadata value as
the title itself (current behavior).

Bug:36470694
Test: make RunSettingsLibRoboTests
Change-Id: I3963534a3a3a27d0455c4ed1ad4fa574b03d67fc

Merge "Rename getIntent() to getRenderIntent()" into oc-dev

Merge changes I6f00614d,I987eeab2 into oc-dev

* changes:
  Convert 16 bit bitmaps to 8 bit on devices that do not have GLES3.0
  Fix HardwareBitmapTests.testBitmapConfigFromRGB565

SurfaceControl: Crash due to invalid JNI cast

The (void*)buffer.get on ARM32 is 4 byte, so the calling convention
will put the argument in [sp, #12]. However, the caller actually
expects a long (the signature of gGraphicBufferClassInfo.builder),
which means it will expect it to be in [sp, #16]

Test: Tested on mtk device
Fixes: 36631082
Fixes: 36974487
Change-Id: I0f723125e612d096c0d76ca3360d895f3f23f286
(cherry picked from commit 98dd5d9a85e8911cf41dea6198d4111f737a5892)

Set android.display thread to top-app cpusets and schedtune group

android.display being in the foreground cpuset group is an issue. As
seen on M/S, during heavily CPU load it is not given core 3 even though
it might be free and causes jank. This patch adds the thread to the
top-app group to ensure it is placed on all cores during scheduling
decisions.

Doing this required a couple of changes:
- new API to set per-thread cpusets
- changes to DisplayManagerService to set the thread to top-app group
- changes to SystemServer to set the policy toward the end, as doing it
  during start of the DisplayManagerService was in issue (issue being
  SystemServer calls setSystemProcess.. -> setProcessGroup which overrides
  the group settings for threads in the system server process, including
  android.display)

Bug: 36631902
Test: Boot and make sure android.display thread is in the top-app group

Change-Id: Icc394ea0ffcf159d11728ad38de114234a29d20f
Signed-off-by: Joel Fernandes <joelaf@google.com>
(cherry picked from commit 474d311cb098e86c078c3f615e1161e2854f1847)

Merge "Use additions/emoji-data.txt instead of hardcoded values" into oc-dev

Merge "Force direction of phone number fields to (practically) LTR" into oc-dev

Merge "Shows a Toast when manual autofill request cannot be fulfilled." into oc-dev

Add UPDATE_TZDATA to config updater intents

Change-Id: I5fcd011e0ab030ebf1aa40818a05853117cc6811
Fixes: 36977046
Test: checkbuild

Merge "Update comment on why Keep is needed." into oc-dev

Don't start a partition when session is pending FillResponse authentication.

BUG: 35707731
Test: fixed testFillResponseAuthBothFields and testFillResponseAuthJustOneField

Change-Id: Ib74e46dd37feb7e5c03e40cfbe8133ace6db0709

Merge "Add a service to rank apps for ResolverActivity." into oc-dev

Add lower case charging string.

The previous charging string is "75% - Charging", this cl add lower
case string to make it "75% - charging".

Bug: 36558216
Test: Build
Change-Id: I683843de0b6cbd45bea6a465c7b6d8ee0b7c4f93

Merge "Revert "Remove SystemProperty gating-flag for virtual displays."" into oc-dev

Add showChargingMethod in BatteryInfo

This toggle is used to decide whether to show charging method.
If it is off, then "Charging over ac" will become "Charging"

Bug: 36558216
Test: RunSettingsLibRoboTests
Change-Id: I98b1f7e7b77d7ce0a7e1aae29e55a33ac79ff0d7
(cherry picked from commit b3c33f63ed120cf4758ec67426bb45291e289fce)

Merge "Disable PiP when VR mode starts." into oc-dev

FingerprintService: check current user correctly

FingerprintService maintains a mCurrentUserId field which keeps track of
the active group/user id in the underlying daemon. This field, however,
can become invalid after the daemon crashes and before
FingerprintService reconnects to a new daemon. (mCurrentUserId is only
set upon updateActiveGroup().)

Previously, mCurrentUserId is wrongly used in the logic of
isCurrentUserOrProfile(), resulting in false negatives when
mCurrentUserId is invalid. In this CL, we use
ActivityManager.getCurrentUser() instead, which has no dependency on the
state of the daemon.

This will also reduce the complexity around mCurrentUserId, and
hopefully will make future clean-ups easier.

Test: manually kill daemon and verify fingerprint functions normally
Bug: 36152864
Change-Id: I4250fad0e7c88bde95a1da1c89e71b4931ee3b8a

Merge "Revert "Handle case when snapshot dimensions don't match"" into oc-dev

Merge "Xform bitmaps to sRGB on SW and PDF canvases" into oc-dev

Merge "Refactor VR state management in ActivityManagerService." into oc-dev

Force direction of phone number fields to (practically) LTR

Previously, phone number fields defaulted to locale direction, which
created problems for apps that were not aware of LTR requirements for
phone numbers.

Now, we look at the direction of the digits for the locale, and use
that to determine the direction of the edit field. (For practically
all major RTL locales, that direction is LTR.)

Test: Manual
Bug: 33643035
Change-Id: I17c70d8462bd403ea6866057971105f1f5772ba3

Merge "Do not show UI when currentID changed." into oc-dev

Merge "Expose ConfigUpdater Intents in SystemApi" into oc-dev

Merge "change how instant app installer is updated" into oc-dev

Merge "ConnectivityService: safer locking" into oc-dev

Rename getIntent() to getRenderIntent()

Bug: 37077433
Test: compile and ColorSpaceTests
Change-Id: I8eb1fa6136582eaac1d509ebfef7f67b0fffe1aa

Merge "Fixed Autofill NPE that crashed some apps." into oc-dev

Convert 16 bit bitmaps to 8 bit on devices that do not have GLES3.0

Bug: 37077308
Test: CtsUiRenderingTests (ran to manually emulate 2.0 devices)
Change-Id: I6f00614d79797835adcfe4716bd331573e1463e3

Fix theming on brightness dialog

Test: visual
Change-Id: Icee6f98a86e7cfb1e1fe0fb34752e84a7ac9bbda
Fixes: 35960463

Synchronize on sysui dependency references

Test: make
Change-Id: I913505241bef231dd0b80de7f1b38ed17fa87d69
Fixes: 36779051

Return inet addr for discoverable printers

... by extending the RecommendationInfo to store the inet addresses not
just the count. Thankfully the recommendation plugins already contained
all the printers, not only the count.

Also expose the print service state as system APIs as they will be used
by the default print service.

Bug: 35765644
Test: Ran AddPrinterActivity on network that had printers to discover
Change-Id: I929a177adb75f6848848ba30a472cf0343d2e67e

ConnectivityService: safer locking

This path changes a dangerous lock path in reportNetworkConnectivity().
This methods is called outside of the main ConnectivityService handler
and takes a lock on a specific NetworkAgentInfo whose connectivity
status is being reported.

While this lock is held, reportNetworkConnectivity() goes on and query
the network policy state for that network, which may ends into
NetworkPolicyManagerService.

Instead, the lock on NetworkAgentInfo is only held long enough to make a
copy of LinkProperties, which is then passed to
NetworkPolicyManagerService without that lock.

Bug: 36902662
Test: could not repro b/36902662, reportNetworkConnectivity() works.
      $ runtest frameworks-net

Change-Id: Iac4b75bcecbdddb0ac695c8b1a87ae755f62f47f

Make sure to synchronize on references to plugin dependencies

Test: make
Change-Id: I361cb3e4ac4ff5d19e2b0322531c8c614f5b7635
Fixes: 36867744