Fix handling of super:: method references in UnnecessaryMethodReference

Fixes #2102

PiperOrigin-RevId: 351900058

Author: cushon

core/src/main/java/com/google/errorprone/bugpatterns/UnnecessaryMethodReference.java

@@ -37,6 +37,7 @@
 import com.google.errorprone.predicates.TypePredicates;
 import com.google.errorprone.util.ASTHelpers.TargetType;
 import com.sun.source.tree.ExpressionTree;
+import com.sun.source.tree.IdentifierTree;
 import com.sun.source.tree.MemberReferenceTree;
 import com.sun.source.tree.MemberReferenceTree.ReferenceMode;
 import com.sun.source.tree.MethodInvocationTree;
@@ -69,6 +70,10 @@ public Description matchMemberReference(MemberReferenceTree tree, VisitorState s
     if (receiver == null) {
       return NO_MATCH;
     }
+    if (receiver instanceof IdentifierTree
+        && ((IdentifierTree) receiver).getName().contentEquals("super")) {
+      return NO_MATCH;
+    }
     if (!isSubtype(getType(receiver), targetType.type(), state)) {
       return NO_MATCH;
     }

core/src/test/java/com/google/errorprone/bugpatterns/UnnecessaryMethodReferenceTest.java

@@ -164,4 +164,23 @@ public void positiveCaseViaConvert() {
             "}")
         .doTest();
   }
+
+  @Test
+  public void ignoreSuper() {
+    helper
+        .addSourceLines(
+            "S.java", //
+            "class S implements Runnable {",
+            "  public void run() {}",
+            "}")
+        .addSourceLines(
+            "Test.java",
+            "abstract class Test extends S {",
+            "  abstract void r(Runnable r);",
+            "  public void run() {",
+            "    r(super::run);",
+            "  }",
+            "}")
+        .doTest();
+  }
 }