Only acquire IEditorOptionsFactoryService once for a contained document

Update roslyn-analyzers to enable non-copyable analyzer

* Add option page registration attributes back

* DiagnosticWindowPackage

Closes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1124280

[Dogfooding] Enforce IDE0060 (Remove unused parameter) as build warni…

Add FindReferences tests for Kind column/SymbolUsageInfo for references

Remove unused parameter

Use project.Name as the display name for Class View

Use named tuple elements

Remove workaround in build scripts for #42166

Use stronger types in the formatter

Fix serialization/deserialization of SymbolUsageInfo in SerializableS…

[Dogfooding] Enforce IDE0011 (Add braces) as a build warning for some…

Show progress bar for "Running code analysis on <Project/Solution>" execution

Fixes #44184
Find References "Kind" column uses `SymbolUsageInfo` for its data. `SymbolUsageInfo` is correctly serialized/deserialized when Find References is invoked via [SymbolFinder.FindReferencesServerCallback](https://github.com/dotnet/roslyn/blob/b8585be8e9ff9e30dbe64980e2c9156ae8b6c2bf/src/Workspaces/Core/Portable/FindSymbols/SymbolFinder.FindReferencesServerCallback.cs#L72-L84), as it uses [SerializableReferenceLocation](https://github.com/dotnet/roslyn/blob/a65653a77b2862329052d84816b5fb324f8bc8bd/src/Workspaces/Core/Portable/Remote/RemoteArguments.cs#L156) which uses `SerializableSymbolUsageInfo`. With https://github.com/dotnet/roslyn/pull/43914/, we added a new [FindUsagesServerCallback](https://github.com/dotnet/roslyn/blob/0852232f315e952fee1dc63a0b374614b55c1c69/src/EditorFeatures/Core/FindUsages/IRemoteFindUsagesService.cs#L32) and switched to it for Find references serialization/deserialization. This callback does not use `SerializableSymbolUsageInfo`, hence loses the SymbolUsageInfo for "Kind" column. This change fixes this part.

Restore the 'isWriteOnly' parameter removed in previous commit, and actually set Modifiers.IsWriteOnly flag based on this parameter value. This was caught by a unit test failure.

My prior attempt to enable this as a build warning in https://github.com/dotnet/roslyn/pull/44164 did not work due to #44201. This PR actually adds the build enforcement and fixes the violations.

NFE should no longer collect dumps by default. Instead, we want them to be controlled remotely if possible, or sampled via watson.

Removing the workaround for https://github.com/dotnet/roslyn/issues/42166 identified that IDE0051/IDE0052 were not getting enforced on the build. I will file a follow-up issue to investigate it, but this commit fixes the existing IDE0051/IDE0052 violations in IDE projects where they are set to warning.

That feature has been implemented and Roslyn repo is currently using a toolset which has the compiler with this feature. Additionally, we no longer run analyzers on local builds, so there is no performance impact with this change.

```
# IDE0060: Remove unused parameter
dotnet_diagnostic.IDE0060.severity = warning
```

The snippet's curly braces should be escaped due to the `$`. The `$` isn't actually necessary, so I removed it.

Add IDE benchmark project targeting .NET Core