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

Commit ea273b5c authored by Jeff Sharkey's avatar Jeff Sharkey Committed by Android (Google) Code Review
Browse files

Merge "Use new VolumeInfo helper methods."

parents 463ae4b2 90daa80c
Loading
Loading
Loading
Loading
+5 −6
Original line number Diff line number Diff line
@@ -53,10 +53,9 @@ import android.widget.EditText;
import com.android.internal.logging.MetricsLogger;
import com.android.internal.util.Preconditions;
import com.android.settings.R;
import com.android.settings.Settings;
import com.android.settings.Settings.StorageUseActivity;
import com.android.settings.SettingsPreferenceFragment;
import com.android.settings.Utils;
import com.android.settings.Settings.StorageUseActivity;
import com.android.settings.applications.ManageApplications;
import com.android.settings.deviceinfo.StorageMeasurement.MeasurementDetails;
import com.android.settings.deviceinfo.StorageMeasurement.MeasurementReceiver;
@@ -75,6 +74,7 @@ import java.util.Objects;
 */
public class PrivateVolumeSettings extends SettingsPreferenceFragment {
    // TODO: disable unmount when providing over MTP/PTP
    // TODO: warn when mounted read-only

    private static final String TAG_RENAME = "rename";
    private static final String TAG_CONFIRM_CLEAR_CACHE = "confirmClearCache";
@@ -129,8 +129,7 @@ public class PrivateVolumeSettings extends SettingsPreferenceFragment {
        addPreferencesFromResource(R.xml.device_info_storage_volume);

        // Find the emulated shared storage layered above this private volume
        mSharedVolume = mStorageManager.findVolumeById(
                mVolume.getId().replace("private", "emulated"));
        mSharedVolume = mStorageManager.findEmulatedForPrivate(mVolume);

        mMeasure = new StorageMeasurement(context, mVolume, mSharedVolume);
        mMeasure.setReceiver(mReceiver);
@@ -171,7 +170,7 @@ public class PrivateVolumeSettings extends SettingsPreferenceFragment {

        screen.removeAll();

        if (mVolume.getState() != VolumeInfo.STATE_MOUNTED) {
        if (!mVolume.isMountedReadable()) {
            return;
        }

@@ -283,7 +282,7 @@ public class PrivateVolumeSettings extends SettingsPreferenceFragment {
        } else {
            rename.setVisible(mVolume.getType() == VolumeInfo.TYPE_PRIVATE);
            mount.setVisible(mVolume.getState() == VolumeInfo.STATE_UNMOUNTED);
            unmount.setVisible(mVolume.getState() == VolumeInfo.STATE_MOUNTED);
            unmount.setVisible(mVolume.isMountedReadable());
            format.setVisible(true);
        }

+2 −2
Original line number Diff line number Diff line
@@ -118,7 +118,7 @@ public class PublicVolumeSettings extends SettingsPreferenceFragment {

        screen.removeAll();

        if (mVolume.getState() == VolumeInfo.STATE_MOUNTED) {
        if (mVolume.isMountedReadable()) {
            screen.addPreference(mGraph);
            screen.addPreference(mTotal);
            screen.addPreference(mAvailable);
@@ -139,7 +139,7 @@ public class PublicVolumeSettings extends SettingsPreferenceFragment {
        if (mVolume.getState() == VolumeInfo.STATE_UNMOUNTED) {
            screen.addPreference(mMount);
        }
        if (mVolume.getState() == VolumeInfo.STATE_MOUNTED) {
        if (mVolume.isMountedReadable()) {
            screen.addPreference(mUnmount);
        }
        screen.addPreference(mFormat);
+1 −1
Original line number Diff line number Diff line
@@ -345,7 +345,7 @@ public class StorageMeasurement {
        final Message finished = mMeasurementHandler.obtainMessage(MeasurementHandler.MSG_COMPLETED,
                details);

        if (mSharedVolume != null && mSharedVolume.getState() == VolumeInfo.STATE_MOUNTED) {
        if (mSharedVolume != null && mSharedVolume.isMountedReadable()) {
            final File basePath = mSharedVolume.getPathForUser(currentUser);

            // Measure media types for emulated storage, or for primary physical
+1 −1
Original line number Diff line number Diff line
@@ -186,7 +186,7 @@ public class StorageSettings extends SettingsPreferenceFragment implements Index
            return true;

        } else if (vol.getType() == VolumeInfo.TYPE_PUBLIC) {
            if (vol.getState() == VolumeInfo.STATE_MOUNTED) {
            if (vol.isMountedReadable()) {
                startActivity(vol.buildBrowseIntent());
                return true;
            } else {
+7 −9
Original line number Diff line number Diff line
@@ -47,14 +47,12 @@ public class StorageVolumePreference extends Preference {
        setKey(volume.getId());
        setTitle(mStorageManager.getBestVolumeDescription(volume));

        switch (volume.getState()) {
            case VolumeInfo.STATE_MOUNTED:
        if (volume.isMountedReadable()) {
            // TODO: move statfs() to background thread
            final File path = volume.getPath();
            final String free = Formatter.formatFileSize(context, path.getFreeSpace());
            final String total = Formatter.formatFileSize(context, path.getTotalSpace());
            setSummary(context.getString(R.string.storage_volume_summary, free, total));
                break;
        }

        // TODO: better icons
@@ -65,7 +63,7 @@ public class StorageVolumePreference extends Preference {
        }

        if (volume.getType() == VolumeInfo.TYPE_PUBLIC
                && volume.getState() == VolumeInfo.STATE_MOUNTED) {
                && volume.isMountedReadable()) {
            setWidgetLayoutResource(R.layout.preference_storage_action);
        }
    }