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

Commit 52fb535e authored by Tomasz Mikolajewski's avatar Tomasz Mikolajewski Committed by Android (Google) Code Review
Browse files

Merge "Do not crash when selecting a lot of files." into nyc-dev

parents 2e7dc2c9 a468d346

src/com/android/documentsui/dirlist/DirectoryFragment.java

+17 −4
Original line number Diff line number Diff line
@@ -45,6 +45,7 @@ import android.graphics.drawable.Drawable; 
import android.net.Uri;

import android.os.AsyncTask;

import android.os.Bundle;

import android.os.Parcel;

import android.os.Parcelable;

import android.provider.DocumentsContract;

import android.provider.DocumentsContract.Document;
@@ -226,7 +227,8 @@ public class DirectoryFragment extends Fragment 
        mStateKey = buildStateKey(mRoot, mDocument);

        mQuery = args.getString(Shared.EXTRA_QUERY);

        mType = args.getInt(Shared.EXTRA_TYPE);

        mSelection = args.getParcelable(Shared.EXTRA_SELECTION);

        final Selection selection = args.getParcelable(Shared.EXTRA_SELECTION);

        mSelection = selection != null ? selection : new Selection();

        mSearchMode = args.getBoolean(Shared.EXTRA_SEARCH_MODE);



        mIconHelper = new IconHelper(context, MODE_GRID);
@@ -290,9 +292,20 @@ public class DirectoryFragment extends Fragment 
        outState.putParcelable(Shared.EXTRA_ROOT, mRoot);

        outState.putParcelable(Shared.EXTRA_DOC, mDocument);

        outState.putString(Shared.EXTRA_QUERY, mQuery);



        // Workaround. To avoid crash, write only up to 512 KB of selection.

        // If more files are selected, then the selection will be lost.

        final Parcel parcel = Parcel.obtain();

        try {

            mSelection.writeToParcel(parcel, 0);

            if (parcel.dataSize() <= 512 * 1024) {

                outState.putParcelable(Shared.EXTRA_SELECTION, mSelection);

        outState.putBoolean(Shared.EXTRA_SEARCH_MODE, mSearchMode);

            }

        } finally {

            parcel.recycle();

        }



        outState.putBoolean(Shared.EXTRA_SEARCH_MODE, mSearchMode);

    }



    @Override