The problem here was when we do cycle detection, we tried to look
at the source of the candidate constructor that we are looking at
delegating to. If we are in a cross-language case, the target might
be from metadata via the skeleton reference and we don't have source.
Rather than fixing the code that assumes it's a source method, I'm
making a change that says if the constructor you're delegating to is
in the base type, then we can't create a cycle and can skip the check
entirely.

We tried to locate a constructor to delegate to; in that case we would
grab the types of the constructor and make sure the original call site
actually can use those types. In the cross-language case, we'd mix up
types from different languages. The correct behavior here is to map
the candiate type back to the language of the call site, since it's
the call site's language rules that matter.

Fixes https://github.com/dotnet/roslyn/issues/49850

Workaround publishing error due to powershell task change

* Initialize VS services on demand on the UI thread.
Avoid tracking "documents" with empty paths (e.g. while diffing).
RDT info does not always contain the hierarchy when closing. Switch to a more reliable cookie-based tracking system instead.

* Addressed feedback

* Use GetRequiredDocument().
Add reference to issue #49879.

* Fixed AutoInsertHandler signature

[master] Update dependencies from dotnet/arcade

Fix crash where we try previewing a generated document; general cleanup

Update our test generator for integration tests to generate multiple files

Since we were previously throwing exceptions if the file couldn't be
found, the default we were using didn't really matter. Now we'll try
something (slightly) more sane.

In the preview changes dialog we'd call this method to try to get an
icon for it; if the file was a new file we'd throw exceptions which
were swallowed by the shell.

1. Remove the statics, most importantly the static TextView which meant
   we would leak the text view when the preview window was closed. The
   instance of the PreviewTagger is now placed in the property bag of
   the view itself.
2. Null annotate things.
3. Rename PreviewUpdater._currentDocument to _currentDocumentId.
3. Remove the logic in UpdateView that was checking if the buffer needed
   to be updated. The content check was expensive (it'd realize the
   buffer into a single string...twice), and was more easily done as a
   check inside of UpdateBuffer to see if the documentId is changing.
4. Replace calls for GetTextAsync() to GetTextSynchronously, since we're
   blocking the UI thread anyways.

* Ensure we don't create duplicate names in AddedSources_Are_Deterministic test
* Use TestBase.GetUniqueName()

Added support to normalize accessor list for a property/indexer/event on a single line when there are no accessors with block body. (#49495)

The discussion started here: https://github.com/dotnet/roslyn/issues/49468

The work consisted in:
- Added support in the SyntaxNormalizer class for the auto-properties (the ones who have null Body property)
- Added more tests for properties including init accessor and initializers
- Fixed the two existing tests (property and indexer) to reflect the changes

The following comparisons shows the results of the actual tests, before and after this PR.

```
Before:
class a
{
  b c
  {
    get;
  }
}

After:
class a
{
  b c { get; }
}

```

```
Before:
class a
{
  int X
  {
    get;
    set;
  }

  = 2;
}

After:
class a
{
  int X { get; set; } = 2;
}
```

```
Before:
class a
{
  int Y
  {
    get;
    set;
  }

  = 99;
}

After:
class a
{
  int Y { get; set; } = 99;
}
```

```
Before:
class a
{
  int Z
  {
    get;
  }
}

After:
class a
{
  int Z { get; }
}

```

```
Before:
class a
{
  int T
  {
    get;
    init;
  }

  int R
  {
    get => 1;
  }
}

After:
class a
{
  int T { get; init; }

  int R { get => 1; }
}

```

```
Before:
class a
{
  int Q
  {
    get
    {
      return 0;
    }

    init
    {
    }
  }

  int R
  {
    get => 1;
  }
}

After:
class a
{
  int Q
  {
    get
    {
      return 0;
    }

    init
    {
    }
  }

  int R { get => 1; }
}

```

```
Before:
class a
{
  int R
  {
    get => 1;
  }
}

After:
class a
{
  int R { get => 1; }
}
```

```
Before:
class a
{
  int S => 2;
}

After:
class a
{
  int S => 2;
}
```

```
Before:
class a
{
  b this[c d]
  {
    get;
  }
}

After:
class a
{
  b this[c d] { get; }
}


```

We had a regression (https://github.com/dotnet/roslyn/pull/49673) which
only broke if a generator produced multiple files. To prevent that
regression from happening again, update our integration test to generate
more than one file, so that case is covered.

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

Ensure that each generic local function gets its own container to cache dynamic call-sites. (#49802)

Fixes #49599.

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

Fixes #49754.

Restore internal API used by VS for Mac

Merge release/dev16.9 to master

Refactor brace completion to split into Features vs EditorFeatures

Update Default RazorDiagnosticMode to Pull API

Revert "Increase CoreCLR dump information"

Stop offering to add DebuggerDisplay on static types 

Merge release/dev16.8 to release/dev16.9

Add StackOverflow InteractiveWindow test for .NET Core