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

Commit 90d323ce authored by Yanting Yang's avatar Yanting Yang
Browse files

Move DWB to the fifth position from the bottom on the homepage

Remove relevant configs, logics and tests.

Bug: 187885890
Test: robotests & visual
Change-Id: I309d5290f19bf0b01692dd6f14c4cdcc52066453
parent b769cf80
Loading
Loading
Loading
Loading
+0 −6
Original line number Diff line number Diff line
@@ -495,12 +495,6 @@
    <!-- Authority of advanced device battery prediction -->
    <string name="config_battery_prediction_authority" translatable="false"></string>

    <!-- Packages for overriding tile positions on the homepage -->
    <string-array name="config_homepage_tile_packages" translatable="false"/>

    <!-- Orders for overriding tile positions on the homepage -->
    <integer-array name="config_homepage_tile_orders"/>

    <!-- Whether to handle slot change events -->
    <bool name="config_handle_sim_slot_change">false</bool>

+0 −24
Original line number Diff line number Diff line
@@ -185,9 +185,6 @@ public class DashboardFeatureProviderImpl implements DashboardFeatureProvider {
                pref.setOrder(order + baseOrder);
            }
        }

        overrideTilePosition(tile, pref);

        return outObservers.isEmpty() ? null : outObservers;
    }

@@ -457,25 +454,4 @@ public class DashboardFeatureProviderImpl implements DashboardFeatureProvider {
        }
        return eligibleUsers;
    }

    private void overrideTilePosition(Tile tile, Preference pref) {
        if (FeatureFlagUtils.isEnabled(mContext, FeatureFlags.SILKY_HOME)
                && TextUtils.equals(tile.getCategory(), CategoryKey.CATEGORY_HOMEPAGE)) {
            final String[] homepageTilePackages = mContext.getResources().getStringArray(
                    R.array.config_homepage_tile_packages);
            final int[] homepageTileOrders = mContext.getResources().getIntArray(
                    R.array.config_homepage_tile_orders);
            if (homepageTilePackages.length == 0
                    || homepageTilePackages.length != homepageTileOrders.length) {
                return;
            }

            for (int i = 0; i < homepageTilePackages.length; i++) {
                if (TextUtils.equals(tile.getPackageName(), homepageTilePackages[i])) {
                    pref.setOrder(homepageTileOrders[i]);
                    return;
                }
            }
        }
    }
}
+0 −10
Original line number Diff line number Diff line
@@ -100,14 +100,4 @@

    <!-- Uri to query non-public Slice Uris. -->
    <string name="config_non_public_slice_query_uri" translatable="false">content://com.android.settings.slices/test</string>

    <!-- Packages for overriding tile positions on the homepage -->
    <string-array name="config_homepage_tile_packages" translatable="false">
        <item>com.android.settings</item>
    </string-array>

    <!-- Orders for overriding tile positions on the homepage -->
    <integer-array name="config_homepage_tile_orders">
        <item>100</item>
    </integer-array>
</resources>
+0 −14
Original line number Diff line number Diff line
@@ -672,18 +672,4 @@ public class DashboardFeatureProviderImplTest {
        assertThat(argument.getValue().getIdentifier()).isEqualTo(0);
        verify(mActivity, never()).getSupportFragmentManager();
    }

    @Test
    @Config(qualifiers = "mcc999")
    public void bindPreference_specificHomepageTile_shouldOverridePosition() {
        FeatureFlagUtils.setEnabled(mContext, FeatureFlags.SILKY_HOME, true);
        final Preference preference = new Preference(RuntimeEnvironment.application);
        final Tile tile = new ActivityTile(mActivityInfo, CategoryKey.CATEGORY_HOMEPAGE);

        mImpl.bindPreferenceToTileAndGetObservers(mActivity, mForceRoundedIcon,
                MetricsEvent.VIEW_UNKNOWN, preference, tile, null /*key */,
                Preference.DEFAULT_ORDER);

        assertThat(preference.getOrder()).isEqualTo(100);
    }
}