From 337c1a8c28e7eb9b3b1fee2f1c9b82c595c39cf5 Mon Sep 17 00:00:00 2001 From: Zekan Qian Date: Fri, 14 Oct 2022 15:45:14 +0800 Subject: [PATCH] Do not throw exception when Spa environment is set, ignore the latter one. Bug: 252671269 Test: m RunSettingsRoboTests Change-Id: Ibf2721e2bb8d94a50f79e1d86d28490f298d0aad --- .../settingslib/spa/framework/common/SpaEnvironment.kt | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/packages/SettingsLib/Spa/spa/src/com/android/settingslib/spa/framework/common/SpaEnvironment.kt b/packages/SettingsLib/Spa/spa/src/com/android/settingslib/spa/framework/common/SpaEnvironment.kt index 3885025652f52..f762f6e9bef1b 100644 --- a/packages/SettingsLib/Spa/spa/src/com/android/settingslib/spa/framework/common/SpaEnvironment.kt +++ b/packages/SettingsLib/Spa/spa/src/com/android/settingslib/spa/framework/common/SpaEnvironment.kt @@ -17,6 +17,9 @@ package com.android.settingslib.spa.framework.common import android.app.Activity +import android.util.Log + +private const val TAG = "SpaEnvironment" object SpaEnvironmentFactory { private var spaEnvironment: SpaEnvironment? = null @@ -28,8 +31,10 @@ object SpaEnvironmentFactory { return spaEnvironment!! } set(env: SpaEnvironment) { - if (spaEnvironment != null) - throw UnsupportedOperationException("Spa environment is already set") + if (spaEnvironment != null) { + Log.w(TAG, "Spa environment is already set, ignore the latter one.") + return + } spaEnvironment = env } }