The following method is essentially valueless and perhaps is never called.
497 public static String unEscapeHtml(String value)
replaceAll returns an immutable result and does not change value. Therefore the return value is never changed unless it is null and then it is replaced with an empty string.
value= value.replaceAll(""", "\"");
Further, looking in the source we have two methods with the same intent and name. The second class does not have this failure.