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

Commit 87a136a1 authored by Weng Su's avatar Weng Su Committed by Android (Google) Code Review
Browse files

Merge "Avoid flicker in Internet dialog" into sc-v2-dev

parents 3695613b 9362b83b

packages/SystemUI/src/com/android/systemui/qs/tiles/dialog/InternetDialog.java

+9 −4
Original line number Diff line number Diff line
@@ -128,7 +128,9 @@ public class InternetDialog extends SystemUIDialog implements 
    private boolean mCanConfigMobileData;



    // Wi-Fi entries

    @VisibleForTesting

    protected WifiEntry mConnectedWifiEntry;

    @VisibleForTesting

    protected int mWifiEntriesCount;



    // Wi-Fi scanning progress bar
@@ -334,6 +336,9 @@ public class InternetDialog extends SystemUIDialog implements 
        mSeeAllLayout.setOnClickListener(v -> onClickSeeMoreButton());

        mWiFiToggle.setOnCheckedChangeListener(

                (buttonView, isChecked) -> {

                    if (isChecked) {

                        mWifiScanNotifyLayout.setVisibility(View.GONE);

                    }

                    buttonView.setChecked(isChecked);

                    mWifiManager.setWifiEnabled(isChecked);

                });
@@ -576,12 +581,12 @@ public class InternetDialog extends SystemUIDialog implements 
    @WorkerThread

    public void onAccessPointsChanged(@Nullable List<WifiEntry> wifiEntries,

            @Nullable WifiEntry connectedEntry) {

        mHandler.post(() -> {

            mConnectedWifiEntry = connectedEntry;

            mWifiEntriesCount = wifiEntries == null ? 0 : wifiEntries.size();

            updateDialog(false /* shouldUpdateMobileNetwork */);

            mAdapter.setWifiEntries(wifiEntries, mWifiEntriesCount);

        mHandler.post(() -> {

            mAdapter.notifyDataSetChanged();

            updateDialog(false /* shouldUpdateMobileNetwork */);

        });

    }

packages/SystemUI/tests/src/com/android/systemui/qs/tiles/dialog/InternetDialogTest.java

+6 −4
Original line number Diff line number Diff line
@@ -99,7 +99,8 @@ public class InternetDialogTest extends SysuiTestCase { 
                mInternetDialogController, true, true, true, mock(UiEventLogger.class), mHandler,

                mBgExecutor);

        mInternetDialog.mAdapter = mInternetAdapter;

        mInternetDialog.onAccessPointsChanged(mWifiEntries, mInternetWifiEntry);

        mInternetDialog.mConnectedWifiEntry = mInternetWifiEntry;

        mInternetDialog.mWifiEntriesCount = mWifiEntries.size();

        mInternetDialog.show();



        mDialogView = mInternetDialog.mDialogView;
@@ -209,7 +210,7 @@ public class InternetDialogTest extends SysuiTestCase { 
    @Test

    public void updateDialog_wifiOnAndNoConnectedWifi_hideConnectedWifi() {

        // The precondition WiFi ON is already in setUp()

        mInternetDialog.onAccessPointsChanged(mWifiEntries, null /* connectedEntry*/);

        mInternetDialog.mConnectedWifiEntry = null;

        doReturn(false).when(mInternetDialogController).activeNetworkIsCellular();



        mInternetDialog.updateDialog(false);
@@ -220,7 +221,7 @@ public class InternetDialogTest extends SysuiTestCase { 
    @Test

    public void updateDialog_wifiOnAndNoWifiList_hideWifiListAndSeeAll() {

        // The precondition WiFi ON is already in setUp()

        mInternetDialog.onAccessPointsChanged(null /* wifiEntries */, mInternetWifiEntry);

        mInternetDialog.mWifiEntriesCount = 0;



        mInternetDialog.updateDialog(false);
@@ -366,7 +367,8 @@ public class InternetDialogTest extends SysuiTestCase { 
    public void showProgressBar_wifiEnabledWithoutWifiEntries_showProgressBarThenHideSearch() {

        Mockito.reset(mHandler);

        when(mWifiManager.isWifiEnabled()).thenReturn(true);

        mInternetDialog.onAccessPointsChanged(null /* wifiEntries */, null /* connectedEntry*/);

        mInternetDialog.mConnectedWifiEntry = null;

        mInternetDialog.mWifiEntriesCount = 0;



        mInternetDialog.showProgressBar();