Remove unnecessary usage of `new String(...)`

According to Findbugs:

  Using the java.lang.String(String) constructor wastes memory because
  the object so constructed will be functionally indistinguishable from
  the String passed as a parameter.

  Just use the argument String directly.

Change-Id: I85a3fd0b85afcae356181eef3e7262d9bf0fa600