Revert "[java-highlighting] Illegal combination of modifiers:"

This reverts commit 7264f47c76068c36b4aa4b33f2f796dc1c2c0312.

GitOrigin-RevId: 3136e780d1cf5511db49b7c1be09df356d05b449

java/java-analysis-impl/src/com/intellij/codeInsight/daemon/impl/analysis/HighlightUtil.java

@@ -956,31 +956,13 @@ public final class HighlightUtil {
       String message = JavaErrorBundle.message("incompatible.modifiers", modifier, incompatible);
       HighlightInfo highlightInfo =
         HighlightInfo.newHighlightInfo(HighlightInfoType.ERROR).range(keyword).descriptionAndTooltip(message).create();
-      String modifierToRemove = getModifierToRemove(modifierList, modifier, incompatible);
-      IntentionAction fix =
-        modifierToRemove == null ? null : getFixFactory().createModifierListFix(modifierList, modifierToRemove, false, false);
-      QuickFixAction.registerQuickFixAction(highlightInfo, fix);
+      QuickFixAction.registerQuickFixAction(highlightInfo, getFixFactory().createModifierListFix(modifierList, modifier, false, false));
       return highlightInfo;
     }
 
     return null;
   }
 
-  @PsiModifier.ModifierConstant @Nullable
-  private static String getModifierToRemove(@NotNull PsiModifierList modifierList,
-                                            @PsiModifier.ModifierConstant @NotNull String modifier,
-                                            String incompatible) {
-    if (PsiModifier.FINAL.equals(incompatible)) {
-      PsiElement parent = modifierList.getParent();
-      if (parent instanceof PsiMethod) {
-        PsiClass aClass = ((PsiMethod)parent).getContainingClass();
-        if (aClass == null) return null;
-        if (aClass.isInterface()) return PsiModifier.FINAL;
-      }
-    }
-    return modifier;
-  }
-
   /**
    * Checks if the supplied modifier list contains incompatible modifiers (e.g. "public private").
    *

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/modifier/afterDefaultFinalMethod1.java

@@ -1,4 +0,0 @@
-// "Make 'x' not final" "true"
-interface X {
-  default void x() {}
-}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/modifier/afterDefaultFinalMethod2.java

@@ -1,4 +0,0 @@
-// "Make 'x' not default" "true"
-abstract class X {
-  final void x() {}
-}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/modifier/afterFinalStaticMethod.java

@@ -1,4 +0,0 @@
-// "Make 'x' not final" "true"
-interface X {
-  static void x() {}
-}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/modifier/beforeDefaultFinalMethod1.java

@@ -1,4 +0,0 @@
-// "Make 'x' not final" "true"
-interface X {
-  default<caret> final void x() {}
-}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/modifier/beforeDefaultFinalMethod2.java

@@ -1,4 +0,0 @@
-// "Make 'x' not default" "true"
-abstract class X {
-  default<caret> final void x() {}
-}

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/modifier/beforeFinalStaticMethod.java

@@ -1,4 +0,0 @@
-// "Make 'x' not final" "true"
-interface X {
-  static<caret> final void x() {}
-}