diff options
author | John Reck <jreck@google.com> | 2016-06-27 07:56:37 -0700 |
---|---|---|
committer | John Reck <jreck@google.com> | 2016-06-27 07:56:37 -0700 |
commit | 474659ccc421aaacfa49a742891aefed46c6f5d2 (patch) | |
tree | e1f7686754d98bfe8c8dd8094e9d039069c4da5f | |
parent | 61ae65f260f4b14aa13f02a7e6e690d30e672255 (diff) | |
download | base-474659ccc421aaacfa49a742891aefed46c6f5d2.tar.gz |
Add missing null checks
Bug: 29743482
Change-Id: Ic4bc5b9d5718eabd41e09a108a765b01aef4e3c8
-rw-r--r-- | core/java/android/view/SurfaceView.java | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/core/java/android/view/SurfaceView.java b/core/java/android/view/SurfaceView.java index ac43bb7adb8c..203b82563ef8 100644 --- a/core/java/android/view/SurfaceView.java +++ b/core/java/android/view/SurfaceView.java @@ -745,6 +745,12 @@ public class SurfaceView extends View { Log.d(TAG, String.format("%d windowPositionLostRT RT, frameNr = %d", System.identityHashCode(this), frameNumber)); } + IWindowSession session = mSession; + MyWindow window = mWindow; + if (session == null || window == null) { + // We got detached prior to receiving this, abort + return; + } if (mRtHandlingPositionUpdates) { mRtHandlingPositionUpdates = false; // This callback will happen while the UI thread is blocked, so we can @@ -757,7 +763,7 @@ public class SurfaceView extends View { "postion = [%d, %d, %d, %d]", System.identityHashCode(this), mWinFrame.left, mWinFrame.top, mWinFrame.right, mWinFrame.bottom)); - mSession.repositionChild(mWindow, mWinFrame.left, mWinFrame.top, + session.repositionChild(window, mWinFrame.left, mWinFrame.top, mWinFrame.right, mWinFrame.bottom, frameNumber, mWinFrame); } catch (RemoteException ex) { Log.e(TAG, "Exception from relayout", ex); |