Merge "Make prefs without an action unselectable" into nyc-dev

Make prefs without an action unselectable

Setting selectable=false is required for proper A11y. Otherwise Talkback
will inform the user that the item can be activated.

Bug: 29025567
Change-Id: I2b61c622c2d69fa83640f7822349613f51f849d5

Merge "Revert "Add an entry in developer options for location tools."" into nyc-dev

Revert "Add an entry in developer options for location tools."

Bug:28251942

This reverts commit e1c1a13cba6d1d8d3ffe9f2031f6234dc4bc467d.

Change-Id: I864a813e8f6a9488b4db2f876a0097e947ceae45

Merge "onResume -> onStart, onPause -> onStop" into nyc-dev

Merge "Start DeviceAdminSettings in a new task from support dialog." into nyc-dev

onResume -> onStart, onPause -> onStop

For SettingsActivity and DashboardSummary

Bug: 28896214
Change-Id: I235d5c82b04b1363bf4416acb5cf5fc1c4db704f

Merge "Fix some top-level settings not updating with language" into nyc-dev

Merge "Cache account prefs so we don't lose a11y focus" into nyc-dev

Cache account prefs so we don't lose a11y focus

Fixes: 28767151
Change-Id: I876712c42b02bb2efaaa492ad951ad42935ecb32

Merge "Fix apps spinner jank." into nyc-dev

Fix apps spinner jank.

Change-Id: I7fbe580627538037fd0f1823787f7dcebbb43252
Fixes: 29056706

Start DeviceAdminSettings in a new task from support dialog.

Bug: 28998790
Change-Id: Ia43d6e719124e0908ec8ccb8c1ba7e80b5297096

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

Import translations. DO NOT MERGE

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

Disable restricted VPN actions onResume

Otherwise it's pretty easy to circumvent any future restrictions by
keeping VPN settings for a particular app open and continuing to use
it even after DISALLOW_CONFIG_VPN is applied.

Two checks:
 1) in onResume (catches almost every case)
 2) in setAlwaysOn (catches every remaining case)

Bug: 28733079
Change-Id: I1c67bb14891ef620df5ed65fbd32678e417b4a65

Merge "Show different strings when PO/DO user approves a cert" into nyc-dev

Merge "Fix RTL issue in Trusted Credentail page" into nyc-dev

Show different strings when PO/DO user approves a cert

1. This is confusing for restricted users who don't have any options other
than 'Trust'.
2. show dialog title of "Check certificate" instead of "Trust or remove certificate"

The change only applies to ProfileOwner and DeviceOwner case

Fix: 28875819
Change-Id: I3f7dc8f3dcac75b4d99814cbc9af67433676a8c7

Fix RTL issue in Trusted Credentail page

Two changes are required to fix the issue.
1. The width of the textview needed to be wrap_content in order
   to resolve the random issue. I guess it is a framework bug, but
   anyway the UI looks the same after the change
2. textAlignment="viewStart" is needed to make the textview RTL
3. Clean up: Change the RelativeLayout to a vertical LinearLayout

Bug: 28757193
Change-Id: I1142d4dd31eb86b80f5a2316c1c7825df287b609

Fix some top-level settings not updating with language

They need to be resources so they can load correctly on locale
change.

Change-Id: Iea167b8df411528185987d37704717d9c0b9f254
Fixes: 28861758

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

Merge "Show only up to two locales in Settings" into nyc-dev

Import translations. DO NOT MERGE

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

Merge "Fix pref availability for search" into nyc-dev

Fix pref availability for search

Change-Id: Ic080e131c6c751456b22d370aade77748d912b6b
Fixes: 28987828

Merge "Don't allow restricted data usage of system" into nyc-dev

Don't allow restricted data usage of system

Cause it will just crash

Change-Id: I4ee43633eb3fa488816b1d2767073cbe404c2c33
Fixes: 28979013

Show only up to two locales in Settings

The list could otherwise grow very long, spamming the screen.

Bug: 28872122
Change-Id: I8c4dbe659e1c088eba8b5ba1b2905b5eb8bc0229

Merge "Force dashboard summary to single-line and ellipsize it" into nyc-dev

Force dashboard summary to single-line and ellipsize it

Previously, the summary could become multiline. In rare combinations
of text and screen width, to indicate to the user that the text is
cut, we now ellipsize it too. Finally, since the ellipsis can get
too close to the end of the line, we add a padding equal to the left
margin of the icon.

Bug: 28872122
Change-Id: I27226e5de7781cf096508ddbef4f9095961ce028

Merge "Add activity label to encryption interstitial" into nyc-dev

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

Merge "Importance slider updates." into nyc-dev

Merge "Fix crash from battery saver switch" into nyc-dev

Importance slider updates.

Icon & tint.

Bug: 28911161
Change-Id: Ic56270c8803557863f914ee0eaf6801f1341387d

Fix crash from battery saver switch

Turns out most things expect the conditions to be loaded immediately
so if the dashboard hasn't been hit, they crash.  Instead load
immediately for everything but the dashboard.

Change-Id: Iaa1114c88b3766e2ac513acb417ef2a55a0f4e7f
Fixes: 28952354

Merge "Use LockPatternUtils.resetKeyStore() to clear keystore" into nyc-dev

Merge "Fix Work Challenge / personal confirm credential is shown on top of keyguard after rotation" into nyc-dev

Merge "Use getConfigForSubId instead of getConfig(int)." into nyc-dev

Import translations. DO NOT MERGE

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

Merge "Guard against IllegalArgumentException" into nyc-dev

Use getConfigForSubId instead of getConfig(int).

Per API review, we want to revert getConfig(int) to
getConfigForSubId(int) (ag/1075011). This CL changes all
callers to use getConfigForSubId(int).

Bug: 28295813
Change-Id: I0e44a5e758aed34ce26740632270700bde66cff6

Merge "Don't crash trying to disable lockdown VPN" into nyc-dev

Guard against IllegalArgumentException

Bug: 28742227
Change-Id: If436ec3f743ca11ea40dce2033b7b993cb6ad170

Merge "Skip indexing content while user is locked." into nyc-dev

Don't update conditions on background threads

Instead build a list and set it on the main thread to avoid
concurrent modification crashes.

Change-Id: I936fbeb70359583e55ae8ddcb0ea46b5753cb3b0
Fixes: 28794277

Skip indexing content while user is locked.

Otherwise we try touching printing services which throw.

Bug: 28912897
Change-Id: I25638794b35fd383fbe67604ac921d16743628ad

Hide work profile key in user credentials screen

Bug: 28878708
Change-Id: Ic9733ea973e1cc37028cb5f740969b5e24fb5aef

Add activity label to encryption interstitial

So that TalkBack refers to the screen by this more specific name.

Bug: 28868846
Change-Id: I93139959dcdbd156d57f01699283e01b178305b8

Merge "Remove option to configure emergency app" into nyc-dev

Fix Work Challenge / personal confirm credential is shown on top of keyguard after rotation

ConfirmDeviceCredentialBaseFragment.SHOW_WHEN_LOCKED has been abused.
We should not set onfirmDeviceCredentialBaseFragment.SHOW_WHEN_LOCKED
for work challenge and most other cases.

It's temp fix for the rotation bug. Because it's inconsistent the activity is shown
on top of keyguard with/without rotation (or any config change). In long term,
SHOW_WHEN_LOCKED should be set only if it's truly on purpose, and remove
isKeyguardLocked checking.

Bug: 28878075
Change-Id: I19db913f5bcaf2644c8c46c25e9584103d7fee8a

Merge "Trust the certificate after unlocking screenlock" into nyc-dev

Use LockPatternUtils.resetKeyStore() to clear keystore

Bug: 28878708
Change-Id: I6730b6947ea917772809bb045470bdf5e95d86ba

Trust the certificate after unlocking screenlock

When trust button is clicked, if ConfirmCredential (CC) is shown,
and user successfully unlock CC, trust the cert immediately

Bug: 28752364
Change-Id: Ied4aeda59a668a9dd2bf079a385b1fecd8eabb9e

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

Merge "Fixed whitelist stats so it's only logged on add." into nyc-dev

Import translations. DO NOT MERGE

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

Merge "Fix Continous loop in unifed screenlock when trying to Trust a CA cert" into nyc-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 28857042
Change-Id: I2ef0467d8a959436073642564afcb8fd25d3d09e

Remove option to configure emergency app

Bug: 28873644
Change-Id: Iddf4e781af8cb3ca3b62d1141c58402933bd8ec8

Merge "Defined help page for Data Saver." into nyc-dev

Fixed whitelist stats so it's only logged on add.

BUG: 28166693
Change-Id: If1f91167c21054e78a966670c20c174c3f73d2a5

Merge "Protect against monkey crash" into nyc-dev

Protect against monkey crash

When rebuild comes in after context is no longer valid

Change-Id: Ib64e93372bff4c6b0e1232ac257f3df0e9d0be0a
Fixes: 28875397

Fix Continous loop in unifed screenlock when trying to Trust a CA cert

Cause: with unified screenlock, ConfirmDeviceCredentialActivity didn't
forward result with FLAG_ACTIVITY_FORWARD_RESULT

Also, fixed that ConfirmDeviceCredentialActivity didn't allow fingerprint
authenication in unified screenlock after keystore unlocked.

In ChooseLockSettingsHelper, add one new util function to allow
extra option to set returnCredentials to false while external to true.

Set StrongAuth to "not required" when it has been successfully unlocked.

Test:
1. PO Unified Screenlock/Work Challenge x fingerprint -> ok to trust cert
   (Also, no credential is returned in intent)
2. WorkMode off -> Reboot -> turn on Work mode
   -> no fingerprint option, PIN unlock successful to turn work mode on

Bug: 28752364
Change-Id: I6dc8865e8f005545f8577d7731afb4495647062b

Merge "Temp fix for NPE when race condition happens in TrustedCredentialsSettings" into nyc-dev

Temp fix for NPE when race condition happens in TrustedCredentialsSettings

Bug: 28762124
Change-Id: I21e46c2008e106db1c43573171e0d852cbe0a68a

Defined help page for Data Saver.

BUG: 28865521
Change-Id: I07930232312df2775ecee5beacee6921f7245847

Don't use ApplicationsState for summaries

Since it grabs a lock that can be slow on the main thread, don't use
ApplicationsState in any of the summaries, instead load the information
directly from the PM.

Change-Id: Ibefe867810d2a9926177a8de4e23a7faea4b1c3b
Fixes: 28435146

Handle return value when Bluetooth is enabled

If the Bluetooth enable(true) returns false currently we are waiting for
a broadcast (which may not happen if enable returns false). We mark the
toggle to have failed (i.e. it is clickable and in OFF state) and the
switch bar text is appropriately shown.

Bug: b/28318203
Change-Id: I09ba46d2e102e7f2c4ef9a72bf38dedc1346364f

Merge "Update work challenge background image." into nyc-dev

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

Update work challenge background image.

Updated so the orange briefcase in the image
isn't central so avoiding clashing with text.

Bug: 28557192
Change-Id: Ic8e4ea71f47449e81619b28b979f59c3d9aaf6a0

Merge "Null check." into nyc-dev

Follow class package change: LocaleList

In order to avoid layering violation, LocaleList needs to be moved from
android.util package to android.os package [1].  This CL follows up that
package change.

No behavior change is intended.

 [1]: Ia8de2ee9df3dd0a42b1fe84574439519b680fe18

Bug: 28819696
Change-Id: Ibd7934b30062046830d63f33d1c6febef32da976

Merge "Choose Bluetooth Device window cant't fill screen" into nyc-dev

Null check.

Change-Id: I16dfe2cf265b4254aa69bc6165a8f72fc3954d7a
Fixes: 28531037

Choose Bluetooth Device window cant't fill screen

b/28684861: fixed the layout file to ensure that the separators cover
the entire screen.

Change-Id: I4471a1da901c9a4c0830ff0dfc943cd087ee1702

Merge "Implemented onRestrictBackgroundBlacklistChanged()." into nyc-dev

Merge "Modify the TTS engine preference UI so that the radio button and the TTS engine name are a single focusable element." into nyc-dev

Don't crash trying to disable lockdown VPN

Bug: 28624328
Change-Id: I2b726453da43feaf5f46d45679a702a52e6f4df0

Modify the TTS engine preference UI so that the radio button and the TTS
engine name are a single focusable element.

Bug: 27136946
Change-Id: I502634e0942b9fe30e290ce9de6ba293f33bf863

Import translations. DO NOT MERGE

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

Implemented onRestrictBackgroundBlacklistChanged().

BUG: 28743623

Change-Id: I2d8620360858660fed0e745b16dc61ce262b90b9

Merge "Fix crash in SetupRedactionInterstitial." into nyc-dev

Merge "Fix crash on rotation." into nyc-dev

Fix crash on rotation.

Change-Id: Ife0a040bfc1a6ecc579d544f78cc45e37f5bca3c
Fixes: 28767083

Fix crash in SetupRedactionInterstitial.

Change-Id: Ia662cd60e4c9eeaccd0eb168431e755c2c79b676
Fixes: 28785168

Merge "Catch rule creation exceptions." into nyc-dev

Catch rule creation exceptions.

Bug: 28775583
Change-Id: I54c4af0352c88b27717660741709ec54e4749656

Rename requestKeyboardShortcutsHelper

Per API Council feedback

Bug: 28775576
Change-Id: Ia19668dceb7da2222e9716fcb1caff8d0ee86c41

Merge "Guard against exception when getting package installer info." into nyc-dev

Guard against exception when getting package installer info.

Bug: 28742227
Change-Id: I8742c882ae6b18cb8f51a9da04170358a20a7d48

Merge "ShortcutManager: Remove developer option from NYC" into nyc-dev

Merge "Don't crash on invalid numbers." into nyc-dev

Don't crash on invalid numbers.

Change-Id: I5c1bff5733c5db5f411c2827177b0c1efa1e120e
Fixes: 28756890

Merge "Fixed calls to update() when configuration changed." into nyc-dev

Merge "Removed onRestrictPowerChanged()." into nyc-dev

ShortcutManager: Remove developer option from NYC

Bug 28704708

Change-Id: Ib2301ea556104e9ffd6a18885e228717843d6acb