Surround with try-catch: disable in resources

There's another fix 'Add catch to try' and this is most likely what people want. Having too many fixes is confusing.

GitOrigin-RevId: c6c03bf52a9b59a14e7e8025e0fa525de7872630

java/java-impl/src/com/intellij/codeInsight/daemon/impl/quickfix/SurroundWithTryCatchFix.java

@@ -26,8 +26,13 @@ public class SurroundWithTryCatchFix implements IntentionAction {
   private PsiElement myElement;
 
   public SurroundWithTryCatchFix(@NotNull PsiElement element) {
+    if (element instanceof PsiExpression &&
+        PsiTreeUtil.getParentOfType(element, PsiResourceListElement.class, false, PsiStatement.class) != null) {
+      // We are inside resource list: there's already a suggestion to add a catch to this try.
+      // Suggesting wrapping with another try-catch is confusing
+      return;
+    }
     if (element instanceof PsiStatement ||
-        element instanceof PsiResourceVariable ||
         (element instanceof PsiExpression &&
          !(element instanceof PsiMethodReferenceExpression) &&
          CodeBlockSurrounder.canSurround(ExpressionUtils.getTopLevelExpression((PsiExpression)element)))) {

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/surroundWithTry/after1.java

@@ -1,17 +0,0 @@
-// "Surround with try/catch" "true"
-class C {
-    static class E1 extends Exception { }
-
-    static class MyResource implements AutoCloseable {
-        public void close() throws E1 { }
-    }
-
-    void m() {
-        try {
-            try (MyResource r = new MyResource()) {
-            }
-        } catch (E1 e1) {
-            <selection>e1.printStackTrace();</selection>
-        }
-    }
-}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/surroundWithTry/before1.java

@@ -1,4 +1,4 @@
-// "Surround with try/catch" "true"
+// "Surround with try/catch" "false"
 class C {
     static class E1 extends Exception { }