Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0a00842b authored by Ziad Youssef's avatar Ziad Youssef
Browse files

Fix default provider logic

Test:  atest com.android.server.webkit.WebViewUpdateServiceTest

Bug: 308907090
Change-Id: I8ec2a7d83840bd52d3411bc43ff40e7b7823bbd9
parent 351cdcd1
Loading
Loading
Loading
Loading
+9 −4
Original line number Diff line number Diff line
@@ -107,16 +107,21 @@ 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.
        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
    public void packageStateChanged(String packageName, int changedState, int userId) {