[cDAC] Remove superfluous modifiers from contract interfaces

[max-charlamb]

#113769 (comment)

[Copilot]

Pull Request Overview

This PR removes redundant modifiers from contract interface method declarations to adhere to modern C# practices.

• Removed explicit "public virtual" modifiers from interface members in multiple files.
• Updated method signatures across various contract interfaces for consistency.

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
IStressLog.cs	Removed modifiers from interface methods.
IStackWalk.cs	Removed redundant modifiers from interface methods.
IRuntimeTypeSystem.cs	Removed explicit modifiers from numerous interface methods.
IObject.cs	Removed modifiers from interface methods.
ILoader.cs	Removed modifiers from interface methods.
IException.cs	Removed modifiers from interface methods.
IEcmaMetadata.cs	Removed modifiers from interface methods.
IDacStreams.cs	Removed modifiers from interface methods.

Comments suppressed due to low confidence (1)

src/native/managed/cdacreader/Microsoft.Diagnostics.DataContractReader.Abstractions/Contracts/IStressLog.cs:40

• [nitpick] The parameter name 'Logs' is in PascalCase; consider renaming it to 'logs' to follow typical camelCase conventions for parameter names.

IEnumerable<ThreadStressLogData> GetThreadStressLogs(TargetPointer Logs) => throw new NotImplementedException();

[max-charlamb]

/azp run runtime-diagnostics

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[dotnet-policy-service]

Tagging subscribers to this area: @tommcdon
See info in area-owners.md if you want to be subscribed.

[max-charlamb]

/ba-g #113883

Known tests failures are unrelated