Bug 551600 - [AutoRefactor immigration #12/117] [cleanup & saveaction]
Use lazy logical operator

- rename method to avoid confusion vis-a-vis GenericVisitor.visitNode()

Change-Id: I160f84fe051b7630ae824b8e2dc99fc07187d159
diff --git a/org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/dom/ASTNodes.java b/org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/dom/ASTNodes.java
index 1430898..dc9b9db 100644
--- a/org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/dom/ASTNodes.java
+++ b/org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/dom/ASTNodes.java
@@ -586,7 +586,7 @@
 	 */
 	public static boolean isPassiveWithoutFallingThrough(final ASTNode node) {
 		final ExprActivityVisitor visitor= new ExprActivityVisitor();
-		visitor.visitNode(node);
+		visitor.traverseNodeInterruptibly(node);
 		return ExprActivity.PASSIVE_WITHOUT_FALLING_THROUGH.equals(visitor.getActivityLevel());
 	}
 
diff --git a/org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/dom/InterruptibleVisitor.java b/org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/dom/InterruptibleVisitor.java
index 1de1b6d..a9a8aa4 100644
--- a/org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/dom/InterruptibleVisitor.java
+++ b/org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/dom/InterruptibleVisitor.java
@@ -21,11 +21,11 @@
  */
 public class InterruptibleVisitor extends ASTVisitor {
     /**
-     * Visit the node.
+     * Traverse the node, silently swallowing {@link AbortSearchException}.
      *
      * @param node The visited node.
      */
-    public void visitNode(ASTNode node) {
+    public void traverseNodeInterruptibly(ASTNode node) {
         try {
             node.accept(this);
         } catch (AbortSearchException e) {