Add Settings Search Regression test

Take a snapshot of the currently available search results, and
verify that search results aren't accidentally removed.

We use 4 items to identify a search result:
- Title
- Data Key
- Slice Uri

We use Title & Key to identify the search result, since they should not change.
The Slice Uri is used to make sure we don't regress on Slice Availability.

Test: runtest --path packages/apps/Settings/tests/unit/src/com/android/settings/search/SettingsSearchResultRegressionTest.java
Change-Id: I22498229bbcd1e90c9e0a026af9df4367a98190a

Merge "Hook up feature flag to mobile network controller"

Merge "Use BluetoothDevice.ACCESS_* instead of CachedBluetoothDevice.ACCESS_*"

[automerger skipped] Merge "DO NOT MERGE Fix summary text for advanced button." am: da38b41b9a  -s ours am: 5c6f6949bf  -s ours
am: b984c81371  -s ours

Change-Id: I6d6584924602bf9f7d8f267cccfae8adce744f66

[automerger skipped] Merge "DO NOT MERGE Fix summary text for advanced button." am: da38b41b9a  -s ours
am: 5c6f6949bf  -s ours

Change-Id: I5857d75e1b068446ccfba8859c11e968bc7dade3

[automerger skipped] Merge "DO NOT MERGE Fix summary text for advanced button."
am: da38b41b9a  -s ours

Change-Id: I1dadf9bcd016e61ea04fe5b851e6430b1d63574d

Merge "DO NOT MERGE Fix summary text for advanced button."

DO NOT MERGE Fix summary text for advanced button.

- the resource for the summary text is in the support library. However,
the prebuilt version in the build does not contain the latest change for
the summary text. Add the overlay for the summary resource here with the
updated translation to remove the redundant summary prefix in the
advanced button.

Change-Id: Ib6db0c402d79a2d965e6060b1851d7b898ae3f94
Fixes: 112986476
Bug: 111195268
Test: manual

Merge "Check StorageWizardFormatConfirm runs smoothly"

Merge "Check getLastCustomNonConfigurationInstance() works well"

Check StorageWizardFormatConfirm runs smoothly

- confirm that StorageWizardFormatConfirm works well after
  changing calling method from showAllowingStateLoss() to show().

Bug: 111150236
Test: manual
Change-Id: I0ad6c54aaa2572fc5798ede900e0a6d10d2fbd35

Check getLastCustomNonConfigurationInstance() works well

- make sure that StorageWizardFormatProgress runs smoothly.
- confirm that mTask has the same object id when changing the screen
orientation.

Bug: 111151113
Test: manual
Change-Id: I730c1b0f08e8a20a1aafa668fb02cd5a09fdd70a

Use BluetoothDevice.ACCESS_* instead of CachedBluetoothDevice.ACCESS_*

1. Use BluetoothDevice.ACCESS_* instead of CachedBluetoothDevice.ACCESS_*
2. Use BluetoothDevice.setPhonebookAccessPermission() directly.
3. Use BluetoothDevice.setMessageAccessPermission() directly.
4. Use BluetoothDevice.getPhonebookAccessPermission() directly.
5. Use BluetoothDevice.getMessageAccessPermission() directly.
6. Use BluetoothDevice.getSimAccessPermission() directly.

Bug: 112517004
Test: make -j42 RunSettingsRoboTests
Change-Id: Ibe6b207b891b9bd2b328a2e2c7264a9a78cb498f

Merge "Fix special access summary text"

Merge "Remove unnecessary check when setting permission summary"

Hook up feature flag to mobile network controller

Send intent to phone process if the flag is false.

Bug: 113069948
Test: RunSettingsRoboTests
Change-Id: Ie9726470e718144557f318fe7ea28e863d63679c

Fix special access summary text

Use ApplicationState to count number of apps using unrestricted_data.

Change-Id: I083ff50e3e516536c87afa71d786b22e83d9a498
Fixes: 69313992
Test: robotests

Remove unnecessary check when setting permission summary

Activity null check is probably necessary when this logic is written in
fragment, now the logic is in a controller where all dependencies are
self contained, so we don't need to check activity null here.

Change-Id: Ifa32e0d6563c411f19bec07f176765a7c94c8a40
Fixes: 112289663
Test: robotests

Fix the time flicker in battery settings page

Don't clear the text when reciever has been re-registered in
onResume(), where quickUpdate has been called.

Change-Id: Ie056dfb225a1bf6c224932b310a97499fc26e8f0
Fixes: 69312991
Test: RunSettingsRoboTests

Merge "Import translations. DO NOT MERGE"

Import translations. DO NOT MERGE

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

[automerger skipped] DO NOT MERGE leverage default_theme string for display_cutout_emulation_device_default am: 12c9be1255  -s ours
am: c843167b3f  -s ours

Change-Id: I503f6e405a90d2e2b83794310da8f45bc4aa91c3

[automerger skipped] DO NOT MERGE leverage default_theme string for display_cutout_emulation_device_default
am: 12c9be1255  -s ours

Change-Id: Iaeffe4999365f975887b336598a7ab4bf078c7b9

Merge "Remove getExtraAction() from DashboardFeatureProvider."

[automerger skipped] Merge "Cutout emulation: string changes and ordering" into pi-dev am: 8d82158036
am: 80c53bce68  -s ours

Change-Id: Ie247f02047b3d1f646dbdd762e94d9daac6ff18b

DO NOT MERGE leverage default_theme string for display_cutout_emulation_device_default

Bug: 112876936
Test: manual
Change-Id: I1a41b96f9f19a5d5854eb1cce214616e303ca4f9

Merge "Remove condition manager v1 code."

Merge "Cutout emulation: string changes and ordering" into pi-dev
am: 8d82158036

Change-Id: I6d6c191e1dc511d2ab6e563f7bbea27962fe7b63

Merge "Cutout emulation: string changes and ordering" into pi-dev

Remove condition manager v1 code.

Change-Id: I866c1f7c493c6080ccf766ecfc8e6b00dafeecd9
Fixes: 112485407
Test: robotests

Merge "Cutout emulation: string changes and ordering"

Merge "Use title in ManagedServiceSetting dialogs"

Merge changes from topics "remove_tile_summary_cache", "remove_tile_title"

* changes:
  Use getSummary() to get tile summary text.
  Use tile.getTitle(context) to get tile title.

Cutout emulation: string changes and ordering

Updates strings according to spec. Also ensures
that the emulation overlays are shown in the
order of their priority.

Bug: 112876936
Test: Open developer options, go to "display cutout", verify strings.
Change-Id: If2d05595d02a277896202ab2a6262c99508a3a17

Cutout emulation: string changes and ordering

Updates strings according to spec. Also ensures
that the emulation overlays are shown in the
order of their priority.

Bug: 112876936
Test: Open developer options, go to "display cutout", verify strings.
Change-Id: If2d05595d02a277896202ab2a6262c99508a3a17
Merged-In: If2d05595d02a277896202ab2a6262c99508a3a17

Use title in ManagedServiceSetting dialogs

- Use the title (not summary) of the preference in its warning dialogs

Test: manual
Bug: 112687996
Change-Id: Ia217e5252cf7a48450da853c0270093ffb57dca7

Merge "Use searchable="false" to suppress nonIndexables."

Fix notifications crash

Change-Id: I75a34beb124de92c0d7d9033e3fe21dbdeb67ed3
Fixes: 112656503
Test: manual

Merge "Add RecyclerView for contextual settings homepage."

Merge "Refactor add network page"

Add RecyclerView for contextual settings homepage.

Bug: 111676964
Test: robotests
Change-Id: I36002a9085ca12ef027726c5a600ba833dd4f47e

Remove getExtraAction() from DashboardFeatureProvider.

It's always set to null and not doing anything.

Bug: 111427617
Test: robotest
Change-Id: I7172a31056950b1597f0b98625a3a2615305e466

Use getSummary() to get tile summary text.

Bug: 77600770
Test: robotests
Change-Id: Iecef09853bb49bc259502494912ed81d52e2d7ce

Use tile.getTitle(context) to get tile title.

And switch to getId() instead of title when comparing 2 tiles. This is
more accurate and more efficient.

Bug: 77600770
Test: robotests
Change-Id: I587d90702d98956bf7b420529ac3280351ca4a10

Merge "Update the method name from isA2dpDevice() to isConnectedA2dpDevice()"

Merge "Update the method name from isHfpDevice() to isConnectedHfpDevice()"

Merge "Don't offer "None" lock type for work profiles."

Merge "Refactor WifiSlice to CustomSliceManager"

Refactor WifiSlice to CustomSliceManager

Modify the WifiSliceCode to follow the pattern for
Slices that do not match existing UI components.

Test: robotests
Bug: 80263568
Change-Id: Id69e019608777282f4b64ff945e8c30c97aaf577

Update the method name from isHfpDevice() to isConnectedHfpDevice()

Bug: 110072747
Test: make -j50 RunSettingsRoboTests
Change-Id: I4f42a4d55a107c870c76761434f4e5999ac76904

Update the method name from isA2dpDevice() to isConnectedA2dpDevice()

Bug: 110072747
Test: make -j50 RunSettingsRoboTests
Change-Id: Id3ceb3a62b888a2bde82a1372edf26454e525ea1

Don't offer "None" lock type for work profiles.

"None" doesn't make sense for work profiles because it makes
profile insecure even if primary user is secure. If the user
wants to get rid of separate challenge, it will be offered in
previous Settings page, "Use one lock", so this shouldn't cause
any confusion.

Test: atest tests/robotests/src/com/android/settings/password/ChooseLockGenericControllerTest.java
Test: manual, tried setting personal an work challenges from Settings and via Intent.
Bug: 33656033
Change-Id: I830b3e372c1fe200fc4e02d59e3c3805bac5f9bb

Merge "Bluetooth: PBAP not disconnected on disabling contact sharing" am: 84bb2293ae am: ff985c6bc7
am: 0c75671a5f

Change-Id: Icc7d90a0e440d3d8f462d61821c602b26f79dbb7

Merge "Bluetooth: PBAP not disconnected on disabling contact sharing" am: 84bb2293ae
am: ff985c6bc7

Change-Id: I9c2c99a0f3999c550195b4ed7eed5a31caea9d2a

Merge "Fix indexing after slices api changes"

Merge "Bluetooth: PBAP not disconnected on disabling contact sharing"
am: 84bb2293ae

Change-Id: Ief10c8b5b3cdae90ba37a6349528f85e59562dc9

[automerger skipped] Merge "Fix memory leaks in Settings" into pi-dev am: be13e74deb
am: 6136194c4d  -s ours

Change-Id: I5499ae3769b11cc22b68f4c1033fcff9d82b21cb

Merge "Fix memory leaks in Settings" into pi-dev
am: be13e74deb

Change-Id: I51e9c6682a0589afbdfcc8799b10598eac496396

Merge "Bluetooth: PBAP not disconnected on disabling contact sharing"

Merge "Fix memory leaks in Settings" into pi-dev

Use searchable="false" to suppress nonIndexables.

When possible, remove or simplify getNonIndexable() logic in fragments,
and use searchable="false" in xml to suppress index.

Change-Id: I5bdf5bc7d5494a64cdd9e230a51321a4b210af69
Fixes: 112608186
Test: robotest and manual search

Refactor add network page

WifiDialog.java can be launched as "full screen mode"
in some use cases. However the way it's done is that
it uses theme to fake the full screen appearance.
It just feels hacky to use a dialog for full screen UI.
So, we created an "AddNetworkFragment" to make this
page can look like a normal fragment.

Also, clean up some useless code about "full screen mode"
of WifiDialog.

Change-Id: Iedd04c6a8e403cbceb872313314e1cba0d514246
Fixes: 111875856
Test: robo test, manual test

Merge changes Idb448ea3,I7dc26369

* changes:
  Fix crash on "Set up Wi-Fi NFC tag" dialog
  Fix bug about Wi-Fi dialog rotation

Merge "Remove LocalBluetoothAdapter from CachedBluetoothDeviceManager::addDevice()"

Fix crash on "Set up Wi-Fi NFC tag" dialog

When user launches "Set up Wi-Fi NFC tag" dialog
and then tries to rotate screen, device will
crash.
Because isShowing() of dialog won't return true
anymore when fragment calls onSaveIntance(),
we can't save status of dialog successfully.
And then, when fragment called onCreateDialog()
again, it can't get any dialog object.
So, we only check dialog whether is null or not.
If dialog is null, it means that there is no dialog
was shown before user rotates the screen.

Fixes: 112741721
Test: NFC tag wifi test, robo test

Change-Id: Idb448ea32c4215d8380c69bfd896cc91d8c1f8d1

Fix bug about Wi-Fi dialog rotation

When user clicks a Wi-Fi access point in WifiSettings,
screen pops up a Wi-Fi point dialog. And then user
rotates the screen, Wi-Fi access dialog changes to
"Add network" full screen dialog.

In old code, we check whether dialog is showing by
dialog.isShowing() in onSaveInstanceState.
For now, this design is not appropriate. Since isShowing()
won't return true anymore when fragment calls onSaveInstanceState.

So, we check dialog object whether is null or not now.
If dialog is null, it means that there is no dialog was shown,
before user rotates the screen.

Change-Id: I7dc26369c005f576fe679abc70327f6a02620935
Fixes: 112624846
Test: manual test, robo test

Merge "Always show items in options menu for Wi-Fi Direct settings" am: 1e160cc8bf am: 0173130d73
am: c31869a940

Change-Id: I545d38612f05c35949d37037c0744a57bcf8164f

Merge "Always show items in options menu for Wi-Fi Direct settings" am: 1e160cc8bf
am: 0173130d73

Change-Id: I7e70b3f2140301673b45beb3fb7ad7ed2d848a1d

[automerger skipped] Merge "Hide SettingsSlice provider" into pi-dev am: 041bfbd2b6
am: e48cf70a96  -s ours

Change-Id: Ibac67440b71883a94d9e008d2cbf395c5b9a2a2e

Merge "Always show items in options menu for Wi-Fi Direct settings"
am: 1e160cc8bf

Change-Id: Ie1edf750d2f205be97b07c84429e2c1468aa6c58

Merge "Hide SettingsSlice provider" into pi-dev
am: 041bfbd2b6

Change-Id: I9e9f0cfc71a6352f7adfa3072a097dd126f38a29

Merge "Add 'Reach to check' to lock screen settings"

Merge "Hide SettingsSlice provider" into pi-dev

Merge "Refined the StrictMode for CardContentProvider"

Merge "Check condition displayable state in parallel."

Merge "Launch work profile activity to add Autofill service"

Refined the StrictMode for CardContentProvider

- Modify CardContentProvider and add Mainthread checking

Bug: 111820446
Test: robotest
Change-Id: I7af25e8938b79c4c0fe225d58d59da4dde15ba45

Check condition displayable state in parallel.

This speeds up condition display by about 40 ms.

Bug: 112485407
Test: robotests still passes
Change-Id: Iac66354492496a9ece9178c438db6506e6fe7be5

Launch work profile activity to add Autofill service

ag/4666330 allowed choosing separate Autofill services for the main and work profile. However, the 'Add service' button for work profile always resolved action_view intent as the main user even when clicking the work profile preference. Now the intent is resolved to the right profile depending on the preference clicked.

BUG: 112610177
Test: Verified manually
Change-Id: I870346d7b5618e7c230edebd59ac7f52f8495d47

Merge "Declare "searchable" attribute for preferences."

Merge "Always show items in options menu for Wi-Fi Direct settings"

Merge "Fix typo in the SD Card Formatting screen."

Declare "searchable" attribute for preferences.

Now we can easily mark a preference nonIndexable in xml instead of
adding key into searchIndexProvider.

Bug: 112608186
Test: robotests
Change-Id: I0ff16d44bb7b6ad148d3d35f09ca0da0163f73f4

Merge "Hide SettingsSlice provider"

Merge "Add colors to whitelist"

Always show items in options menu for Wi-Fi Direct settings

Remove MenuItem.SHOW_AS_ACTION_IF_ROOM flag for menu items
to avoid showing truncated texts on action bar.

Bug: 112671955
Test: Manual
Change-Id: I1c9678321442169bc86d719e820d4af68261dee1

Remove test-purpose constructor from SavedBluetoothDeviceUpdater

- replace mock object with ShadowBluetoothAdapter

Bug: 111848213
Test: make -j50 RunSettingsRoboTests
Change-Id: Iff69ed511b23846078925609bb603ae414956ea3

Remove test-purpose constructor from AvailableMediaBluetoothDeviceUpdater

- replace mock object with ShadowCachedBluetoothDeviceManager to test CachedBluetoothDevice
- rename cachedDevices to mCachedDevices

Bug: 111848213
Test: make -j50 RunSettingsRoboTests
Change-Id: I3028a6fe06c39c48e7cee33976bdfcab2c8b73c8

Remove test-purpose constructor from ConnectedBluetoothDeviceUpdater

- replace mock object with ShadowCachedBluetoothDeviceManager to test CachedBluetoothDevice
- rename cachedDevices to mCachedDevices

Bug: 111848213
Test: make -j50 RunSettingsRoboTests
Change-Id: Ib024a3e9c3af745b1ab0be36361165a547cfa756

Merge "Refresh subscription info when subid changes" am: 5226ac6110 am: 114cacf2a8
am: 611d824110

Change-Id: I29492ef016c8ca9836ecf28c787e441629cfecb0

Merge "Refresh subscription info when subid changes" am: 5226ac6110
am: 114cacf2a8

Change-Id: If8e2d3a2045b7db65856673a7aed5192d7ff4bc9

Add colors to whitelist

export ANDROID_LINT_JARS=$(gettop)/prebuilts/checkcolor/checkcolor.jar
lint --check HardCodedColor --xml color-check-baseline.xml .

Bug: 112005540

Test: builds
Change-Id: Ibac23dda42e797fc01141cbe85d232ba5ea63139

Merge "Refresh subscription info when subid changes"
am: 5226ac6110

Change-Id: I874fb7e8f3d3146cb614de3ee088e93c1863a592

Fix typo in the SD Card Formatting screen.

credit: cl/208558166

TEST:none
Change-Id: I15e2f396ad1e1dd2364c9a2d38e3a34203e121ba

Fix indexing after slices api changes

Test: make -j40 RunSettingsGoogleRoboTests
      make -j40 RunSettingsRoboTests ROBOTEST_FILTER="Slice"
Change-Id: Ib599a8c814942a80d7b60fc3b40b284ce2b55775

Merge "Implement createPrefControllers for tether page"

Hide SettingsSlice provider

Explicitly set the Settings SliceBroadcastReceiver to
be non-exported and remove the intent-filter.

Add a second provider: SliceRelayReceiver to receive
broadcasts from SysUI to alert Settings to potential
changes to bound Settings Slices. The new receiver is
exported, but only notifies changes to Settings, and
doesn't make any changes itself.

Change-Id: I80d070f7636614135ebe4f57a16f12a3eb6dee81
Fixes: 111330641
Test: boot, robolectric, Slicebrowser

Hide SettingsSlice provider

Explicitly set the Settings SliceBroadcastReceiver to
be non-exported and remove the intent-filter.

Add a second provider: SliceRelayReceiver to receive
broadcasts from SysUI to alert Settings to potential
changes to bound Settings Slices. The new receiver is
exported, but only notifies changes to Settings, and
doesn't make any changes itself.

Change-Id: I422c0b07a61efa8996e9fdfa398eee84bbc1796f
Merged-In: I80d070f7636614135ebe4f57a16f12a3eb6dee81
Fixes: 111330641
Test: boot, robolectric, Slicebrowser

Merge "Refresh subscription info when subid changes"

Merge "Disable bg restriction for "Disallow apps control""