RFR: 8267870: Remove unnecessary char_converter during class loading [v2]

[Coleen Phillimore]

> We create a String object from a Symbol, then swap the '/' for '.' characters, and create another String object.  Seems wasteful.  Removed java_lang_String::char_converted and made externalize_classname take a Symbol.
> 
> Tested with tier1-3 in mach5.

Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:

  Changed name of parameter, some nearby indentation.

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/4173/files
  - new: https://git.openjdk.java.net/jdk/pull/4173/files/00cd12f0..afbffd45

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=4173&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=4173&range=00-01

  Stats: 5 lines in 3 files changed: 1 ins; 0 del; 4 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4173.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4173/head:pull/4173

PR: https://git.openjdk.java.net/jdk/pull/4173