Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 01b56bb0 authored by Dave Mankoff's avatar Dave Mankoff Committed by Android (Google) Code Review
Browse files

Merge "Clean up proguard.flags." into udc-dev

parents 7000dc95 9941659a

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);

        }