diff options
27 files changed, 114 insertions, 36 deletions
| diff --git a/apex/jobscheduler/service/java/com/android/server/job/JobServiceContext.java b/apex/jobscheduler/service/java/com/android/server/job/JobServiceContext.java index e636f604ddb0..dfa91835bad3 100644 --- a/apex/jobscheduler/service/java/com/android/server/job/JobServiceContext.java +++ b/apex/jobscheduler/service/java/com/android/server/job/JobServiceContext.java @@ -1327,13 +1327,13 @@ public final class JobServiceContext implements ServiceConnection {      private void handleOpTimeoutLocked() {          switch (mVerb) {              case VERB_BINDING: -                onSlowAppResponseLocked(/* reschedule */ false, /* updateStopReasons */ true, +                // The system may have been too busy. Don't drop the job or trigger an ANR. +                onSlowAppResponseLocked(/* reschedule */ true, /* updateStopReasons */ true,                          /* texCounterMetricId */                          "job_scheduler.value_cntr_w_uid_slow_app_response_binding",                          /* debugReason */ "timed out while binding",                          /* anrMessage */ "Timed out while trying to bind", -                        CompatChanges.isChangeEnabled(ANR_PRE_UDC_APIS_ON_SLOW_RESPONSES, -                            mRunningJob.getUid())); +                        /* triggerAnr */ false);                  break;              case VERB_STARTING:                  // Client unresponsive - wedged or failed to respond in time. We don't really diff --git a/apex/jobscheduler/service/java/com/android/server/job/JobStore.java b/apex/jobscheduler/service/java/com/android/server/job/JobStore.java index 0a7bffc786cc..4b4e51290abd 100644 --- a/apex/jobscheduler/service/java/com/android/server/job/JobStore.java +++ b/apex/jobscheduler/service/java/com/android/server/job/JobStore.java @@ -1118,6 +1118,7 @@ public final class JobStore {              }              boolean needFileMigration = false;              long nowElapsed = sElapsedRealtimeClock.millis(); +            int numDuplicates = 0;              synchronized (mLock) {                  for (File file : files) {                      final AtomicFile aFile = createJobFile(file); @@ -1126,6 +1127,16 @@ public final class JobStore {                          if (jobs != null) {                              for (int i = 0; i < jobs.size(); i++) {                                  JobStatus js = jobs.get(i); +                                final JobStatus existingJob = this.jobSet.get( +                                        js.getUid(), js.getNamespace(), js.getJobId()); +                                if (existingJob != null) { +                                    numDuplicates++; +                                    // Jobs are meant to have unique uid-namespace-jobId +                                    // combinations, but we've somehow read multiple jobs with the +                                    // combination. Drop the latter one since keeping both will +                                    // result in other issues. +                                    continue; +                                }                                  js.prepareLocked();                                  js.enqueueTime = nowElapsed;                                  this.jobSet.add(js); @@ -1174,6 +1185,10 @@ public final class JobStore {                  migrateJobFilesAsync();              } +            if (numDuplicates > 0) { +                Slog.wtf(TAG, "Encountered " + numDuplicates + " duplicate persisted jobs"); +            } +              // Log the count immediately after loading from boot.              mCurrentJobSetSize = numJobs;              mScheduledJob30MinHighWaterMark = mCurrentJobSetSize; diff --git a/api/StubLibraries.bp b/api/StubLibraries.bp index 857077d60a5d..79efaced998a 100644 --- a/api/StubLibraries.bp +++ b/api/StubLibraries.bp @@ -436,6 +436,26 @@ java_api_library {      full_api_surface_stub: "android_module_lib_stubs_current_full.from-text",  } +// This module generates a stub jar that is a union of the test and module lib +// non-updatable api contributions. Modules should not depend on the stub jar +// generated from this module, as this module is strictly used for hiddenapi only. +java_api_library { +    name: "android-non-updatable.stubs.test_module_lib", +    api_surface: "module_lib", +    api_contributions: [ +        "api-stubs-docs-non-updatable.api.contribution", +        "system-api-stubs-docs-non-updatable.api.contribution", +        "test-api-stubs-docs-non-updatable.api.contribution", +        "module-lib-api-stubs-docs-non-updatable.api.contribution", +    ], +    defaults: ["android-non-updatable_from_text_defaults"], +    full_api_surface_stub: "android_test_module_lib_stubs_current.from-text", + +    // This module is only used for hiddenapi, and other modules should not +    // depend on this module. +    visibility: ["//visibility:private"], +} +  java_defaults {      name: "android_stubs_dists_default",      dist: { @@ -739,6 +759,30 @@ java_api_library {  }  java_api_library { +    name: "android_test_module_lib_stubs_current.from-text", +    api_surface: "module-lib", +    defaults: [ +        "android_stubs_current_contributions", +        "android_system_stubs_current_contributions", +        "android_test_stubs_current_contributions", +        "android_module_lib_stubs_current_contributions", +    ], +    libs: [ +        "android_module_lib_stubs_current_full.from-text", +        "stub-annotations", +    ], +    api_contributions: [ +        "test-api-stubs-docs-non-updatable.api.contribution", +    ], + +    // This module is only used to build android-non-updatable.stubs.test_module_lib +    // and other modules should not depend on this module. +    visibility: [ +        "//visibility:private", +    ], +} + +java_api_library {      name: "android_system_server_stubs_current.from-text",      api_surface: "system-server",      api_contributions: [ diff --git a/core/jni/AndroidRuntime.cpp b/core/jni/AndroidRuntime.cpp index d90d5f8c5eac..b2131f6fdfad 100644 --- a/core/jni/AndroidRuntime.cpp +++ b/core/jni/AndroidRuntime.cpp @@ -1314,7 +1314,16 @@ void AndroidRuntime::exit(int code)          ALOGI("VM exiting with result code %d.", code);          onExit(code);      } + +#ifdef __ANDROID_CLANG_COVERAGE__ +    // When compiled with coverage, a function is registered with atexit to call +    // `__llvm_profile_write_file` when the process exit. +    // For Clang code coverage to work, call exit instead of _exit to run hooks +    // registered with atexit. +    ::exit(code); +#else      ::_exit(code); +#endif  }  void AndroidRuntime::onVmCreated(JNIEnv* env) diff --git a/media/java/android/media/AudioManager.java b/media/java/android/media/AudioManager.java index ab2a22bd8857..0dea2c859fcd 100644 --- a/media/java/android/media/AudioManager.java +++ b/media/java/android/media/AudioManager.java @@ -1485,8 +1485,7 @@ public class AudioManager {       * Returns the volume group id associated to the given {@link AudioAttributes}.       *       * @param attributes The {@link AudioAttributes} to consider. -     * @return {@link android.media.audiopolicy.AudioVolumeGroup} id supporting the given -     * {@link AudioAttributes} if found, +     * @return audio volume group id supporting the given {@link AudioAttributes} if found,       * {@code android.media.audiopolicy.AudioVolumeGroup.DEFAULT_VOLUME_GROUP} otherwise.       */      public int getVolumeGroupIdForAttributes(@NonNull AudioAttributes attributes) { @@ -1601,7 +1600,7 @@ public class AudioManager {       * <p> Call first in prior {@link #getVolumeGroupIdForAttributes(AudioAttributes)} to retrieve       * the volume group id supporting the given {@link AudioAttributes}.       * -     * @param groupId of the {@link android.media.audiopolicy.AudioVolumeGroup} to consider. +     * @param groupId of the audio volume group to consider.       * @param direction The direction to adjust the volume. One of       *            {@link #ADJUST_LOWER}, {@link #ADJUST_RAISE}, or       *            {@link #ADJUST_SAME}. @@ -1645,8 +1644,8 @@ public class AudioManager {       * <p> Call first in prior {@link #getVolumeGroupIdForAttributes(AudioAttributes)} to retrieve       * the volume group id supporting the given {@link AudioAttributes}.       * -     * @param groupId of the {@link android.media.audiopolicy.AudioVolumeGroup} to consider. -     * @return The mute state for the given {@link android.media.audiopolicy.AudioVolumeGroup} id. +     * @param groupId of the audio volume group to consider. +     * @return The mute state for the given audio volume group id.       * @see #adjustVolumeGroupVolume(int, int, int)       */      public boolean isVolumeGroupMuted(int groupId) { diff --git a/media/java/android/media/projection/OWNERS b/media/java/android/media/projection/OWNERS index cc9be9c5126a..880ec8fdef88 100644 --- a/media/java/android/media/projection/OWNERS +++ b/media/java/android/media/projection/OWNERS @@ -4,3 +4,4 @@ michaelwr@google.com  santoscordon@google.com  chaviw@google.com  nmusgrave@google.com +dakinola@google.com diff --git a/services/core/java/com/android/server/PackageWatchdog.java b/services/core/java/com/android/server/PackageWatchdog.java index f4f5c951faaa..d256aead97e8 100644 --- a/services/core/java/com/android/server/PackageWatchdog.java +++ b/services/core/java/com/android/server/PackageWatchdog.java @@ -580,7 +580,6 @@ public class PackageWatchdog {                       PackageHealthObserverImpact.USER_IMPACT_LEVEL_10,                       PackageHealthObserverImpact.USER_IMPACT_LEVEL_30,                       PackageHealthObserverImpact.USER_IMPACT_LEVEL_50, -                     PackageHealthObserverImpact.USER_IMPACT_LEVEL_60,                       PackageHealthObserverImpact.USER_IMPACT_LEVEL_70,                       PackageHealthObserverImpact.USER_IMPACT_LEVEL_100})      public @interface PackageHealthObserverImpact { @@ -591,7 +590,6 @@ public class PackageWatchdog {          /* Actions having medium user impact, user of a device will likely notice. */          int USER_IMPACT_LEVEL_30 = 30;          int USER_IMPACT_LEVEL_50 = 50; -        int USER_IMPACT_LEVEL_60 = 60;          int USER_IMPACT_LEVEL_70 = 70;          /* Action has high user impact, a last resort, user of a device will be very frustrated. */          int USER_IMPACT_LEVEL_100 = 100; diff --git a/services/core/java/com/android/server/locksettings/LockSettingsStorage.java b/services/core/java/com/android/server/locksettings/LockSettingsStorage.java index 2fa637e030f1..18f84c03d04f 100644 --- a/services/core/java/com/android/server/locksettings/LockSettingsStorage.java +++ b/services/core/java/com/android/server/locksettings/LockSettingsStorage.java @@ -45,7 +45,7 @@ import com.android.internal.util.IndentingPrintWriter;  import com.android.internal.util.Preconditions;  import com.android.internal.widget.LockPatternUtils;  import com.android.server.LocalServices; -import com.android.server.PersistentDataBlockManagerInternal; +import com.android.server.pdb.PersistentDataBlockManagerInternal;  import java.io.ByteArrayInputStream;  import java.io.ByteArrayOutputStream; diff --git a/services/core/java/com/android/server/oemlock/OemLockService.java b/services/core/java/com/android/server/oemlock/OemLockService.java index 4c6110b99efd..d00268899a07 100644 --- a/services/core/java/com/android/server/oemlock/OemLockService.java +++ b/services/core/java/com/android/server/oemlock/OemLockService.java @@ -35,8 +35,8 @@ import android.service.oemlock.IOemLockService;  import android.util.Slog;  import com.android.server.LocalServices; -import com.android.server.PersistentDataBlockManagerInternal;  import com.android.server.SystemService; +import com.android.server.pdb.PersistentDataBlockManagerInternal;  import com.android.server.pm.UserManagerInternal;  import com.android.server.pm.UserManagerInternal.UserRestrictionsListener;  import com.android.server.pm.UserRestrictionsUtils; diff --git a/services/core/java/com/android/server/os/NativeTombstoneManager.java b/services/core/java/com/android/server/os/NativeTombstoneManager.java index 7b0eddc505f4..e5616d04554d 100644 --- a/services/core/java/com/android/server/os/NativeTombstoneManager.java +++ b/services/core/java/com/android/server/os/NativeTombstoneManager.java @@ -567,6 +567,10 @@ public final class NativeTombstoneManager {          @Override          public void onEvent(int event, @Nullable String path) { +            if (path == null) { +                Slog.w(TAG, "path is null at TombstoneWatcher.onEvent()"); +                return; +            }              mHandler.post(() -> {                  // Ignore .tmp files.                  if (path.endsWith(".tmp")) { diff --git a/services/core/java/com/android/server/pdb/OWNERS b/services/core/java/com/android/server/pdb/OWNERS new file mode 100644 index 000000000000..6f322eea0f2e --- /dev/null +++ b/services/core/java/com/android/server/pdb/OWNERS @@ -0,0 +1,2 @@ +victorhsieh@google.com +swillden@google.com diff --git a/services/core/java/com/android/server/PersistentDataBlockManagerInternal.java b/services/core/java/com/android/server/pdb/PersistentDataBlockManagerInternal.java index 21fa9f9a9401..66ad7169d6ec 100644 --- a/services/core/java/com/android/server/PersistentDataBlockManagerInternal.java +++ b/services/core/java/com/android/server/pdb/PersistentDataBlockManagerInternal.java @@ -14,7 +14,7 @@   * limitations under the License.   */ -package com.android.server; +package com.android.server.pdb;  /**   * Internal interface for storing and retrieving persistent data. diff --git a/services/core/java/com/android/server/PersistentDataBlockService.java b/services/core/java/com/android/server/pdb/PersistentDataBlockService.java index 6fd6afed49b9..870ca788eb3f 100644 --- a/services/core/java/com/android/server/PersistentDataBlockService.java +++ b/services/core/java/com/android/server/pdb/PersistentDataBlockService.java @@ -14,7 +14,7 @@   * limitations under the License.   */ -package com.android.server; +package com.android.server.pdb;  import static com.android.internal.util.Preconditions.checkArgument; @@ -37,6 +37,9 @@ import com.android.internal.R;  import com.android.internal.annotations.GuardedBy;  import com.android.internal.util.DumpUtils;  import com.android.server.pm.UserManagerInternal; +import com.android.server.LocalServices; +import com.android.server.SystemServerInitThreadPool; +import com.android.server.SystemService;  import libcore.io.IoUtils; diff --git a/services/core/java/com/android/server/rollback/RollbackPackageHealthObserver.java b/services/core/java/com/android/server/rollback/RollbackPackageHealthObserver.java index 0ca560398657..2007079ea5ca 100644 --- a/services/core/java/com/android/server/rollback/RollbackPackageHealthObserver.java +++ b/services/core/java/com/android/server/rollback/RollbackPackageHealthObserver.java @@ -121,7 +121,7 @@ final class RollbackPackageHealthObserver implements PackageHealthObserver {              impact = PackageHealthObserverImpact.USER_IMPACT_LEVEL_30;          } else if (getAvailableRollback(failedPackage) != null) {              // Rollback is available, we may get a callback into #execute -            impact = PackageHealthObserverImpact.USER_IMPACT_LEVEL_60; +            impact = PackageHealthObserverImpact.USER_IMPACT_LEVEL_30;          } else if (anyRollbackAvailable) {              // If any rollbacks are available, we will commit them              impact = PackageHealthObserverImpact.USER_IMPACT_LEVEL_70; diff --git a/services/core/java/com/android/server/testharness/TestHarnessModeService.java b/services/core/java/com/android/server/testharness/TestHarnessModeService.java index bfe34049e1e5..9a9b83602832 100644 --- a/services/core/java/com/android/server/testharness/TestHarnessModeService.java +++ b/services/core/java/com/android/server/testharness/TestHarnessModeService.java @@ -41,8 +41,8 @@ import com.android.internal.messages.nano.SystemMessageProto.SystemMessage;  import com.android.internal.notification.SystemNotificationChannels;  import com.android.internal.widget.LockPatternUtils;  import com.android.server.LocalServices; -import com.android.server.PersistentDataBlockManagerInternal;  import com.android.server.SystemService; +import com.android.server.pdb.PersistentDataBlockManagerInternal;  import com.android.server.pm.UserManagerInternal;  import java.io.ByteArrayInputStream; diff --git a/services/core/java/com/android/server/vibrator/OWNERS b/services/core/java/com/android/server/vibrator/OWNERS index 08f0a90485cc..da5a476e9eb8 100644 --- a/services/core/java/com/android/server/vibrator/OWNERS +++ b/services/core/java/com/android/server/vibrator/OWNERS @@ -1,3 +1,5 @@ +# Bug component: 345036 +khalilahmad@google.com  lsandrade@google.com  michaelwr@google.com -sbowden@google.com
\ No newline at end of file +roosa@google.com diff --git a/services/core/java/com/android/server/wm/ActivityRecord.java b/services/core/java/com/android/server/wm/ActivityRecord.java index f6fa51e3018a..18a6254ca115 100644 --- a/services/core/java/com/android/server/wm/ActivityRecord.java +++ b/services/core/java/com/android/server/wm/ActivityRecord.java @@ -8225,7 +8225,11 @@ final class ActivityRecord extends WindowToken implements WindowManagerService.A      private void clearSizeCompatModeAttributes() {          mInSizeCompatModeForBounds = false; +        final float lastSizeCompatScale = mSizeCompatScale;          mSizeCompatScale = 1f; +        if (mSizeCompatScale != lastSizeCompatScale) { +            forAllWindows(WindowState::updateGlobalScale, false /* traverseTopToBottom */); +        }          mSizeCompatBounds = null;          mCompatDisplayInsets = null;          mLetterboxUiController.clearInheritedCompatDisplayInsets(); @@ -8233,11 +8237,7 @@ final class ActivityRecord extends WindowToken implements WindowManagerService.A      @VisibleForTesting      void clearSizeCompatMode() { -        final float lastSizeCompatScale = mSizeCompatScale;          clearSizeCompatModeAttributes(); -        if (mSizeCompatScale != lastSizeCompatScale) { -            forAllWindows(WindowState::updateGlobalScale, false /* traverseTopToBottom */); -        }          // Clear config override in #updateCompatDisplayInsets().          final int activityType = getActivityType();          final Configuration overrideConfig = getRequestedOverrideConfiguration(); diff --git a/services/core/jni/Android.bp b/services/core/jni/Android.bp index 6f2455e073a7..62864485c0e2 100644 --- a/services/core/jni/Android.bp +++ b/services/core/jni/Android.bp @@ -68,7 +68,7 @@ cc_library_static {          "com_android_server_UsbHostManager.cpp",          "com_android_server_vibrator_VibratorController.cpp",          "com_android_server_vibrator_VibratorManagerService.cpp", -        "com_android_server_PersistentDataBlockService.cpp", +        "com_android_server_pdb_PersistentDataBlockService.cpp",          "com_android_server_am_LowMemDetector.cpp",          "com_android_server_pm_PackageManagerShellCommandDataLoader.cpp",          "com_android_server_sensor_SensorService.cpp", diff --git a/services/core/jni/com_android_server_PersistentDataBlockService.cpp b/services/core/jni/com_android_server_pdb_PersistentDataBlockService.cpp index 97e69fb4bb40..fc5a11360c8c 100644 --- a/services/core/jni/com_android_server_PersistentDataBlockService.cpp +++ b/services/core/jni/com_android_server_pdb_PersistentDataBlockService.cpp @@ -76,7 +76,7 @@ namespace android {          return ret;      } -    static jlong com_android_server_PersistentDataBlockService_getBlockDeviceSize(JNIEnv *env, jclass, jstring jpath) +    static jlong com_android_server_pdb_PersistentDataBlockService_getBlockDeviceSize(JNIEnv *env, jclass, jstring jpath)      {          ScopedUtfChars path(env, jpath);          int fd = open(path.c_str(), O_RDONLY); @@ -91,7 +91,7 @@ namespace android {          return size;      } -    static int com_android_server_PersistentDataBlockService_wipe(JNIEnv *env, jclass, jstring jpath) { +    static int com_android_server_pdb_PersistentDataBlockService_wipe(JNIEnv *env, jclass, jstring jpath) {          ScopedUtfChars path(env, jpath);          int fd = open(path.c_str(), O_WRONLY); @@ -107,13 +107,13 @@ namespace android {      static const JNINativeMethod sMethods[] = {           /* name, signature, funcPtr */ -        {"nativeGetBlockDeviceSize", "(Ljava/lang/String;)J", (void*)com_android_server_PersistentDataBlockService_getBlockDeviceSize}, -        {"nativeWipe", "(Ljava/lang/String;)I", (void*)com_android_server_PersistentDataBlockService_wipe}, +        {"nativeGetBlockDeviceSize", "(Ljava/lang/String;)J", (void*)com_android_server_pdb_PersistentDataBlockService_getBlockDeviceSize}, +        {"nativeWipe", "(Ljava/lang/String;)I", (void*)com_android_server_pdb_PersistentDataBlockService_wipe},      }; -    int register_android_server_PersistentDataBlockService(JNIEnv* env) +    int register_android_server_pdb_PersistentDataBlockService(JNIEnv* env)      { -        return jniRegisterNativeMethods(env, "com/android/server/PersistentDataBlockService", +        return jniRegisterNativeMethods(env, "com/android/server/pdb/PersistentDataBlockService",                                          sMethods, NELEM(sMethods));      } diff --git a/services/core/jni/onload.cpp b/services/core/jni/onload.cpp index 290ad8de9547..4078606afb0f 100644 --- a/services/core/jni/onload.cpp +++ b/services/core/jni/onload.cpp @@ -46,7 +46,7 @@ int register_android_server_connectivity_Vpn(JNIEnv* env);  int register_android_server_devicepolicy_CryptoTestHelper(JNIEnv*);  int register_android_server_tv_TvUinputBridge(JNIEnv* env);  int register_android_server_tv_TvInputHal(JNIEnv* env); -int register_android_server_PersistentDataBlockService(JNIEnv* env); +int register_android_server_pdb_PersistentDataBlockService(JNIEnv* env);  int register_android_server_Watchdog(JNIEnv* env);  int register_android_server_HardwarePropertiesManagerService(JNIEnv* env);  int register_android_server_SyntheticPasswordManager(JNIEnv* env); @@ -104,7 +104,7 @@ extern "C" jint JNI_OnLoad(JavaVM* vm, void* /* reserved */)      register_android_server_BatteryStatsService(env);      register_android_server_tv_TvUinputBridge(env);      register_android_server_tv_TvInputHal(env); -    register_android_server_PersistentDataBlockService(env); +    register_android_server_pdb_PersistentDataBlockService(env);      register_android_server_HardwarePropertiesManagerService(env);      register_android_server_storage_AppFuse(env);      register_android_server_SyntheticPasswordManager(env); diff --git a/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java b/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java index ab218a671823..c1cc73bb9d10 100644 --- a/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java +++ b/services/devicepolicy/java/com/android/server/devicepolicy/DevicePolicyManagerService.java @@ -485,13 +485,13 @@ import com.android.server.AlarmManagerInternal;  import com.android.server.LocalManagerRegistry;  import com.android.server.LocalServices;  import com.android.server.LockGuard; -import com.android.server.PersistentDataBlockManagerInternal;  import com.android.server.SystemServerInitThreadPool;  import com.android.server.SystemService;  import com.android.server.SystemServiceManager;  import com.android.server.devicepolicy.ActiveAdmin.TrustAgentInfo;  import com.android.server.inputmethod.InputMethodManagerInternal;  import com.android.server.net.NetworkPolicyManagerInternal; +import com.android.server.pdb.PersistentDataBlockManagerInternal;  import com.android.server.pm.DefaultCrossProfileIntentFilter;  import com.android.server.pm.DefaultCrossProfileIntentFiltersUtils;  import com.android.server.pm.PackageManagerLocal; diff --git a/services/java/com/android/server/SystemServer.java b/services/java/com/android/server/SystemServer.java index 1792964fe3af..3610542c7caf 100644 --- a/services/java/com/android/server/SystemServer.java +++ b/services/java/com/android/server/SystemServer.java @@ -158,6 +158,7 @@ import com.android.server.os.BugreportManagerService;  import com.android.server.os.DeviceIdentifiersPolicyService;  import com.android.server.os.NativeTombstoneManagerService;  import com.android.server.os.SchedulingPolicyService; +import com.android.server.pdb.PersistentDataBlockService;  import com.android.server.people.PeopleService;  import com.android.server.permission.access.AccessCheckingService;  import com.android.server.pm.ApexManager; diff --git a/services/tests/mockingservicestests/src/com/android/server/rollback/RollbackPackageHealthObserverTest.java b/services/tests/mockingservicestests/src/com/android/server/rollback/RollbackPackageHealthObserverTest.java index 35d4ffdd94d0..a14073006c31 100644 --- a/services/tests/mockingservicestests/src/com/android/server/rollback/RollbackPackageHealthObserverTest.java +++ b/services/tests/mockingservicestests/src/com/android/server/rollback/RollbackPackageHealthObserverTest.java @@ -145,7 +145,7 @@ public class RollbackPackageHealthObserverTest {                  observer.onHealthCheckFailed(null,                          PackageWatchdog.FAILURE_REASON_NATIVE_CRASH, 1));          // non-native crash for the package -        assertEquals(PackageWatchdog.PackageHealthObserverImpact.USER_IMPACT_LEVEL_60, +        assertEquals(PackageWatchdog.PackageHealthObserverImpact.USER_IMPACT_LEVEL_30,                  observer.onHealthCheckFailed(testFailedPackage,                          PackageWatchdog.FAILURE_REASON_APP_CRASH, 1));          // non-native crash for a different package diff --git a/services/tests/servicestests/src/com/android/server/devicepolicy/DevicePolicyManagerServiceTestable.java b/services/tests/servicestests/src/com/android/server/devicepolicy/DevicePolicyManagerServiceTestable.java index 9b32a809d2b5..de3cfbf859ff 100644 --- a/services/tests/servicestests/src/com/android/server/devicepolicy/DevicePolicyManagerServiceTestable.java +++ b/services/tests/servicestests/src/com/android/server/devicepolicy/DevicePolicyManagerServiceTestable.java @@ -54,8 +54,8 @@ import com.android.internal.widget.LockPatternUtils;  import com.android.internal.widget.LockSettingsInternal;  import com.android.server.AlarmManagerInternal;  import com.android.server.LocalServices; -import com.android.server.PersistentDataBlockManagerInternal;  import com.android.server.net.NetworkPolicyManagerInternal; +import com.android.server.pdb.PersistentDataBlockManagerInternal;  import com.android.server.pm.PackageManagerLocal;  import com.android.server.pm.UserManagerInternal;  import com.android.server.wm.ActivityTaskManagerInternal; diff --git a/services/tests/servicestests/src/com/android/server/devicepolicy/MockSystemServices.java b/services/tests/servicestests/src/com/android/server/devicepolicy/MockSystemServices.java index 16fdfb16444a..76aa40c04e3d 100644 --- a/services/tests/servicestests/src/com/android/server/devicepolicy/MockSystemServices.java +++ b/services/tests/servicestests/src/com/android/server/devicepolicy/MockSystemServices.java @@ -77,7 +77,7 @@ import com.android.internal.util.test.FakeSettingsProvider;  import com.android.internal.widget.LockPatternUtils;  import com.android.internal.widget.LockSettingsInternal;  import com.android.server.AlarmManagerInternal; -import com.android.server.PersistentDataBlockManagerInternal; +import com.android.server.pdb.PersistentDataBlockManagerInternal;  import com.android.server.net.NetworkPolicyManagerInternal;  import com.android.server.pm.PackageManagerLocal;  import com.android.server.pm.UserManagerInternal; diff --git a/services/tests/servicestests/src/com/android/server/locksettings/LockSettingsStorageTestable.java b/services/tests/servicestests/src/com/android/server/locksettings/LockSettingsStorageTestable.java index 36dc6c5f9f95..cfb3b3497745 100644 --- a/services/tests/servicestests/src/com/android/server/locksettings/LockSettingsStorageTestable.java +++ b/services/tests/servicestests/src/com/android/server/locksettings/LockSettingsStorageTestable.java @@ -22,7 +22,7 @@ import static org.mockito.Mockito.mock;  import android.content.Context; -import com.android.server.PersistentDataBlockManagerInternal; +import com.android.server.pdb.PersistentDataBlockManagerInternal;  import org.mockito.invocation.InvocationOnMock;  import org.mockito.stubbing.Answer; diff --git a/services/tests/servicestests/src/com/android/server/locksettings/LockSettingsStorageTests.java b/services/tests/servicestests/src/com/android/server/locksettings/LockSettingsStorageTests.java index 10ed882f343f..39e2792a3a34 100644 --- a/services/tests/servicestests/src/com/android/server/locksettings/LockSettingsStorageTests.java +++ b/services/tests/servicestests/src/com/android/server/locksettings/LockSettingsStorageTests.java @@ -53,8 +53,8 @@ import androidx.test.runner.AndroidJUnit4;  import com.android.internal.util.test.FakeSettingsProvider;  import com.android.internal.util.test.FakeSettingsProviderRule; -import com.android.server.PersistentDataBlockManagerInternal;  import com.android.server.locksettings.LockSettingsStorage.PersistentData; +import com.android.server.pdb.PersistentDataBlockManagerInternal;  import org.junit.After;  import org.junit.Before; |