Extract Execute method checks to reduce code duplication and improve performance for other registration method

[sebastien-marichal]

#8414 brought performance improvement for SyntaxNodeAction registration.
Unfortunately, this creates some code duplication of the Execute method, especially regarding the checks on whether or not the SyntaxTree should be analyzed.

Here are the checks to cache:

• HasMatchingScope
  • This is partially covered by Reduce time spend in HasMatchingScope for SyntaxNode registration #8399 only for SyntaxNodeAction
• ShouldAnalyzeTree (see Cache RegisterNodeAction checks result per SyntaxTree #8414)
• SonarAnalysisContext.LegacyIsRegisteredActionEnabled (see Cache RegisterNodeAction checks result per SyntaxTree #8414)
• ShouldAnalyzeRazorFile (see Cache RegisterNodeAction checks result per SyntaxTree #8414)

The idea is to follow what was done for RegisterNodeAction and apply it to other registration methods: Register for CompilationStartAction, do the checks or get it from the cache, and execute this action.

This should be possible for all registration actions except for:

• CompilationStartAction
• CompilationAction

Possible solutions:

• Have the checks in CompilationStartActionContext, as of today (for RegisterNodeAction)
• Have the checks in a dedicated method in SonarAnalysisContext.