From 30a66e364f85a806456fb11542bce9fefccb9d33 Mon Sep 17 00:00:00 2001 From: Dave Mankoff Date: Wed, 12 Feb 2020 13:59:12 -0500 Subject: [PATCH] Remove final call to Dep.get(FalsingManager.class) CarStatusBar had one last reference in its dump method. FalsingManager, however, directly registers itself with the DumpController, so this call is redundant. Bug: 136279712 Test: atest SystemUITests Change-Id: I3731e7ecac2f147e5d3967d4219f22d17e810843 --- .../src/com/android/systemui/statusbar/car/CarStatusBar.java | 1 - packages/SystemUI/src/com/android/systemui/Dependency.java | 3 --- 2 files changed, 4 deletions(-) diff --git a/packages/CarSystemUI/src/com/android/systemui/statusbar/car/CarStatusBar.java b/packages/CarSystemUI/src/com/android/systemui/statusbar/car/CarStatusBar.java index 210dd321933a6..5af3ad59c0af4 100644 --- a/packages/CarSystemUI/src/com/android/systemui/statusbar/car/CarStatusBar.java +++ b/packages/CarSystemUI/src/com/android/systemui/statusbar/car/CarStatusBar.java @@ -916,7 +916,6 @@ public class CarStatusBar extends StatusBar implements CarBatteryController.Batt Dependency.get(KeyguardUpdateMonitor.class).dump(fd, pw, args); } - Dependency.get(FalsingManager.class).dump(pw); FalsingLog.dump(pw); pw.println("SharedPreferences:"); diff --git a/packages/SystemUI/src/com/android/systemui/Dependency.java b/packages/SystemUI/src/com/android/systemui/Dependency.java index 27fe37ef4e69d..d317b7ebd8e7a 100644 --- a/packages/SystemUI/src/com/android/systemui/Dependency.java +++ b/packages/SystemUI/src/com/android/systemui/Dependency.java @@ -48,7 +48,6 @@ import com.android.systemui.keyguard.WakefulnessLifecycle; import com.android.systemui.model.SysUiState; import com.android.systemui.plugins.ActivityStarter; import com.android.systemui.plugins.DarkIconDispatcher; -import com.android.systemui.plugins.FalsingManager; import com.android.systemui.plugins.PluginDependencyProvider; import com.android.systemui.plugins.VolumeDialogController; import com.android.systemui.plugins.statusbar.StatusBarStateController; @@ -315,7 +314,6 @@ public class Dependency { @Inject Lazy mDockManager; @Inject Lazy mChannelEditorDialogController; @Inject Lazy mINotificationManager; - @Inject Lazy mFalsingManager; @Inject Lazy mSysUiStateFlagsContainer; @Inject Lazy mAlarmManager; @Inject Lazy mKeyguardSecurityModel; @@ -509,7 +507,6 @@ public class Dependency { mProviders.put(DockManager.class, mDockManager::get); mProviders.put(ChannelEditorDialogController.class, mChannelEditorDialogController::get); mProviders.put(INotificationManager.class, mINotificationManager::get); - mProviders.put(FalsingManager.class, mFalsingManager::get); mProviders.put(SysUiState.class, mSysUiStateFlagsContainer::get); mProviders.put(AlarmManager.class, mAlarmManager::get); mProviders.put(KeyguardSecurityModel.class, mKeyguardSecurityModel::get);