From 38bb49b055a31a7df1b109895750d50f02b800f6 Mon Sep 17 00:00:00 2001 From: "Tadashi G. Takaoka" Date: Tue, 8 Oct 2019 15:00:42 +0900 Subject: [PATCH] Fix potential NPE in SystemUITests Bug: 142293305 Test: atest QSTileHostTest Change-Id: I0835ef3e5d7bf0cbfcec28fd9e79b8639264bd1f --- .../tests/src/com/android/systemui/qs/QSTileHostTest.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/SystemUI/tests/src/com/android/systemui/qs/QSTileHostTest.java b/packages/SystemUI/tests/src/com/android/systemui/qs/QSTileHostTest.java index 07fbbcf4f1526..77520140b1a3b 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/qs/QSTileHostTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/qs/QSTileHostTest.java @@ -60,6 +60,7 @@ import java.io.FileDescriptor; import java.io.PrintWriter; import java.io.StringWriter; import java.util.List; +import java.util.Objects; import javax.inject.Provider; @@ -190,8 +191,7 @@ public class QSTileHostTest extends SysuiTestCase { // changed String newSetting = Settings.Secure.getStringForUser(getContext().getContentResolver(), TILES_SETTING, ActivityManager.getCurrentUser()); - // newSetting is not null, as it has just been set. - if (!newSetting.equals(previousSetting)) { + if (!Objects.equals(newSetting, previousSetting)) { onTuningChanged(TILES_SETTING, newSetting); } }