From aeeb10aa864bbeb0ae6a0542f948164e12490c0d Mon Sep 17 00:00:00 2001 From: Winson Chung Date: Sat, 27 Jun 2020 23:47:00 -0700 Subject: [PATCH] Add null check for NPE before logging Bug: 160026277 Test: Just adding a null check Change-Id: Ifb51ced88a2c96aecb9fc2181bb97cdd033d77b8 --- .../com/android/systemui/pip/phone/PipTouchHandler.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/pip/phone/PipTouchHandler.java b/packages/SystemUI/src/com/android/systemui/pip/phone/PipTouchHandler.java index 79f99f459ace9..3e06c1eb1ca3f 100644 --- a/packages/SystemUI/src/com/android/systemui/pip/phone/PipTouchHandler.java +++ b/packages/SystemUI/src/com/android/systemui/pip/phone/PipTouchHandler.java @@ -303,8 +303,11 @@ public class PipTouchHandler { hideDismissTarget(); }); - MetricsLoggerWrapper.logPictureInPictureDismissByDrag(mContext, - PipUtils.getTopPipActivity(mContext, mActivityManager)); + Pair topPipActivity = PipUtils.getTopPipActivity(mContext, + mActivityManager); + if (topPipActivity.first != null) { + MetricsLoggerWrapper.logPictureInPictureDismissByDrag(mContext, topPipActivity); + } } });