Automated rollback of commit 09c78e0.

*** Reason for rollback ***

-Xdoclint is still broken

*** Original change description ***

Remove an obsolete TODO

The javac bug is long fixed. I noticed this debugging b/451598162, but it's unrelated.

***

PiperOrigin-RevId: 827977814

Author: cushon

core/src/main/java/com/google/errorprone/bugpatterns/javadoc/InvalidLink.java

@@ -24,6 +24,7 @@
 import static com.google.errorprone.bugpatterns.javadoc.Utils.getStartPosition;
 import static com.google.errorprone.bugpatterns.javadoc.Utils.replace;
 import static com.google.errorprone.matchers.Description.NO_MATCH;
+import static com.google.errorprone.util.ErrorProneLog.deferredDiagnosticHandler;
 
 import com.google.common.collect.ImmutableSet;
 import com.google.errorprone.BugPattern;
@@ -46,6 +47,7 @@
 import com.sun.tools.javac.api.JavacTrees;
 import com.sun.tools.javac.tree.DCTree.DCDocComment;
 import com.sun.tools.javac.tree.DCTree.DCText;
+import com.sun.tools.javac.util.Log;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 import javax.lang.model.element.Element;
@@ -143,9 +145,20 @@ public Void visitLink(LinkTree linkTree, Void unused) {
         return super.visitLink(linkTree, null);
       }
       String reference = linkTree.getReference().getSignature();
-      Element element =
-          JavacTrees.instance(state.context)
-              .getElement(new DocTreePath(getCurrentPath(), linkTree.getReference()));
+      Element element = null;
+      Log log = Log.instance(state.context);
+      // Install a deferred diagnostic handler before calling DocTrees.getElement(DocTreePath)
+
+      Log.DeferredDiagnosticHandler deferredDiagnosticHandler = deferredDiagnosticHandler(log);
+      try {
+        element =
+            JavacTrees.instance(state.context)
+                .getElement(new DocTreePath(getCurrentPath(), linkTree.getReference()));
+      } catch (NullPointerException | AssertionError e) {
+        // TODO: cushon - remove if https://bugs.openjdk.org/browse/JDK-8371248 is fixed
+      } finally {
+        log.popDiagnosticHandler(deferredDiagnosticHandler);
+      }
       // Don't warn about fully qualified types; they won't always be known at compile-time.
       if (element != null || reference.contains(".")) {
         return super.visitLink(linkTree, null);