Merge "Clean up proguard.flags." into udc-dev am: 01b56bb057 (137ab5ef) · Commits · e / os / android_frameworks_base

packages/SystemUI/docs/dagger.md

+6 −13

Original line number	Diff line number	Diff line
		@@ -108,20 +108,13 @@ You can then include your module in one of three places:

		### Using injection with Fragments

		Fragments are created as part of the FragmentManager, so they need to be
		setup so the manager knows how to create them. To do that, add a method
		to com.android.systemui.fragments.FragmentService$FragmentCreator that
		returns your fragment class. That is all that is required, once the method
		exists, FragmentService will automatically pick it up and use injection
		whenever your fragment needs to be created.
		Fragments are created as part of the FragmentManager, so injectable Fragments need to be registered
		so the manager knows how to create them. This is done via
		[FragmentService#addFragmentInstantiationProvider](../src/com/android/systemui/fragments/FragmentService.java).
		Pass it the class of your fragment and a `Provider` for your fragment at some time before your
		Fragment is accessed.

		```java
		public interface FragmentCreator {
		NavigationBarFragment createNavigationBar();
		}
		```

		If you need to create your fragment (i.e. for the add or replace transaction),
		When you need to create your fragment (i.e. for the add or replace transaction),
		then the FragmentHostManager can do this for you.

		```java

packages/SystemUI/proguard.flags

+5 −148

Original line number	Diff line number	Diff line
		# Preserve line number information for debugging stack traces.
		-keepattributes SourceFile,LineNumberTable
		-include proguard_common.flags

		# Preserve relationship information that can impact simple class naming.
		-keepattributes EnclosingMethod,InnerClasses

		-keep class com.android.systemui.recents.OverviewProxyRecentsImpl
		-keep class com.android.systemui.statusbar.car.CarStatusBar
		-keep class com.android.systemui.statusbar.phone.CentralSurfaces
		-keep class com.android.systemui.statusbar.tv.TvStatusBar
		-keep class ** extends com.android.systemui.SystemUIInitializer {
		-keep class com.android.systemui.SystemUIInitializerImpl {
		*;
		}
		-keep class * extends com.android.systemui.CoreStartable
		-keep class * implements com.android.systemui.CoreStartable$Injector

		# Needed for builds to properly initialize KeyFrames from xml scene
		-keepclassmembers class * extends androidx.constraintlayout.motion.widget.Key {
		public <init>();
		}

		# Needed to ensure callback field references are kept in their respective
		# owning classes when the downstream callback registrars only store weak refs.
		# TODO(b/264686688): Handle these cases with more targeted annotations.
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		private com.android.keyguard.KeyguardUpdateMonitorCallback *;
		private com.android.systemui.privacy.PrivacyConfig$Callback *;
		private com.android.systemui.privacy.PrivacyItemController$Callback *;
		private com.android.systemui.settings.UserTracker$Callback *;
		private com.android.systemui.statusbar.phone.StatusBarWindowCallback *;
		private com.android.systemui.util.service.Observer$Callback *;
		private com.android.systemui.util.service.ObservableServiceConnection$Callback *;
		}
		# Note that these rules are temporary companions to the above rules, required
		# for cases like Kotlin where fields with anonymous types use the anonymous type
		# rather than the supertype.
		-if class * extends com.android.keyguard.KeyguardUpdateMonitorCallback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.privacy.PrivacyConfig$Callback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.privacy.PrivacyItemController$Callback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.settings.UserTracker$Callback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.statusbar.phone.StatusBarWindowCallback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.util.service.Observer$Callback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.util.service.ObservableServiceConnection$Callback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}

		-keepclasseswithmembers class * {
		public <init>(android.content.Context, android.util.AttributeSet);
		}

		-keep class ** extends androidx.preference.PreferenceFragment
		-keep class com.android.systemui.tuner.*

		# The plugins subpackage acts as a shared library that might be referenced in
		# dynamically-loaded plugin APKs.
		-keep class com.android.systemui.plugins.** {
		*;
		}
		-keep class com.android.systemui.fragments.FragmentService$FragmentCreator {
		-keep class com.android.systemui.tv.TvSystemUIInitializer {
		*;
		}
		-keep class androidx.core.app.CoreComponentFactory

		-keep public class * extends com.android.systemui.CoreStartable {
		public <init>(android.content.Context);
		}

		# Keep the wm shell lib
		-keep class com.android.wm.shell.*
		# Keep the protolog group methods that are called by the generated code
		-keepclassmembers class com.android.wm.shell.protolog.ShellProtoLogGroup {
		*;
		}

		-keep,allowoptimization,allowaccessmodification class com.android.systemui.dagger.GlobalRootComponent { !synthetic *; }
		-keep,allowoptimization,allowaccessmodification class com.android.systemui.dagger.GlobalRootComponent$SysUIComponentImpl { !synthetic *; }
		-keep,allowoptimization,allowaccessmodification class com.android.systemui.dagger.Dagger** { !synthetic *; }
		-keep,allowoptimization,allowaccessmodification class com.android.systemui.tv.Dagger** { !synthetic *; }

		# Prevent optimization or access modification of any referenced code that may
		# conflict with code in the bootclasspath.
		# TODO(b/222468116): Resolve such collisions in the build system.
		-keepnames class android..nano. { *; }
		-keepnames class com.android..nano. { *; }
		-keepnames class com.android.internal.protolog.** { *; }
		-keepnames class android.hardware.common.** { *; }

		# Allows proguard to make private and protected methods and fields public as
		# part of optimization. This lets proguard inline trivial getter/setter methods.
		-allowaccessmodification

		# Removes runtime checks added through Kotlin to JVM code genereration to
		# avoid linear growth as more Kotlin code is converted / added to the codebase.
		# These checks are generally applied to Java platform types (values returned
		# from Java code that don't have nullness annotations), but we remove them to
		# avoid code size increases.
		#
		# See also https://kotlinlang.org/docs/reference/java-interop.html
		#
		# TODO(b/199941987): Consider standardizing these rules in a central place as
		# Kotlin gains adoption with other platform targets.
		-assumenosideeffects class kotlin.jvm.internal.Intrinsics {
		# Remove check for method parameters being null
		static void checkParameterIsNotNull(java.lang.Object, java.lang.String);

		# When a Java platform type is returned and passed to Kotlin NonNull method,
		# remove the null check
		static void checkExpressionValueIsNotNull(java.lang.Object, java.lang.String);
		static void checkNotNullExpressionValue(java.lang.Object, java.lang.String);

		# Remove check that final value returned from method is null, if passing
		# back Java platform type.
		static void checkReturnedValueIsNotNull(java.lang.Object, java.lang.String, java.lang.String);
		static void checkReturnedValueIsNotNull(java.lang.Object, java.lang.String);

		# Null check for accessing a field from a parent class written in Java.
		static void checkFieldIsNotNull(java.lang.Object, java.lang.String, java.lang.String);
		static void checkFieldIsNotNull(java.lang.Object, java.lang.String);

		# Removes code generated from !! operator which converts Nullable type to
		# NonNull type. These would throw an NPE immediate after on access.
		static void checkNotNull(java.lang.Object, java.lang.String);
		static void checkNotNullParameter(java.lang.Object, java.lang.String);

		# Removes lateinit var check being used before being set. Check is applied
		# on every field access without this.
		static void throwUninitializedPropertyAccessException(java.lang.String);
		}
		# Strip verbose logs.
		-assumenosideeffects class android.util.Log {
		static *** v(...);
		static *** isLoggable(...);
		}
		-assumenosideeffects class android.util.Slog {
		static *** v(...);
		}
		-maximumremovedandroidloglevel 2
		-keep,allowoptimization,allowaccessmodification class com.android.systemui.dagger.DaggerReferenceGlobalRootComponent** { !synthetic *; }
		-keep,allowoptimization,allowaccessmodification class com.android.systemui.tv.DaggerTvGlobalRootComponent** { !synthetic *; }
		No newline at end of file

packages/SystemUI/proguard_common.flags

0 → 100644

+141 −0

Original line number	Diff line number	Diff line
		# Preserve line number information for debugging stack traces.
		-keepattributes SourceFile,LineNumberTable

		-keep class com.android.systemui.VendorServices

		# the `#inject` methods are accessed via reflection to work on ContentProviders
		-keepclassmembers class * extends com.android.systemui.dagger.SysUIComponent { void inject(***); }

		# Needed for builds to properly initialize KeyFrames from xml scene
		-keepclassmembers class * extends androidx.constraintlayout.motion.widget.Key {
		public <init>();
		}

		# Needed to ensure callback field references are kept in their respective
		# owning classes when the downstream callback registrars only store weak refs.
		# TODO(b/264686688): Handle these cases with more targeted annotations.
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		private com.android.keyguard.KeyguardUpdateMonitorCallback *;
		private com.android.systemui.privacy.PrivacyConfig$Callback *;
		private com.android.systemui.privacy.PrivacyItemController$Callback *;
		private com.android.systemui.settings.UserTracker$Callback *;
		private com.android.systemui.statusbar.phone.StatusBarWindowCallback *;
		private com.android.systemui.util.service.Observer$Callback *;
		private com.android.systemui.util.service.ObservableServiceConnection$Callback *;
		}
		# Note that these rules are temporary companions to the above rules, required
		# for cases like Kotlin where fields with anonymous types use the anonymous type
		# rather than the supertype.
		-if class * extends com.android.keyguard.KeyguardUpdateMonitorCallback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.privacy.PrivacyConfig$Callback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.privacy.PrivacyItemController$Callback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.settings.UserTracker$Callback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.statusbar.phone.StatusBarWindowCallback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.util.service.Observer$Callback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}
		-if class * extends com.android.systemui.util.service.ObservableServiceConnection$Callback
		-keepclassmembers,allowaccessmodification class com.android.systemui., com.android.keyguard. {
		<1> *;
		}

		-keepclasseswithmembers class * {
		public <init>(android.content.Context, android.util.AttributeSet);
		}

		-keep class ** extends androidx.preference.PreferenceFragment
		-keep class com.android.systemui.tuner.*

		# The plugins subpackage acts as a shared library that might be referenced in
		# dynamically-loaded plugin APKs.
		-keep class com.android.systemui.plugins.** {
		*;
		}
		-keep class com.android.systemui.fragments.FragmentService$FragmentCreator {
		*;
		}
		-keep class androidx.core.app.CoreComponentFactory

		# Keep the wm shell lib
		-keep class com.android.wm.shell.*
		# Keep the protolog group methods that are called by the generated code
		-keepclassmembers class com.android.wm.shell.protolog.ShellProtoLogGroup {
		*;
		}

		# Prevent optimization or access modification of any referenced code that may
		# conflict with code in the bootclasspath.
		# TODO(b/222468116): Resolve such collisions in the build system.
		-keepnames class android..nano. { *; }
		-keepnames class com.android..nano. { *; }
		-keepnames class com.android.internal.protolog.** { *; }
		-keepnames class android.hardware.common.** { *; }

		# Allows proguard to make private and protected methods and fields public as
		# part of optimization. This lets proguard inline trivial getter/setter methods.
		-allowaccessmodification

		# Removes runtime checks added through Kotlin to JVM code genereration to
		# avoid linear growth as more Kotlin code is converted / added to the codebase.
		# These checks are generally applied to Java platform types (values returned
		# from Java code that don't have nullness annotations), but we remove them to
		# avoid code size increases.
		#
		# See also https://kotlinlang.org/docs/reference/java-interop.html
		#
		# TODO(b/199941987): Consider standardizing these rules in a central place as
		# Kotlin gains adoption with other platform targets.
		-assumenosideeffects class kotlin.jvm.internal.Intrinsics {
		# Remove check for method parameters being null
		static void checkParameterIsNotNull(java.lang.Object, java.lang.String);

		# When a Java platform type is returned and passed to Kotlin NonNull method,
		# remove the null check
		static void checkExpressionValueIsNotNull(java.lang.Object, java.lang.String);
		static void checkNotNullExpressionValue(java.lang.Object, java.lang.String);

		# Remove check that final value returned from method is null, if passing
		# back Java platform type.
		static void checkReturnedValueIsNotNull(java.lang.Object, java.lang.String, java.lang.String);
		static void checkReturnedValueIsNotNull(java.lang.Object, java.lang.String);

		# Null check for accessing a field from a parent class written in Java.
		static void checkFieldIsNotNull(java.lang.Object, java.lang.String, java.lang.String);
		static void checkFieldIsNotNull(java.lang.Object, java.lang.String);

		# Removes code generated from !! operator which converts Nullable type to
		# NonNull type. These would throw an NPE immediate after on access.
		static void checkNotNull(java.lang.Object, java.lang.String);
		static void checkNotNullParameter(java.lang.Object, java.lang.String);

		# Removes lateinit var check being used before being set. Check is applied
		# on every field access without this.
		static void throwUninitializedPropertyAccessException(java.lang.String);
		}


		# Strip verbose logs.
		-assumenosideeffects class android.util.Log {
		static *** v(...);
		static *** isLoggable(...);
		}
		-assumenosideeffects class android.util.Slog {
		static *** v(...);
		}
		-maximumremovedandroidloglevel 2

packages/SystemUI/src/com/android/systemui/dagger/SystemUIModule.java

+3 −3

Original line number	Diff line number	Diff line
		@@ -48,7 +48,6 @@ import com.android.systemui.dreams.dagger.DreamModule;
		import com.android.systemui.dump.DumpManager;
		import com.android.systemui.flags.FeatureFlags;
		import com.android.systemui.flags.FlagsModule;
		import com.android.systemui.fragments.FragmentService;
		import com.android.systemui.keyboard.KeyboardModule;
		import com.android.systemui.keyguard.data.BouncerViewModule;
		import com.android.systemui.log.dagger.LogModule;
		@@ -64,6 +63,7 @@ import com.android.systemui.privacy.PrivacyModule;
		import com.android.systemui.qrcodescanner.dagger.QRCodeScannerModule;
		import com.android.systemui.qs.FgsManagerController;
		import com.android.systemui.qs.FgsManagerControllerImpl;
		import com.android.systemui.qs.QSFragmentStartableModule;
		import com.android.systemui.qs.footer.dagger.FooterActionsModule;
		import com.android.systemui.recents.Recents;
		import com.android.systemui.screenrecord.ScreenRecordModule;
		@@ -169,6 +169,7 @@ import javax.inject.Named;
		PolicyModule.class,
		PrivacyModule.class,
		QRCodeScannerModule.class,
		QSFragmentStartableModule.class,
		ScreenshotModule.class,
		SensorModule.class,
		SecurityRepositoryModule.class,
		@@ -198,8 +199,7 @@ import javax.inject.Named;
		DozeComponent.class,
		ExpandableNotificationRowComponent.class,
		KeyguardBouncerComponent.class,
		NotificationShelfComponent.class,
		FragmentService.FragmentCreator.class
		NotificationShelfComponent.class
		})
		public abstract class SystemUIModule {

packages/SystemUI/src/com/android/systemui/fragments/FragmentHostManager.java

+14 −21

Original line number	Diff line number	Diff line
		@@ -39,15 +39,16 @@ import com.android.settingslib.applications.InterestingConfigChanges;
		import com.android.systemui.plugins.Plugin;
		import com.android.systemui.util.leak.LeakDetector;

		import dagger.assisted.Assisted;
		import dagger.assisted.AssistedFactory;
		import dagger.assisted.AssistedInject;

		import java.io.FileDescriptor;
		import java.io.PrintWriter;
		import java.lang.reflect.InvocationTargetException;
		import java.util.ArrayList;
		import java.util.HashMap;

		import dagger.assisted.Assisted;
		import dagger.assisted.AssistedFactory;
		import dagger.assisted.AssistedInject;
		import javax.inject.Provider;

		public class FragmentHostManager {

		@@ -322,25 +323,17 @@ public class FragmentHostManager {
		return instantiateWithInjections(context, className, arguments);
		}

		private Fragment instantiateWithInjections(
		Context context, String className, Bundle args) {
		FragmentService.FragmentInstantiationInfo fragmentInstantiationInfo =
		private Fragment instantiateWithInjections(Context context, String className, Bundle args) {
		Provider<? extends Fragment> fragmentProvider =
		mManager.getInjectionMap().get(className);
		if (fragmentInstantiationInfo != null) {
		try {
		Fragment f = (Fragment) fragmentInstantiationInfo
		.mMethod
		.invoke(fragmentInstantiationInfo.mDaggerComponent);
		if (fragmentProvider != null) {
		Fragment f = fragmentProvider.get();
		// Setup the args, taken from Fragment#instantiate.
		if (args != null) {
		args.setClassLoader(f.getClass().getClassLoader());
		f.setArguments(args);
		}
		return f;
		} catch (IllegalAccessException \| InvocationTargetException e) {
		throw new Fragment.InstantiationException("Unable to instantiate " + className,
		e);
		}
		}
		return Fragment.instantiate(context, className, args);
		}