diff options
| author | 2024-02-21 11:22:03 +0000 | |
|---|---|---|
| committer | 2024-02-21 11:22:03 +0000 | |
| commit | 0a00842bf4239eae4ce69585fd64ffdc6b29a8d0 (patch) | |
| tree | 527dad8f06be656e4b8dc3d59ea3e788a640f514 | |
| parent | 351cdcd11f08ce6f4ceef29239fd2582b6cb6d1c (diff) | |
Fix default provider logic
Test: atest com.android.server.webkit.WebViewUpdateServiceTest
Bug: 308907090
Change-Id: I8ec2a7d83840bd52d3411bc43ff40e7b7823bbd9
| -rw-r--r-- | services/core/java/com/android/server/webkit/WebViewUpdateServiceImpl2.java | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/services/core/java/com/android/server/webkit/WebViewUpdateServiceImpl2.java b/services/core/java/com/android/server/webkit/WebViewUpdateServiceImpl2.java index b3c8b0b3a47a..27c80c43cd94 100644 --- a/services/core/java/com/android/server/webkit/WebViewUpdateServiceImpl2.java +++ b/services/core/java/com/android/server/webkit/WebViewUpdateServiceImpl2.java @@ -107,15 +107,20 @@ class WebViewUpdateServiceImpl2 implements WebViewUpdateServiceInterface { mContext = context; mSystemInterface = systemInterface; WebViewProviderInfo[] webviewProviders = getWebViewPackages(); + + WebViewProviderInfo defaultProvider = null; for (WebViewProviderInfo provider : webviewProviders) { if (provider.availableByDefault) { - mDefaultProvider = provider; + defaultProvider = provider; break; } } - // This should be unreachable because the config parser enforces that there is at least one - // availableByDefault provider. - throw new AndroidRuntimeException("No available by default WebView Provider."); + if (defaultProvider == null) { + // This should be unreachable because the config parser enforces that there is at least + // one availableByDefault provider. + throw new AndroidRuntimeException("No available by default WebView Provider."); + } + mDefaultProvider = defaultProvider; } @Override |