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

[Matthias Baesken]

On Mon, 24 Apr 2023 08:13:48 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - Merge branch 'openjdk:master' into goetz_backport_8254711
>  - Backport 3644e26cef71c00e1a2638d2b8bed9c1bda965ca
>  - Backport bc2af47e1e4e6e96020e03a60ce098ddd17f63ba

Marked as reviewed by mbaesken (Reviewer).

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

PR Review: https://git.openjdk.org/jdk17u-dev/pull/1260#pullrequestreview-1399342454