Fix RequiresFeature lint: setPaymentRequestEnabled should only be called if the feature PAYMENT_REQUEST is present

Ensures the function works correctly even when called without prior WebViewFeature.isFeatureSupported check.

ref. https://developers.google.com/pay/api/android/guides/recipes/using-android-webview#enable-payment-request-api

Author: mataku

packages/webview_flutter/webview_flutter_android/android/src/main/java/io/flutter/plugins/webviewflutter/WebSettingsCompatProxyApi.java

@@ -7,6 +7,7 @@
 import android.webkit.WebSettings;
 import androidx.annotation.NonNull;
 import androidx.webkit.WebSettingsCompat;
+import androidx.webkit.WebViewFeature;
 
 /**
  * Host api implementation for {@link WebSettingsCompat}.
@@ -20,6 +21,8 @@ public WebSettingsCompatProxyApi(@NonNull ProxyApiRegistrar pigeonRegistrar) {
 
   @Override
   public void setPaymentRequestEnabled(@NonNull WebSettings webSettings, boolean enabled) {
-    WebSettingsCompat.setPaymentRequestEnabled(webSettings, enabled);
+    if (WebViewFeature.isFeatureSupported(WebViewFeature.PAYMENT_REQUEST)) {
+      WebSettingsCompat.setPaymentRequestEnabled(webSettings, enabled);
+    }
   }
 }

packages/webview_flutter/webview_flutter_android/android/src/test/java/io/flutter/plugins/webviewflutter/WebSettingsCompatTest.java

@@ -7,22 +7,51 @@
 import static org.junit.Assert.fail;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.mockStatic;
+import static org.mockito.Mockito.never;
 
 import android.webkit.WebSettings;
 import androidx.webkit.WebSettingsCompat;
+import androidx.webkit.WebViewFeature;
+
 import org.junit.Test;
 import org.mockito.MockedStatic;
 
 public class WebSettingsCompatTest {
   @Test
-  public void setPaymentRequestEnabled() {
+  public void setPaymentRequestEnabled_PAYMENT_REQUEST_is_supported() {
     final PigeonApiWebSettingsCompat api =
         new TestProxyApiRegistrar().getPigeonApiWebSettingsCompat();
 
     final WebSettings webSettings = mock(WebSettings.class);
+
+    try (MockedStatic<WebSettingsCompat> mockedStatic = mockStatic(WebSettingsCompat.class)) {
+      try(MockedStatic<WebViewFeature> mockedWebViewFeature = mockStatic(WebViewFeature.class)) {
+        mockedWebViewFeature.when(() -> WebViewFeature.isFeatureSupported(WebViewFeature.PAYMENT_REQUEST)).thenReturn(true);
+        api.setPaymentRequestEnabled(webSettings, true);
+        mockedStatic.verify(() -> WebSettingsCompat.setPaymentRequestEnabled(webSettings, true));
+      } catch (Exception e) {
+        fail(e.toString());
+      }
+    } catch (Exception e) {
+      fail(e.toString());
+    }
+  }
+
+  @Test
+  public void setPaymentRequestEnabled_PAYMENT_REQUEST_is_NOT_supported() {
+    final PigeonApiWebSettingsCompat api =
+            new TestProxyApiRegistrar().getPigeonApiWebSettingsCompat();
+
+    final WebSettings webSettings = mock(WebSettings.class);
+
     try (MockedStatic<WebSettingsCompat> mockedStatic = mockStatic(WebSettingsCompat.class)) {
-      api.setPaymentRequestEnabled(webSettings, true);
-      mockedStatic.verify(() -> WebSettingsCompat.setPaymentRequestEnabled(webSettings, true));
+      try(MockedStatic<WebViewFeature> mockedWebViewFeature = mockStatic(WebViewFeature.class)) {
+        mockedWebViewFeature.when(() -> WebViewFeature.isFeatureSupported(WebViewFeature.PAYMENT_REQUEST)).thenReturn(false);
+        api.setPaymentRequestEnabled(webSettings, true);
+        mockedStatic.verify(() -> WebSettingsCompat.setPaymentRequestEnabled(webSettings, true), never());
+      } catch (Exception e) {
+        fail(e.toString());
+      }
     } catch (Exception e) {
       fail(e.toString());
     }