Use String.valueOf()

This has a slightly better performance
since there is no BuildBuffer construction

Signed-off-by: Alexei Barantsev <[email protected]>

Author: asashour

java/client/src/org/openqa/selenium/os/ProcessUtils.java

@@ -124,7 +124,7 @@ private static int killWinProcess(Process process) {
     int exitValue;
 
     try {
-      killPID("" + getProcessId(process));
+      killPID(String.valueOf(getProcessId(process)));
       exitValue = waitForProcessDeath(process, 10000);
     } catch (Exception ex) {
       LOG.log(Level.WARNING, "Process refused to die after 10 seconds, and couldn't taskkill it", ex);

java/client/src/org/openqa/selenium/remote/RemoteWebDriver.java

@@ -679,7 +679,7 @@ protected void log(SessionId sessionId, String commandName, Object toLog, When w
     if (!logger.isLoggable(level)) {
       return;
     }
-    String text = "" + toLog;
+    String text = String.valueOf(toLog);
     if (commandName.equals(DriverCommand.EXECUTE_SCRIPT)
         || commandName.equals(DriverCommand.EXECUTE_ASYNC_SCRIPT)) {
       if (text.length() > 100 && Boolean.getBoolean("webdriver.remote.shorten_log_messages")) {

java/client/test/org/openqa/selenium/TypingTest.java

@@ -505,7 +505,7 @@ public void testChordReveseShiftHomeSelectionDeletes() {
     element.sendKeys("done" + Keys.HOME);
     assertThat(element.getAttribute("value"), is("done"));
 
-    element.sendKeys("" + Keys.SHIFT + "ALL " + Keys.HOME);
+    element.sendKeys(Keys.SHIFT + "ALL " + Keys.HOME);
     assertThat(element.getAttribute("value"), is("ALL done"));
 
     element.sendKeys(Keys.DELETE);
@@ -516,7 +516,7 @@ public void testChordReveseShiftHomeSelectionDeletes() {
     assertThat( // Note: trailing SHIFT up here
                 result.getText().trim(), containsString(" up: 16"));
 
-    element.sendKeys("" + Keys.DELETE);
+    element.sendKeys(Keys.DELETE);
     assertThat(element.getAttribute("value"), is(""));
   }