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

Commit e12712dd authored by ykhung's avatar ykhung Committed by YUKAI HUNG
Browse files

Add package name into the metric for enable/disable app user actions 

add package name into the metric per @wangkelly request to analyze users behavior in the battery setting relative CUJ:
go/battery-setting-enums

Bug: 199807285
Test: make SettingsRoboTests
Change-Id: Ica6da59b933b0dcff940215d4e537a670bac52cb
parent 855f9e51

src/com/android/settings/applications/appinfo/AppButtonsPreferenceController.java

+15 −3
Original line number Original line Diff line number Diff line
@@ -226,7 +226,8 @@ public class AppButtonsPreferenceController extends BasePreferenceController imp 
                uninstallDaIntent.putExtra(DeviceAdminAdd.EXTRA_DEVICE_ADMIN_PACKAGE_NAME,

                uninstallDaIntent.putExtra(DeviceAdminAdd.EXTRA_DEVICE_ADMIN_PACKAGE_NAME,

                        packageName);

                        packageName);

                mMetricsFeatureProvider.action(mActivity,

                mMetricsFeatureProvider.action(mActivity,

                        SettingsEnums.ACTION_SETTINGS_UNINSTALL_DEVICE_ADMIN);

                        SettingsEnums.ACTION_SETTINGS_UNINSTALL_DEVICE_ADMIN,

                        getPackageNameForMetric());

                mFragment.startActivityForResult(uninstallDaIntent, mRequestRemoveDeviceAdmin);

                mFragment.startActivityForResult(uninstallDaIntent, mRequestRemoveDeviceAdmin);

                return;

                return;

            }

            }
@@ -253,7 +254,8 @@ public class AppButtonsPreferenceController extends BasePreferenceController imp 
                            mActivity,

                            mActivity,

                            mAppEntry.info.enabled

                            mAppEntry.info.enabled

                                    ? SettingsEnums.ACTION_SETTINGS_DISABLE_APP

                                    ? SettingsEnums.ACTION_SETTINGS_DISABLE_APP

                                    : SettingsEnums.ACTION_SETTINGS_ENABLE_APP);

                                    : SettingsEnums.ACTION_SETTINGS_ENABLE_APP,

                                    getPackageNameForMetric());

                    AsyncTask.execute(new DisableChangerRunnable(mPm, mAppEntry.info.packageName,

                    AsyncTask.execute(new DisableChangerRunnable(mPm, mAppEntry.info.packageName,

                            PackageManager.COMPONENT_ENABLED_STATE_DEFAULT));

                            PackageManager.COMPONENT_ENABLED_STATE_DEFAULT));

                }

                }
@@ -270,7 +272,9 @@ public class AppButtonsPreferenceController extends BasePreferenceController imp 
        @Override

        @Override

        public void onClick(View v) {

        public void onClick(View v) {

             mMetricsFeatureProvider.action(

             mMetricsFeatureProvider.action(

                     mActivity, SettingsEnums.ACTION_APP_INFO_FORCE_STOP);

                     mActivity,

                     SettingsEnums.ACTION_APP_INFO_FORCE_STOP,

                     getPackageNameForMetric());

            // force stop

            // force stop

            if (mPm.isPackageStateProtected(mAppEntry.info.packageName, mUserId)) {

            if (mPm.isPackageStateProtected(mAppEntry.info.packageName, mUserId)) {

                RestrictedLockUtils.sendShowAdminSupportDetailsIntent(mActivity,

                RestrictedLockUtils.sendShowAdminSupportDetailsIntent(mActivity,
@@ -757,6 +761,14 @@ public class AppButtonsPreferenceController extends BasePreferenceController imp 
                || AppUtils.isMainlineModule(mPm, mAppEntry.info.packageName));

                || AppUtils.isMainlineModule(mPm, mAppEntry.info.packageName));

    }

    }





    private String getPackageNameForMetric() {

        final String packageName =

                mAppEntry != null && mAppEntry.info != null

                        ? mAppEntry.info.packageName

                        : null;

        return packageName != null ? packageName : "";

    }



    /**

    /**

     * Changes the status of disable/enable for a package

     * Changes the status of disable/enable for a package

     */

     */