Merge "Revert "Only call Drawable.setVisible(false) for visible outgoing drawables"" into nyc-dev (6823bc1a) · Commits · e / os / android_frameworks_base

core/java/android/view/View.java

+2 −14

Original line number	Diff line number	Diff line
		@@ -18079,13 +18079,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,
		* to clear the previous drawable. setVisible first while we still have the callback set.
		*/
		if (mBackground != null) {
		// It's possible for this method to be invoked from the View constructor before
		// subclass constructors have run. Drawables can and should trigger invalidations
		// and other activity with their callback on visibility changes, which shouldn't
		// happen before subclass constructors finish. However, we won't have set the
		// drawable as visible until the view becomes attached. This guard below keeps
		// multiple calls to this method from constructors from causing issues.
		if (mBackground.isVisible()) {
		if (isAttachedToWindow()) {
		mBackground.setVisible(false, false);
		}
		mBackground.setCallback(null);
		@@ -18320,13 +18314,7 @@ public class View implements Drawable.Callback, KeyEvent.Callback,
		}

		if (mForegroundInfo.mDrawable != null) {
		// It's possible for this method to be invoked from the View constructor before
		// subclass constructors have run. Drawables can and should trigger invalidations
		// and other activity with their callback on visibility changes, which shouldn't
		// happen before subclass constructors finish. However, we won't have set the
		// drawable as visible until the view becomes attached. This guard below keeps
		// multiple calls to this method from constructors from causing issues.
		if (mForegroundInfo.mDrawable.isVisible()) {
		if (isAttachedToWindow()) {
		mForegroundInfo.mDrawable.setVisible(false, false);
		}
		mForegroundInfo.mDrawable.setCallback(null);

core/java/android/widget/ImageView.java

+3 −9

Original line number	Diff line number	Diff line
		@@ -911,17 +911,11 @@ public class ImageView extends View {
		}

		if (mDrawable != null) {
		// It's possible for this method to be invoked from the constructor before
		// subclass constructors have run. Drawables can and should trigger invalidations
		// and other activity with their callback on visibility changes, which shouldn't
		// happen before subclass constructors finish. However, we won't have set the
		// drawable as visible until the view becomes attached. This guard below keeps
		// multiple calls to this method from constructors from causing issues.
		if (mDrawable.isVisible()) {
		mDrawable.setVisible(false, false);
		}
		mDrawable.setCallback(null);
		unscheduleDrawable(mDrawable);
		if (isAttachedToWindow()) {
		mDrawable.setVisible(false, false);
		}
		}

		mDrawable = d;