Fixed problem with open documents not always setting preservation mode to PreserveIdentity.  (changeset 1400158)

When showing completion after a MemberBindingExpression in C#, we used to find the closest parent ConditionalAccessExpression. With this approach, invoking completion before "b" in "a?.b?.c" will show members of b instead of a. We must instead walk up the tree until we find the first ConditionalAccessExpression whose question token appears before the start of the MemberBindingExpression (this is also what the compiler does). This is exposed as an extension method so other IDE features can consume it. (changeset 1400157)

     This just changes the FakeSign tool to have the same code as the corefx OSS Sign process.  Their code used the metadata reader and was much more compact.

     Chatted with them about creating a single tool / lib.  But the code involved now is so small we deemed it unnecessary.  (changeset 1400143)

If an ISymUnmanaged* method returns E_NOTIMPL, we should treat that the same way we treat E_FAIL - as an empty result.  An incomplete implementation of ISymUnmanaged* should not prevent the result of expression compilation from succeeding.

Note that we continue to swallow only expected failures - we still want to know about unexpected failures, even at the expense of showing "Internal error" in the UI.

TODO: This change won't actually accomplish anything until the Concord ISymUnmanaged* interfaces are updated to return hresults, rather than throw.

CR: PaNelson, ChuckS (changeset 1399307)

There's no reason not to fake sign builds when we're not producing a real-signed build. This makes it easier to run tests in some environments and also reduces some of the differences between public and internal builds.
***NO_CI***
 (changeset 1399282)

-- Rename DiagnosticDescriptor.HelpLink to HelpLinkUri.
-- Make DiagnosticDescriptor implement IEquatable<DiagnosticDescriptor> (because it already overrides Equals and GetHashCode).
-- Make CodeAction.GetOperationsAsync return an ImmutableArray instead of an IEnumerable.
-- Rename CodeFixContext.RegisterFix methods to "RegisterCodeFix".
-- Rename CodeFixProvider.ComputeFixesAsync to "RegiserCodeFixesAsync".
-- Replace CodeFixProvider.GetFixableDiagnosticsIds method with a FixableDiagnosticsIds property.
 (changeset 1399202)

Add a new internal type DeclaredSymbolInfo which is populated by the syntax trees for NavigateTo to use instead of realizing the compilation/ISymbols. (changeset 1399184)

Added ability for MSBuildWorkspace to add/remove metadata, project and analyzer references during TryApplyChanges. (changeset 1399167)

Add test with the original repro steps from Bug 529772 (Implement BC31499: '{0} is a MustOverride event in the base class 'AbstEvent.Base'. Visual Basic does not support event overriding ...) (changeset 1399133)

1.) Use shorter, friendlier symbol names for the error message.

2.) Create a diagnostic for each source location where a symbol is declared, not just one. This provides better behavior for partial types.

3.) If the fixer can't map a node back to a declaration, walk up the tree until we find one that can.

When mapping a diagnostic location to a node, we might end up on one of the child nodes of a declaration rather than the declaration itself. We need to walk up the tree to find the node representing the declaration itself before we can get a symbol.

4.) Ignore blank lines in the PublicAPI.txt file. Otherwise we get to the end of the compilation and complain that a symbol named "" (empty string) was declared as part of the API, but not found in the compilation. (changeset 1398999)

Align ExportCodeRefactoringProviderAttribute shape with that of ExportCodeFixProviderAttribute. (changeset 1398570)

***NO_CI***
 (changeset 1398445)

I kept this separate to make the diffs clearer.

CR: ToMat; ChuckS (changeset 1398423)

When comparing symbols in "ignore assemblies" mode, we should not compare the ContainingSymbol of the global namespace. That ContainingSymbol is an assembly or module, which we shouldn't compare in "ignore assemblies" mode. (changeset 1398398)

Remove use of specialized Exception in test code and just use InvalidOperationException instead. (changeset 1397892)

Adds some exception filters to Outlining Service and tweaks SafeContinueWithFromAsync so to include the continuation function qualified name in the exception data to make crash dump analysis easier. (changeset 1397871)

added a space to experience e2e getting sources, making change, and subm...

this also moves managedesent.dll to a newer version that has a guard to an issue in esent.dll which time to time crash VS. (changeset 1397816)

     except initialization phase, it is called from etw thread anyway so no need to put it in worker thread. also, it could interfere with CLR shutdown. (changeset 1397806)

Despite what one might expect, the static properties on System.Text.Encoding do not return singletons (calling it twice may not yield the same object). (changeset 1397756)

  <Reference Include="xunit">
    <HintPath>..\..\..\packages\xunit.1.9.2\lib\net20\xunit.dll</HintPath>
  </Reference>

rather than the recently introduced:

  <Reference Include="..\..\..\packages\xunit.1.9.2\lib\net20\xunit.dll" /> (changeset 1397721)

Assembly-qualified names in custom debug info should be produced using the same code path as assembly-qualified names in emitted code.  This fixes an unhandled case (pointers as type arguments).

This also brings us closer to the native behavior: we had started emitting the assembly name for types in the current assembly, which the native compiler did not.

CR: alekseyt; tomat (changeset 1397706)

Assert that Syntax of FieldORPropertyInitializer is never null - which wasn't the case when enum members with implicit values were added. (changeset 1397098)

Remove arbitrary timeout in unit test support. This was causing flakiness during builds if the machine is loaded. (changeset 1397093)

Fix Bug 1102800: Uniformly support fullwidth delimiters in VB interpolated strings, fixes a failing assert (some improvements based on code review feedback) (changeset 1396988)