提交 1f698622 编写于 作者: S sla

8026199: serviceability/sa/jmap-hprof/JMapHProfLargeHeapTest.java Compilation failed

Summary: Fixed a compilation failure due to changed method name
Reviewed-by: sla, jbachorik
Contributed-by: fredrik.arvidsson@oracle.com
上级 290d84e4
...@@ -59,7 +59,7 @@ public class JMapHProfLargeHeapTest { ...@@ -59,7 +59,7 @@ public class JMapHProfLargeHeapTest {
// If we are on MacOSX, test if JMap tool is signed, otherwise return // If we are on MacOSX, test if JMap tool is signed, otherwise return
// since test will fail with privilege error. // since test will fail with privilege error.
if (Platform.isOSX()) { if (Platform.isOSX()) {
String jmapToolPath = JDKToolFinder.getCurrentJDKTool("jmap"); String jmapToolPath = JDKToolFinder.getTestJDKTool("jmap");
ProcessBuilder codesignProcessBuilder = new ProcessBuilder( ProcessBuilder codesignProcessBuilder = new ProcessBuilder(
"codesign", "-v", jmapToolPath); "codesign", "-v", jmapToolPath);
Process codesignProcess = codesignProcessBuilder.start(); Process codesignProcess = codesignProcessBuilder.start();
...@@ -107,7 +107,7 @@ public class JMapHProfLargeHeapTest { ...@@ -107,7 +107,7 @@ public class JMapHProfLargeHeapTest {
System.out.println("Extracted pid: " + pid); System.out.println("Extracted pid: " + pid);
JDKToolLauncher jMapLauncher = JDKToolLauncher JDKToolLauncher jMapLauncher = JDKToolLauncher
.create("jmap", false); .createUsingTestJDK("jmap");
jMapLauncher.addToolArg("-dump:format=b,file=" + pid + "-" jMapLauncher.addToolArg("-dump:format=b,file=" + pid + "-"
+ HEAP_DUMP_FILE_NAME); + HEAP_DUMP_FILE_NAME);
jMapLauncher.addToolArg(String.valueOf(pid)); jMapLauncher.addToolArg(String.valueOf(pid));
......
...@@ -56,7 +56,7 @@ public class JDKToolLauncher { ...@@ -56,7 +56,7 @@ public class JDKToolLauncher {
if (useCompilerJDK) { if (useCompilerJDK) {
executable = JDKToolFinder.getJDKTool(tool); executable = JDKToolFinder.getJDKTool(tool);
} else { } else {
executable = JDKToolFinder.getCurrentJDKTool(tool); executable = JDKToolFinder.getTestJDKTool(tool);
} }
vmArgs.addAll(Arrays.asList(ProcessTools.getPlatformSpecificVMArgs())); vmArgs.addAll(Arrays.asList(ProcessTools.getPlatformSpecificVMArgs()));
} }
...@@ -74,17 +74,15 @@ public class JDKToolLauncher { ...@@ -74,17 +74,15 @@ public class JDKToolLauncher {
} }
/** /**
* Creates a new JDKToolLauncher for the specified tool. * Creates a new JDKToolLauncher for the specified tool in the Tested JDK.
* *
* @param tool * @param tool
* The name of the tool * The name of the tool
* @param useCompilerPath *
* If true use the compiler JDK path, otherwise use the tested
* JDK path.
* @return A new JDKToolLauncher * @return A new JDKToolLauncher
*/ */
public static JDKToolLauncher create(String tool, boolean useCompilerJDK) { public static JDKToolLauncher createUsingTestJDK(String tool) {
return new JDKToolLauncher(tool, useCompilerJDK); return new JDKToolLauncher(tool, false);
} }
/** /**
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册