diff options
| author | George Mount <mount@google.com> | 2014-10-23 11:11:17 -0700 |
|---|---|---|
| committer | George Mount <mount@google.com> | 2014-10-24 15:26:32 -0700 |
| commit | 83c692efd3c53050fce132dfd2ef21763d3cf010 (patch) | |
| tree | a3960265ff9e5d4c1f23f005ae416dbe59ed9891 /core/java | |
| parent | 104d2484ec605d9673e74cc125d92d41b6968327 (diff) | |
Fix NPE when rejected snapshots are null.
Bug 18055705
Change-Id: Id0b98e7da5510af438c5d66c0ddb00489d581864
Diffstat (limited to 'core/java')
| -rw-r--r-- | core/java/android/app/EnterTransitionCoordinator.java | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/core/java/android/app/EnterTransitionCoordinator.java b/core/java/android/app/EnterTransitionCoordinator.java index add67f2ec8a5..9bf8b3c8ea6b 100644 --- a/core/java/android/app/EnterTransitionCoordinator.java +++ b/core/java/android/app/EnterTransitionCoordinator.java @@ -322,6 +322,7 @@ class EnterTransitionCoordinator extends ActivityTransitionCoordinator { if (mListener != null) { mListener.onRejectSharedElements(rejectedSnapshots); } + removeNullViews(rejectedSnapshots); startRejectedAnimations(rejectedSnapshots); // Now start shared element transition @@ -370,6 +371,16 @@ class EnterTransitionCoordinator extends ActivityTransitionCoordinator { } } + private static void removeNullViews(ArrayList<View> views) { + if (views != null) { + for (int i = views.size() - 1; i >= 0; i--) { + if (views.get(i) == null) { + views.remove(i); + } + } + } + } + private void onTakeSharedElements() { if (!mIsReadyForTransition || mSharedElementsBundle == null) { return; |
