GH-395 no checks are needed if heap dump completes without exception

Author: thurka

visualvm/jmx/src/org/graalvm/visualvm/jmx/impl/JmxSupport.java

@@ -35,7 +35,6 @@
 import java.lang.management.ThreadInfo;
 import java.lang.management.ThreadMXBean;
 import java.nio.file.Files;
-import java.nio.file.LinkOption;
 import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.text.SimpleDateFormat;
@@ -370,19 +369,19 @@ private void printLock(StringBuilder sb,LockInfo lock) {
     boolean takeHeapDump(String fileName) {
         HotSpotDiagnosticMXBean hsDiagnostic = getHotSpotDiagnostic();
         if (hsDiagnostic != null) {
-            Path f = Paths.get(fileName);
             try {
                 hsDiagnostic.dumpHeap(fileName,true);
             } catch (IOException ex) {
                 LOGGER.log(Level.INFO,"takeHeapDump", ex); // NOI18N
                 try {
+                    Path f = Paths.get(fileName);
                     Files.deleteIfExists(f);
                 } catch (IOException ex1) {
                     LOGGER.log(Level.INFO,"takeHeapDump", ex1); // NOI18N
                 }
                 return false;
             }
-            return Files.isRegularFile(f, LinkOption.NOFOLLOW_LINKS) && Files.isReadable(f);
+            return true;
         }
         return false;
     }