Fix an unintentional switch-case fall-through

This is a follow up CL to my previous CL [1], which introduced a
classical fall through bug due to a missing return statement.

 [1]: Ic614f112f960382280acd8891b3af56d47679f08
      ce82eb2d865e5ee1b69705ca7a0aedd0e92d0d4e

Test: Manually made sure that calling
      InputConnection#performPrivateCommand() no longer causes
      IllegalStateException("Already recycled.").
Bug: 35079353
Fixes: 35681994
Change-Id: I9aa3c0497385e053718d829aad49495771b22b59

Merge "Remove abstract from AccessibilityService methods"

Merge "VideoView: option for audio focus, support for AudioAttributes"

Merge "Don't copy ringtones when profile sync goes off"

Merge "renaming telephony internal hidden intents" am: 09c663a84c am: 979e4690ec
am: 201320492c

Change-Id: I4e37359a98fb8ecdff63b45a54f08b61e7be25ad

resolve merge conflicts of ac9b2c4fab42 to stage-aosp-master am: 4fb8988854
am: 609cfc916b  -s ours

Change-Id: Ib912a18c3bea70d268cc6fa8e3de8bbb9fd414b0

Merge "OMS: integrate OverlayManagerService into framework"

Merge "resolve merge conflicts of f17c1d488840 to master"

Merge "Reconcile apps in 2 phases"

Merge "Add queryStatsForPackage() API."

Merge "Start locking down /data/media access."

Merge "Slowly deprecate hidden getPackageSizeInfo() API."

Merge "Fix concurrent modification crash in onAlarm"

Merge "Add missing sound to important channels."

Don't copy ringtones when profile sync goes off

Experimentally, it makes more sense to more people to have the parent
setting as an overlay not a concrete thing.

Test: make cts -j30 && cts-tradefed run cts --module CtsDevicePolicyManagerTestCases --test 'com.android.cts.devicepolicy.ManagedProfileTest#testRingtoneSyncAutoDisableRingtone' </dev/null 2>&1
Bug: 34730524
Change-Id: I5f804713def9e54921b90e4f5cea742ba8aaa685

Merge "Add "Use open Wi-Fi automatically" setting."

Merge "context_hub HAL uses "default" service name"

Merge "renaming telephony internal hidden intents" am: 09c663a84c
am: 979e4690ec

Change-Id: I8cce065cc9ce6a87fa0ce9753a5e9170da77da59

Merge "renaming telephony internal hidden intents"
am: 09c663a84c

Change-Id: Ic20ae9e6816d629c979859a7d18f1827bafcc034

Merge "renaming telephony internal hidden intents"

Fix concurrent modification crash in onAlarm

Change-Id: Idfd094f3c9ea59356440d6851ccd5abda36ca6ba
Fixes: 35640585
Test: manual; after boot, opened the power menu to force
system_server to spin up a ThreadedRenderer instance
(and thus register itself as a callback on GraphicsStatsService).
Then manually set the date forward by a day to trigger onAlarm
and verified the system didn't reboot/crash. A systrace capture
verified that the alarm fired and package:android (system_server)
had a log rotation event.

resolve merge conflicts of ac9b2c4fab42 to stage-aosp-master
am: 4fb8988854

Change-Id: Ic679f452ad245cbe58ea8069c34c15e50ee9e5c2

Merge "Fix build."

resolve merge conflicts of ac9b2c4fab42 to stage-aosp-master

Change-Id: I4a6094c41ab267f89a15c5e8caa25e6b683419d0

Merge "Fix a memory leak in AutoFillManager"

Merge "TelephonyManager#setAllowedCarriers: comment update"

Merge "Fixed missing auto-fill save callbacks on TextView."

Reconcile apps in 2 phases

During boot app data folders are reconciled in 2 phases:
 - in the constructor only core apps are reconciled. prepareAppData
   for remaining apps is deferred and run on a separate thread (phase 2)
 - Phase 2 must finish before third-party apps can start

Also moved GC to final stages of system server init. GC alone takes ~200 ms.

Overall boot time improvement: ~1 second

Before:
02-17 18:33:33 D/BaseBootTest: successive-boot :
28835.0,29638.0,30205.0,29793.0,29752.0,28228.0,30125.0,28983.0,28487.0,28865.0,
02-17 18:33:33 D/BaseBootTest: successive-boot_avg : 29291.1
02-17 18:33:33 D/BaseBootTest:
SystemServerTiming_StartPackageManagerService :
3150.0,3615.0,3515.0,3495.0,3814.0,3158.0,3746.0,3274.0,3222.0,3607.0,
02-17 18:33:33 D/BaseBootTest:
SystemServerTiming_StartPackageManagerService_avg : 3459.6
02-17 18:33:33 D/BaseBootTest: SystemServerTiming_StartServices :
8244.0,8863.0,9035.0,9832.0,8998.0,8096.0,8719.0,8209.0,8279.0,8754.0,
02-17 18:33:33 D/BaseBootTest: SystemServerTiming_StartServices_avg :
8702.9

After:
02-17 17:59:51 D/BaseBootTest: successive-boot :
27711.0,27607.0,28408.0,28968.0,28397.0,28063.0,27885.0,28483.0,27917.0,29317.0,
02-17 17:59:51 D/BaseBootTest: successive-boot_avg : 28275.6
02-17 17:59:51 D/BaseBootTest:
SystemServerTiming_StartPackageManagerService :
2467.0,2489.0,2369.0,2548.0,2647.0,2523.0,2497.0,2553.0,2482.0,2657.0,
02-17 17:59:51 D/BaseBootTest:
SystemServerTiming_StartPackageManagerService_avg : 2523.2
02-17 17:59:51 D/BaseBootTest: SystemServerTiming_StartServices :
7686.0,7538.0,7598.0,7869.0,7884.0,7950.0,7971.0,8370.0,7696.0,7885.0,
02-17 17:59:51 D/BaseBootTest: SystemServerTiming_StartServices_avg :
7844.7

Test: manual
Bug: 28750609
Change-Id: I3543ef577af1365394775318e40907584ddbe950

Merge "Add a flag to enable/disable the cache quota calc."

resolve merge conflicts of f17c1d488840 to master

OCL: http://ag/1932572

Test: refactoring CL, with unit tests

Change-Id: I5da27ee70afb8bf6e5fe9f6f3b545cd26d5c8d20

Slowly deprecate hidden getPackageSizeInfo() API.

It was never a public API, and we now have a support and CTS tested
public API over at StorageStatsManager that apps should be using.

Test: builds, boots
Bug: 34643504
Change-Id: Iad3f2c000dd9d7c1b1e70d0bf2d281826737322e

Fix build.

Test: make -j48 tests
Change-Id: I51287ddae3c7c689a44ca45edbae838e90b88e98

Merge "Add UI prompt for MMI code failure on 3gpp"

Add queryStatsForPackage() API.

This gives developers a way to collect package-level stats, even if
it means we have to use manual calculation for sharedUserId apps.

Also round size of storage devices to nice power-of-two values so we
do a better job of matching retail packaging.

Test: builds, boots
Bug: 35294241
Change-Id: I24946c443bb9dc4b0411a8149a0656702ac1fd24

Start locking down /data/media access.

The new sdcardfs filesystem requires that we have stricter access
controls around /data/media style locations.  Start by taking away
the "media_rw" GID from apps requesting the WRITE_MEDIA_STORAGE
permission.

Common use-cases like music playback appear to continue working fine,
as clients should only be attempting to use /data/media paths after
calling maybeTranslateEmulatedPathToInternal().

Test: builds, boots, music playback works
Bug: 35447080
Change-Id: Iba9f3ef41d3277c75497f675a1fe6d3406cf4542

Replace WearWifi/Cell services with WearConnectivityService am: 5555bccc70
am: abfd5bccb9

Change-Id: If051c765d3ea81cce2bedc2dfa02d64744088054

Merge "Revert "Ensure network connectivity on app start.""

Replace WearWifi/Cell services with WearConnectivityService
am: 5555bccc70

Change-Id: I5d8c97b8d5c394aeae99b7ddbe0dfdd5706a4293

Add "Use open Wi-Fi automatically" setting.

- Add constant in NetworkScoreManager for the meta-data key required
for NetworkRecommendationProviders to specify which package provides
this feature.
- Add Setting to specify which package is enabled for providing this
feature.

Bug: 34773276
Test: make
Change-Id: I3f8209c21b8b219c242650f97ba407b5985a5250

Revert "Ensure network connectivity on app start."

This reverts commit 27e693158398059d00ff20e60640cb105e255625.

Change-Id: I4c41ce7c1839d3b91931bc8a8f117f32bf6cd381

Merge "Revert "Notify NPMS when uid becomes active or idle.""

Merge "Revert "Set default wait-for-network timeout to 0.""

Merge "AAPT2: Rename strip phase to optimize"

Merge "Don't show the percentage twice when charging."

Revert "Notify NPMS when uid becomes active or idle."

This reverts commit ac0a61bad30e5a2ff45a85aaa6cf910e08911a2b.

Change-Id: I87d239cef51fc00567054644efceeca1ad2b463d

Revert "Set default wait-for-network timeout to 0."

This reverts commit d4f081cca4aa2b3b0fd1065839aedc1587dac94a.

Change-Id: Id55a08d00119c048c3d738b9f6f2cc5626df9bbf

Merge "AAPT2: Shared library support"

Remove abstract from AccessibilityService methods

Two methods on AccessibilityService were declared abstract, but
there doesn't seem to be a good reason for them to be abstract, as
it's possible to write an accessibility service that doesn't do
anything useful with the methods. Removing the abstract qualifier
to reduce boilerplate for service developers.

Bug: 31705365
Test: Trivial change, so I just verified that it builds.
Change-Id: I2af39848c8ab343d562361fdc1c3f8ca3ecb5f00

Merge "Updated some internal apps to use TYPE_APPLICATION_OVERLAY"

Merge "Ensure that we tint PiP actions to be consistent."

Merge "VRManager: Set initial sleep state to 'AWAKE'"

Merge "Split cache clearing into two phases."

Merge "Create am no-home-screen command."

Merge "Fix bug in auto launching with one entry."

Merge "Disable settings nav drawer entirely."

VideoView: option for audio focus, support for AudioAttributes

Add API for VideoView to select whether it uses audio focus during
  playback, and how.
Add support for AudioAttributes

Test: cts-tradefed run cts -m CtsWidgetTestCases -t android.widget.cts.VideoViewTest
Bug 30955183
Bug 30258418

Change-Id: I581d32c79c78b8197ded2319e0d5bfdc35b93c5e

Merge "underline is now only tracked by android"

Fixed missing auto-fill save callbacks on TextView.

It was accidentally broken during a previous commit where the CTS test
failure was ignored by a known issue being fixed in another change -
this change fixes both problems.

Bug: 33550221
Bug: 3380254
Test: CtsAutoFillServiceTestCases pass (for real now :-)
Change-Id: I5b2aa6060995ca255e51d85aa6ec93a886fa522f

Merge "Default android app icon should use <adaptive-icon> Test: install apk without application icon. manually check on default launcher."

Merge "Fix issues with assistant stack visibility."

Add missing sound to important channels.

Test: runtest systemui-notification
Change-Id: I7b091d312220e3830f2cd613b7ac08f75c5f0a8c

Merge "Unhide root MIME type."

Merge "FloatingToolbar: Menu items with icon + text."

AAPT2: Rename strip phase to optimize

- Allow resource deduping, version collapsing, and sparse resource
  encoding.

Test: manual
Change-Id: Ia4aa892ab5b06ba1d5ea4a6efb51b00bc3a980c4

Merge "AOD: readd AOD settings on debuggable builds"

Merge "Improved documentation and added @CallSuper on onProvideAutoFillStructure()."

Merge "Rename CameraService to CameraServiceProxy"

Merge "Import translations. DO NOT MERGE"

Merge "Grant notification listener access to overlay pkgs"

Merge "Import translations. DO NOT MERGE"

Merge "Revert "Don't copy NoCopySpans in SpannableStringInternal""

Merge "Import translations. DO NOT MERGE"

Add a flag to enable/disable the cache quota calc.

Bug: 34770259
Test: ServicesTests
Change-Id: I74155203c4802b1a3b89117859002bf40b1ca435

AAPT2: Shared library support

Test: make aapt2_tests
Change-Id: I98dddf1367e6c0ac425bb20be46e6ff05f4f2f45

Ensure that we tint PiP actions to be consistent.

Test: Launch PiP with actions, ensure they are white
Bug: 34281934
Change-Id: I3d301928af80a14ee6d161492c55a62b720d3cca

Merge "Introduce APIs for RTT calls (part 1)" am: 27ad05169e am: d83b06cd31
am: ba6c607728

Change-Id: I02e7291091d2452cb7cdb7351c8744606b340e76

Merge "Introduce APIs for RTT calls (part 1)" am: 27ad05169e
am: d83b06cd31

Change-Id: Id3608cf918d829ed785d7b75dffa6b6ce6990f34

Merge "Import translations. DO NOT MERGE"

Merge "Added auto-fill listeners for RadioGroup and CompoundButton."

Merge "Introduce APIs for RTT calls (part 1)"
am: 27ad05169e

Change-Id: I3fb3cb89b467e5df808967dcd478964a84e2e61f

Merge "Fixes dragging divider up in splitscreen puts the docked stack behind"

Merge "Introduce APIs for RTT calls (part 1)"

Updated some internal apps to use TYPE_APPLICATION_OVERLAY

TYPE_SYSTEM_ALERT is deprecated as of O.

Test: builds
Change-Id: Ic335ff0aa2f0356737c2e67debe8e71ea9eb2f41

Split cache clearing into two phases.

Use newly defined "V2_DEFY_QUOTA" flag to split cache clearing into
two phases: clearing data for apps above their quotas, and then
pushing deeper by clearing data for apps below their quotas.

Add placeholder comments for other data types that we're planning to
add shortly.  Route all clearing behavior through this new method,
which remains guarded behind a flag for now.

Test: builds, boots
Bug: 34692014
Change-Id: I678d7b4e2bf6c837dd8a9adbc36a53015907f75f

Merge "Make precentage calculation round up by 0.5%"

Merge "Fixed some notification leaks"

Merge "Add "Instant apps" filter to app listing"

Disable settings nav drawer entirely.

Change-Id: I3e9bdd3759a47fc09a654b6dad00b611138d5de6
Fix: 34806148
Test: make RunSettingsLibRoboTests

Merge "Limit Instant App notifications"

Fix bug in auto launching with one entry.

bug: 35422006
Test: Confirmed linked bug is fixed with manual test.

Change-Id: Ie5d237c2b85f274b26f913dba3dff585c2547a2b

Rename CameraService to CameraServiceProxy

Bug: 32976450
Test: on marlin, camera app works fine
Change-Id: I71890c6814f97e461b0161fed6c31c041c9bfe59

VRManager: Set initial sleep state to 'AWAKE'

Problem:
Home screen isn't allowed to enter VR mode as part of boot.

Root-cause:
VRManager is not indicating VR Mode is allowed until the system sleep
state gets synchronized.
Because the initial sleep state during boot is AWAKE and remains AWAKE if no
lock-screen is set, VrManager fails to mark VrMode as available due to a
mismatch between its initial internal sleep state and the systems sleep state.
VrManager's initial sleep state is SLEEP, where the system's initial sleep
state is AWAKE.

Solution: Set VrManager's initial sleep state to AWAKE to match the system.

Bug: 35469244
Test: - Validated VrHome comes up properly
      - Tested VrMode is not allowed when the lock-screen is shown
      - Tested VrMode is not allowed when the display is off
      - Tested VrMode is allowed when no lock-screen is set AND the display
        is on.

Change-Id: I42e9da07db21b3ed034c4173fe329cbb7796ee7e

Fix issues with assistant stack visibility.

- Ensure that we don't mark the docked or fullscreen stacks as
  invisible when the assistant stack is over them and translucent.

Bug: 35363819
Test: android.server.cts.ActivityManagerAssistantStackTests
Test: #testTranslucentAssistantActivityStackVisibility

Change-Id: I86698beec4963942dd3816e7d3f43c6614d892ac

Merge "AOD: Enable double tap in AOD"

Improved documentation and added @CallSuper on onProvideAutoFillStructure().

onProvideAutoFillStructure() sets mandatory auto-fill information like the
auto-fill id, so we must enforce subclass to call it.

Bug: 35395043
Test: m update-api
Change-Id: I525690b38639d970f516a906ddeb3a6bd6e01d8c

Import translations. DO NOT MERGE

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

FloatingToolbar: Menu items with icon + text.

- introduced one layout file to rule them all
- removed specialized layout files

Test: Manual tests checking expected appearance
Bug: 34777048
Change-Id: Id5b84d60b48e84b40554c983bf8c144efd42c504

underline is now only tracked by android

Change-Id: I189c57dd47ad413b8964d7e5805dfd6d4af11982

AOD: readd AOD settings on debuggable builds

Bug: 30876804
Test: On user build, verify that "Ambient Display" does not appear in SystemUI Tuner.
Change-Id: I75ba7ce11346f9cb8bf5683b17debe77edca20bf

Merge "Remove conflicting package information for junit.runner and junit.textui" am: f3f4775a35 am: 9b5522f8c4
am: da4597769b

Change-Id: Ibbcb838b9d41391b559a555111ebafce437a5679