RFR: 8354949: JFR: Split up the EventInstrumentation class [v4]

[Erik Gahlin]

> Could I have a review of a PR that cleans up the EventInstrumentation class and moves the non-modifying parts into a separate class (ClassInspector)? 
> 
> I deleted the code that removes the instrumentation (buildUninstrumented), because it has never been used. If there should be a need for it in the future, it's not hard to add code that inserts empty method bodies.
> 
> Testing: tier1 + jdk/jdk/jfr/test
> 
> Thanks
> Erik

Erik Gahlin has updated the pull request incrementally with one additional commit since the last revision:

  Remove accidentally added printStackTrace()

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/24726/files
  - new: https://git.openjdk.org/jdk/pull/24726/files/07a56859..d6de2665

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=24726&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=24726&range=02-03

  Stats: 2 lines in 1 file changed: 0 ins; 2 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/24726.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24726/head:pull/24726

PR: https://git.openjdk.org/jdk/pull/24726