[jdk17u-dev] RFR: 8254711: Add java.security.Provider.getService JFR Event [v2]

[Matthias Baesken]

On Wed, 19 Apr 2023 07:15:02 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

>> I backport this for parity with 17.0.8-oracle.
>> 
>> src/java.base/share/classes/java/security/Provider.java
>> I had to redo the change for getService().
>> I basically changed the control flow to only
>> return at the end and then added the Event
>> generation before the return.
>> 
>> src/jdk.jfr/share/classes/jdk/jfr/internal/instrument/JDKEvents.java
>> Resolved three hunks due to context.
>> 
>> test/lib/jdk/test/lib/jfr/EventNames.java
>> Resolved three hunks due to context.
>> 
>> I added JDK-8294673 JFR: Add SecurityProviderService#threshold to TestActiveSettingEvent.java. Else that test is failing.
>
> Goetz Lindenmaier has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Backport 3644e26cef71c00e1a2638d2b8bed9c1bda965ca

Hi Götz, regarding Provider.java, do we need the  legacyMap.remove  that can be found in head ?  I do not see it in your coding, but maybe it is not needed in 17 ?

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1260#issuecomment-1515995343