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

Commit 4a71842d authored by George Mount's avatar George Mount Committed by Android (Google) Code Review
Browse files

Merge "Address API council feedback." into oc-dev

parents a97608ca a0ffafff

api/current.txt

+1 −1
Original line number Original line Diff line number Diff line
@@ -4854,7 +4854,6 @@ package android.app { 
    method public abstract android.app.FragmentTransaction remove(android.app.Fragment);

    method public abstract android.app.FragmentTransaction remove(android.app.Fragment);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment, java.lang.String);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment, java.lang.String);

    method public abstract android.app.FragmentTransaction setAllowOptimization(boolean);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(int);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(int);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(java.lang.CharSequence);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(java.lang.CharSequence);

    method public abstract android.app.FragmentTransaction setBreadCrumbTitle(int);

    method public abstract android.app.FragmentTransaction setBreadCrumbTitle(int);
@@ -4862,6 +4861,7 @@ package android.app { 
    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int);

    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int);

    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int, int, int);

    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int, int, int);

    method public abstract android.app.FragmentTransaction setPrimaryNavigationFragment(android.app.Fragment);

    method public abstract android.app.FragmentTransaction setPrimaryNavigationFragment(android.app.Fragment);

    method public abstract android.app.FragmentTransaction setReorderingAllowed(boolean);

    method public abstract android.app.FragmentTransaction setTransition(int);

    method public abstract android.app.FragmentTransaction setTransition(int);

    method public abstract android.app.FragmentTransaction setTransitionStyle(int);

    method public abstract android.app.FragmentTransaction setTransitionStyle(int);

    method public abstract android.app.FragmentTransaction show(android.app.Fragment);

    method public abstract android.app.FragmentTransaction show(android.app.Fragment);

api/system-current.txt

+1 −1
Original line number Original line Diff line number Diff line
@@ -5022,7 +5022,6 @@ package android.app { 
    method public abstract android.app.FragmentTransaction remove(android.app.Fragment);

    method public abstract android.app.FragmentTransaction remove(android.app.Fragment);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment, java.lang.String);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment, java.lang.String);

    method public abstract android.app.FragmentTransaction setAllowOptimization(boolean);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(int);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(int);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(java.lang.CharSequence);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(java.lang.CharSequence);

    method public abstract android.app.FragmentTransaction setBreadCrumbTitle(int);

    method public abstract android.app.FragmentTransaction setBreadCrumbTitle(int);
@@ -5030,6 +5029,7 @@ package android.app { 
    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int);

    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int);

    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int, int, int);

    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int, int, int);

    method public abstract android.app.FragmentTransaction setPrimaryNavigationFragment(android.app.Fragment);

    method public abstract android.app.FragmentTransaction setPrimaryNavigationFragment(android.app.Fragment);

    method public abstract android.app.FragmentTransaction setReorderingAllowed(boolean);

    method public abstract android.app.FragmentTransaction setTransition(int);

    method public abstract android.app.FragmentTransaction setTransition(int);

    method public abstract android.app.FragmentTransaction setTransitionStyle(int);

    method public abstract android.app.FragmentTransaction setTransitionStyle(int);

    method public abstract android.app.FragmentTransaction show(android.app.Fragment);

    method public abstract android.app.FragmentTransaction show(android.app.Fragment);

api/test-current.txt

+1 −1
Original line number Original line Diff line number Diff line
@@ -4867,7 +4867,6 @@ package android.app { 
    method public abstract android.app.FragmentTransaction remove(android.app.Fragment);

    method public abstract android.app.FragmentTransaction remove(android.app.Fragment);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment, java.lang.String);

    method public abstract android.app.FragmentTransaction replace(int, android.app.Fragment, java.lang.String);

    method public abstract android.app.FragmentTransaction setAllowOptimization(boolean);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(int);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(int);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(java.lang.CharSequence);

    method public abstract android.app.FragmentTransaction setBreadCrumbShortTitle(java.lang.CharSequence);

    method public abstract android.app.FragmentTransaction setBreadCrumbTitle(int);

    method public abstract android.app.FragmentTransaction setBreadCrumbTitle(int);
@@ -4875,6 +4874,7 @@ package android.app { 
    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int);

    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int);

    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int, int, int);

    method public abstract android.app.FragmentTransaction setCustomAnimations(int, int, int, int);

    method public abstract android.app.FragmentTransaction setPrimaryNavigationFragment(android.app.Fragment);

    method public abstract android.app.FragmentTransaction setPrimaryNavigationFragment(android.app.Fragment);

    method public abstract android.app.FragmentTransaction setReorderingAllowed(boolean);

    method public abstract android.app.FragmentTransaction setTransition(int);

    method public abstract android.app.FragmentTransaction setTransition(int);

    method public abstract android.app.FragmentTransaction setTransitionStyle(int);

    method public abstract android.app.FragmentTransaction setTransitionStyle(int);

    method public abstract android.app.FragmentTransaction show(android.app.Fragment);

    method public abstract android.app.FragmentTransaction show(android.app.Fragment);

core/java/android/app/BackStackRecord.java

+13 −15
Original line number Original line Diff line number Diff line
@@ -16,8 +16,6 @@ 




package android.app;

package android.app;





import android.content.Context;

import android.content.pm.ApplicationInfo;

import android.os.Build;

import android.os.Build;

import android.os.Parcel;

import android.os.Parcel;

import android.os.Parcelable;

import android.os.Parcelable;
@@ -45,7 +43,7 @@ final class BackStackState implements Parcelable { 
    final CharSequence mBreadCrumbShortTitleText;

    final CharSequence mBreadCrumbShortTitleText;

    final ArrayList<String> mSharedElementSourceNames;

    final ArrayList<String> mSharedElementSourceNames;

    final ArrayList<String> mSharedElementTargetNames;

    final ArrayList<String> mSharedElementTargetNames;

    final boolean mAllowOptimization;

    final boolean mReorderingAllowed;





    public BackStackState(FragmentManagerImpl fm, BackStackRecord bse) {

    public BackStackState(FragmentManagerImpl fm, BackStackRecord bse) {

        final int numOps = bse.mOps.size();

        final int numOps = bse.mOps.size();
@@ -75,7 +73,7 @@ final class BackStackState implements Parcelable { 
        mBreadCrumbShortTitleText = bse.mBreadCrumbShortTitleText;

        mBreadCrumbShortTitleText = bse.mBreadCrumbShortTitleText;

        mSharedElementSourceNames = bse.mSharedElementSourceNames;

        mSharedElementSourceNames = bse.mSharedElementSourceNames;

        mSharedElementTargetNames = bse.mSharedElementTargetNames;

        mSharedElementTargetNames = bse.mSharedElementTargetNames;

        mAllowOptimization = bse.mAllowOptimization;

        mReorderingAllowed = bse.mReorderingAllowed;

    }

    }





    public BackStackState(Parcel in) {

    public BackStackState(Parcel in) {
@@ -90,7 +88,7 @@ final class BackStackState implements Parcelable { 
        mBreadCrumbShortTitleText = TextUtils.CHAR_SEQUENCE_CREATOR.createFromParcel(in);

        mBreadCrumbShortTitleText = TextUtils.CHAR_SEQUENCE_CREATOR.createFromParcel(in);

        mSharedElementSourceNames = in.createStringArrayList();

        mSharedElementSourceNames = in.createStringArrayList();

        mSharedElementTargetNames = in.createStringArrayList();

        mSharedElementTargetNames = in.createStringArrayList();

        mAllowOptimization = in.readInt() != 0;

        mReorderingAllowed = in.readInt() != 0;

    }

    }





    public BackStackRecord instantiate(FragmentManagerImpl fm) {

    public BackStackRecord instantiate(FragmentManagerImpl fm) {
@@ -133,7 +131,7 @@ final class BackStackState implements Parcelable { 
        bse.mBreadCrumbShortTitleText = mBreadCrumbShortTitleText;

        bse.mBreadCrumbShortTitleText = mBreadCrumbShortTitleText;

        bse.mSharedElementSourceNames = mSharedElementSourceNames;

        bse.mSharedElementSourceNames = mSharedElementSourceNames;

        bse.mSharedElementTargetNames = mSharedElementTargetNames;

        bse.mSharedElementTargetNames = mSharedElementTargetNames;

        bse.mAllowOptimization = mAllowOptimization;

        bse.mReorderingAllowed = mReorderingAllowed;

        bse.bumpBackStackNesting(1);

        bse.bumpBackStackNesting(1);

        return bse;

        return bse;

    }

    }
@@ -154,7 +152,7 @@ final class BackStackState implements Parcelable { 
        TextUtils.writeToParcel(mBreadCrumbShortTitleText, dest, 0);

        TextUtils.writeToParcel(mBreadCrumbShortTitleText, dest, 0);

        dest.writeStringList(mSharedElementSourceNames);

        dest.writeStringList(mSharedElementSourceNames);

        dest.writeStringList(mSharedElementTargetNames);

        dest.writeStringList(mSharedElementTargetNames);

        dest.writeInt(mAllowOptimization ? 1 : 0);

        dest.writeInt(mReorderingAllowed ? 1 : 0);

    }

    }





    public static final Parcelable.Creator<BackStackState> CREATOR

    public static final Parcelable.Creator<BackStackState> CREATOR
@@ -218,7 +216,7 @@ final class BackStackRecord extends FragmentTransaction implements 
    String mName;

    String mName;

    boolean mCommitted;

    boolean mCommitted;

    int mIndex = -1;

    int mIndex = -1;

    boolean mAllowOptimization;

    boolean mReorderingAllowed;





    ArrayList<Runnable> mCommitRunnables;

    ArrayList<Runnable> mCommitRunnables;
@@ -370,7 +368,7 @@ final class BackStackRecord extends FragmentTransaction implements 




    public BackStackRecord(FragmentManagerImpl manager) {

    public BackStackRecord(FragmentManagerImpl manager) {

        mManager = manager;

        mManager = manager;

        mAllowOptimization = mManager.getTargetSdk() > Build.VERSION_CODES.N_MR1;

        mReorderingAllowed = mManager.getTargetSdk() > Build.VERSION_CODES.N_MR1;

    }

    }





    public int getId() {

    public int getId() {
@@ -665,8 +663,8 @@ final class BackStackRecord extends FragmentTransaction implements 
    }

    }





    @Override

    @Override

    public FragmentTransaction setAllowOptimization(boolean allowOptimization) {

    public FragmentTransaction setReorderingAllowed(boolean reorderingAllowed) {

        mAllowOptimization = allowOptimization;

        mReorderingAllowed = reorderingAllowed;

        return this;

        return this;

    }

    }
@@ -800,11 +798,11 @@ final class BackStackRecord extends FragmentTransaction implements 
                default:

                default:

                    throw new IllegalArgumentException("Unknown cmd: " + op.cmd);

                    throw new IllegalArgumentException("Unknown cmd: " + op.cmd);

            }

            }

            if (!mAllowOptimization && op.cmd != OP_ADD && f != null) {

            if (!mReorderingAllowed && op.cmd != OP_ADD && f != null) {

                mManager.moveFragmentToExpectedState(f);

                mManager.moveFragmentToExpectedState(f);

            }

            }

        }

        }

        if (!mAllowOptimization) {

        if (!mReorderingAllowed) {

            // Added fragments are added at the end to comply with prior behavior.

            // Added fragments are added at the end to comply with prior behavior.

            mManager.moveToState(mManager.mCurState, true);

            mManager.moveToState(mManager.mCurState, true);

        }

        }
@@ -859,11 +857,11 @@ final class BackStackRecord extends FragmentTransaction implements 
                default:

                default:

                    throw new IllegalArgumentException("Unknown cmd: " + op.cmd);

                    throw new IllegalArgumentException("Unknown cmd: " + op.cmd);

            }

            }

            if (!mAllowOptimization && op.cmd != OP_REMOVE && f != null) {

            if (!mReorderingAllowed && op.cmd != OP_REMOVE && f != null) {

                mManager.moveFragmentToExpectedState(f);

                mManager.moveFragmentToExpectedState(f);

            }

            }

        }

        }

        if (!mAllowOptimization && moveToState) {

        if (!mReorderingAllowed && moveToState) {

            mManager.moveToState(mManager.mCurState, true);

            mManager.moveToState(mManager.mCurState, true);

        }

        }

    }

    }

core/java/android/app/Fragment.java

+1 −1
Original line number Original line Diff line number Diff line
@@ -2417,7 +2417,7 @@ public class Fragment implements ComponentCallbacks2, OnCreateContextMenuListene 
     * enabled.

     * enabled.

     *

     *

     * @see Activity#postponeEnterTransition()

     * @see Activity#postponeEnterTransition()

     * @see FragmentTransaction#setAllowOptimization(boolean)

     * @see FragmentTransaction#setReorderingAllowed(boolean)

     */

     */

    public void postponeEnterTransition() {

    public void postponeEnterTransition() {

        ensureAnimationInfo().mEnterTransitionPostponed = true;

        ensureAnimationInfo().mEnterTransitionPostponed = true;