Fix saving naming styles options page

Merge pull request #34195 from jasonmalinowski/fix-race-in-getopendocumentincurrentcontextwithchanges

Fix race in GetOpenDocumentInCurrentContextWithChanges

Respect the iteration parameter in IterationData

Workaround a deadlock caused by watching .editorconfigs

Skip test DecimalBinaryOp_03 (#34199)

- Allow page controls to store settings to option store before updating service
- Reload page control settings after a cancel or save

This was reading Workspace.CurrentSolution twice, so if the document
was removed between two reads it'd end up throwing exceptions or
triggering asserts.

Add a hack to ensure documents are connected to open files

We use the CodingConventions library from Visual Studio that parses
and processes .editorconfig files for us. This library under the covers
uses the Visual Studio file watching service for watching files.
In Visual Studio 2017 the file watching service made the guarantee that
watching files could be done in a free-threaded manner in the background
and wouldn't marshal. The .editorconfig library then passed this
guarantee along to us.

In Dev16, the file watching service was rewritten, and accidentally
lost that guarantee in specific cases. Much of our interactions with
the file change service also changed so it doesn't matter, but
the .editorconfig library's use of the file watching service is causing
deadlocks in some scenarios. This change is a workaround to ensure
we don't start watching .editorconfig files on the background thread
in places where we need to ensure we don't touch the UI thread.

Ideally we'd be fixing the file change service itself to restore the
guarantee, but that's too risky of a change for our current needs and
isn't something we can do quickly enough to avoid customer issues.
Since we're rewriting our .editorconfig handling entirely which will
cause us to stop using the .editorconfig library (and by extension,
it's use of this problematic file change service path), we can do
this workaround now and let the problem go away on it's own soon when
we simply delete all of this.

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/810494

include the correct compiler resources assemblies for pt-BR

In https://devdiv.visualstudio.com/DevDiv/_workitems/edit/728035, a file
is opened in an invisible editor and contents are written to it. The
file isn't saved, but it's added to the workspace; we won't have yet
hooked up to the open file since that work was deferred. Since we're on
the UI thread here, we can ensure those are all wired up since the
analysis of this document may depend on that other file. We choose to do
this here rather than in the project system code when it's added because
we don't want to pay the penalty of checking the RDT for all files being
opened on the UI thread if we really don't need it.

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/728035.

handle extension methods in "Sync Namespace" refactoring

Co-Authored-By: Ngenlu <genlu@users.noreply.github.com>

Close the Find Source window in case it's open

Split CloseFile into CloseCodeFile and CloseDesignerFile

Fixes #26442
Fixes #26443
Fixes #26444

See #33785

Port #33631 to dev16.0 (Low priority for code wrapping refactorings)

Code wrapping refactorings should be lower pri than other refactorings

(cherry picked from commit 8d530655)

Allow specific exception without crash in integration tests

Delayed failures in integration tests

This change ensures that logs are created even if the application fails
to capture a screenshot.

VisualStudioInstanceFactory failures now occur in a context where they
are reported as test failures instead of test runner crashes.

This was called from either ProcessQueuedWorkOnUIThread or externally
from the workspace when it's being created. It's easier for us to just
have ProcessQueuedWorkOnUIThread be the single method that's used
everywhere.