diff --git a/services/core/java/com/android/server/webkit/WebViewUpdater.java b/services/core/java/com/android/server/webkit/WebViewUpdater.java index 56311a3675772..37479c89fd357 100644 --- a/services/core/java/com/android/server/webkit/WebViewUpdater.java +++ b/services/core/java/com/android/server/webkit/WebViewUpdater.java @@ -315,15 +315,6 @@ class WebViewUpdater { } } - // Could not find any installed and enabled package either, use the most stable and - // default-available provider. - // TODO(gsennton) remove this when we have a functional WebView stub. - for (ProviderAndPackageInfo providerAndPackage : providers) { - if (providerAndPackage.provider.availableByDefault) { - return providerAndPackage.packageInfo; - } - } - // This should never happen during normal operation (only with modified system images). mAnyWebViewInstalled = false; throw new WebViewPackageMissingException("Could not find a loadable WebView package"); diff --git a/services/tests/servicestests/src/com/android/server/webkit/WebViewUpdateServiceTest.java b/services/tests/servicestests/src/com/android/server/webkit/WebViewUpdateServiceTest.java index 925f41490e4cd..9c8007a9d4894 100644 --- a/services/tests/servicestests/src/com/android/server/webkit/WebViewUpdateServiceTest.java +++ b/services/tests/servicestests/src/com/android/server/webkit/WebViewUpdateServiceTest.java @@ -1219,14 +1219,11 @@ public class WebViewUpdateServiceTest { runWebViewBootPreparationOnMainSync(); - checkPreparationPhasesForPackage(testPackageName, 1 /* first preparation phase */); - // TODO(gsennton) change this logic to use the code below when we have created a functional - // stub. - //Mockito.verify(mTestSystemImpl, Mockito.never()).onWebViewProviderChanged( - // Matchers.anyObject()); - //WebViewProviderResponse response = mWebViewUpdateServiceImpl.waitForAndGetProvider(); - //assertEquals(WebViewFactory.LIBLOAD_FAILED_LISTING_WEBVIEW_PACKAGES, response.status); - //assertEquals(null, mWebViewUpdateServiceImpl.getCurrentWebViewPackage()); + Mockito.verify(mTestSystemImpl, Mockito.never()).onWebViewProviderChanged( + Matchers.anyObject()); + WebViewProviderResponse response = mWebViewUpdateServiceImpl.waitForAndGetProvider(); + assertEquals(WebViewFactory.LIBLOAD_FAILED_LISTING_WEBVIEW_PACKAGES, response.status); + assertEquals(null, mWebViewUpdateServiceImpl.getCurrentWebViewPackage()); } @Test