Fix razor completion

src/EditorFeatures/CSharpTest/ChangeSignature/RemoveParametersTests.cs

@@ -273,7 +273,7 @@ public void ChangeSignatureCommandDisabledInSubmission()
         {
             var exportProvider = ExportProviderCache
                 .GetOrCreateExportProviderFactory(
-                    TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(typeof(InteractiveTextBufferSupportsFeatureService)))
+                    TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(typeof(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService)))
                 .CreateExportProvider();
 
             using (var workspace = TestWorkspace.Create(XElement.Parse(@"

src/EditorFeatures/CSharpTest/EncapsulateField/EncapsulateFieldCommandHandlerTests.cs

@@ -194,7 +194,7 @@ public void EncapsulateFieldCommandDisabledInSubmission()
         {
             var exportProvider = ExportProviderCache
                 .GetOrCreateExportProviderFactory(
-                    TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(typeof(InteractiveTextBufferSupportsFeatureService)))
+                    TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(typeof(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService)))
                 .CreateExportProvider();
 
             using (var workspace = TestWorkspace.Create(XElement.Parse(@"

src/EditorFeatures/CSharpTest/ExtractInterface/ExtractInterfaceTests.cs

@@ -1062,7 +1062,7 @@ public void M(T a, U b) { }
         public void ExtractInterfaceCommandDisabledInSubmission()
         {
             var exportProvider = ExportProviderCache
-                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(typeof(InteractiveTextBufferSupportsFeatureService)))
+                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(typeof(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService)))
                 .CreateExportProvider();
 
             using (var workspace = TestWorkspace.Create(XElement.Parse(@"

src/EditorFeatures/CSharpTest/ExtractMethod/ExtractMethodTests.cs

@@ -10353,7 +10353,7 @@ public async Task ExtractMethod_Argument2()
         public void ExtractMethodCommandDisabledInSubmission()
         {
             var exportProvider = ExportProviderCache
-                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(typeof(InteractiveTextBufferSupportsFeatureService)))
+                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(typeof(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService)))
                 .CreateExportProvider();
 
             using (var workspace = TestWorkspace.Create(XElement.Parse(@"

src/EditorFeatures/CSharpTest/Organizing/OrganizeTypeDeclarationTests.cs

@@ -1077,7 +1077,7 @@ void B()
         public void OrganizingCommandsDisabledInSubmission()
         {
             var exportProvider = ExportProviderCache
-                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(typeof(InteractiveTextBufferSupportsFeatureService)))
+                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(typeof(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService)))
                 .CreateExportProvider();
 
             using (var workspace = TestWorkspace.Create(XElement.Parse(@"

src/EditorFeatures/Core.Wpf/Interactive/InteractiveSupportsFeatureService.cs

@@ -0,0 +1,77 @@
+// Copyright (c) Microsoft.  All Rights Reserved.  Licensed under the Apache License, Version 2.0.  See License.txt in the project root for license information.
+
+using System.Composition;
+using Microsoft.CodeAnalysis.Editor.Shared;
+using Microsoft.CodeAnalysis.Host.Mef;
+using Microsoft.VisualStudio.Text;
+using Microsoft.VisualStudio.InteractiveWindow;
+using Microsoft.CodeAnalysis.Shared;
+
+namespace Microsoft.CodeAnalysis.Editor.Implementation.Interactive
+{
+    internal sealed class InteractiveSupportsFeatureService
+    {
+        [ExportWorkspaceService(typeof(ITextBufferSupportsFeatureService), WorkspaceKind.Interactive), Shared]
+        internal class InteractiveTextBufferSupportsFeatureService : ITextBufferSupportsFeatureService
+        {
+            public bool SupportsCodeFixes(ITextBuffer textBuffer)
+            {
+                if (textBuffer != null)
+                {
+                    var evaluator = (IInteractiveEvaluator)textBuffer.Properties[typeof(IInteractiveEvaluator)];
+                    var window = evaluator?.CurrentWindow;
+                    if (window?.CurrentLanguageBuffer == textBuffer)
+                    {
+                        // These are only correct if we're on the UI thread.
+                        // Otherwise, they're guesses and they might change immediately even if they're correct.
+                        // If we return true and the buffer later becomes readonly, it appears that the 
+                        // the code fix simply has no effect.
+                        return !window.IsResetting && !window.IsRunning;
+                    }
+                }
+
+                return false;
+            }
+
+            public bool SupportsRefactorings(ITextBuffer textBuffer)
+            {
+                return false;
+            }
+
+            public bool SupportsRename(ITextBuffer textBuffer)
+            {
+                return false;
+            }
+
+            public bool SupportsNavigationToAnyPosition(ITextBuffer textBuffer)
+            {
+                return true;
+            }
+        }
+
+        [ExportWorkspaceService(typeof(IDocumentSupportsFeatureService), WorkspaceKind.Interactive), Shared]
+        internal class InteractiveDocumentSupportsFeatureService : IDocumentSupportsFeatureService
+        {
+            public bool SupportsCodeFixes(Document document)
+            {
+                // TODO: Implement this.
+                return false;
+            }
+
+            public bool SupportsRefactorings(Document document)
+            {
+                return false;
+            }
+
+            public bool SupportsRename(Document document)
+            {
+                return false;
+            }
+
+            public bool SupportsNavigationToAnyPosition(Document document)
+            {
+                return true;
+            }
+        }
+    }
+}

src/EditorFeatures/Core/Shared/ITextBufferSupportsFeatureService.cs

@@ -1,6 +1,7 @@
 // Copyright (c) Microsoft.  All Rights Reserved.  Licensed under the Apache License, Version 2.0.  See License.txt in the project root for license information.
 
 using Microsoft.CodeAnalysis.Host;
+using Microsoft.CodeAnalysis.Shared;
 using Microsoft.VisualStudio.Text;
 
 namespace Microsoft.CodeAnalysis.Editor.Shared

src/EditorFeatures/Test2/Rename/RenameCommandHandlerTests.vb

@@ -51,7 +51,7 @@ Namespace Microsoft.CodeAnalysis.Editor.UnitTests.Rename
         <Trait(Traits.Feature, Traits.Features.Interactive)>
         Public Sub RenameCommandDisabledInSubmission()
             Dim exportProvider = ExportProviderCache _
-                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveTextBufferSupportsFeatureService))) _
+                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService))) _
                 .CreateExportProvider()
 
             Using workspace = TestWorkspace.Create(

src/EditorFeatures/VisualBasicTest/ChangeSignature/RemoveParametersTests.vb

@@ -96,7 +96,7 @@ End Module
         <Trait(Traits.Feature, Traits.Features.Interactive)>
         Public Sub TestChangeSignatureCommandDisabledInSubmission()
             Dim exportProvider = ExportProviderCache _
-                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveTextBufferSupportsFeatureService))) _
+                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService))) _
                 .CreateExportProvider()
 
             Using workspace = TestWorkspace.Create(

src/EditorFeatures/VisualBasicTest/EncapsulateField/EncapsulateFieldCommandHandlerTests.vb

@@ -134,7 +134,7 @@ End Class
         <Trait(Traits.Feature, Traits.Features.Interactive)>
         Public Sub EncapsulateFieldCommandDisabledInSubmission()
             Dim exportProvider = ExportProviderCache _
-                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveTextBufferSupportsFeatureService))) _
+                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService))) _
                 .CreateExportProvider()
 
             Using workspace = TestWorkspace.Create(

src/EditorFeatures/VisualBasicTest/ExtractInterface/ExtractInterfaceTests.vb

@@ -1270,7 +1270,7 @@ End Namespace
         <Trait(Traits.Feature, Traits.Features.Interactive)>
         Public Sub TestExtractInterfaceCommandDisabledInSubmission()
             Dim exportProvider = ExportProviderCache _
-                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveTextBufferSupportsFeatureService))) _
+                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService))) _
                 .CreateExportProvider()
 
             Using workspace = TestWorkspace.Create(

src/EditorFeatures/VisualBasicTest/ExtractMethod/ExtractMethodTests.LanguageInteraction.vb

@@ -3367,7 +3367,7 @@ End Namespace"
             <Trait(Traits.Feature, Traits.Features.Interactive)>
             Public Sub TestExtractMethodCommandDisabledInSubmission()
                 Dim exportProvider = ExportProviderCache _
-                    .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveTextBufferSupportsFeatureService))) _
+                    .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService))) _
                     .CreateExportProvider()
 
                 Using workspace = TestWorkspace.Create(

src/EditorFeatures/VisualBasicTest/Organizing/OrganizeTypeDeclarationTests.vb

@@ -939,7 +939,7 @@ End Namespace</element>
         <Trait(Traits.Feature, Traits.Features.Interactive)>
         Public Sub TestOrganizingCommandsDisabledInSubmission()
             Dim exportProvider = ExportProviderCache _
-                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveTextBufferSupportsFeatureService))) _
+                .GetOrCreateExportProviderFactory(TestExportProvider.EntireAssemblyCatalogWithCSharpAndVisualBasic.WithParts(GetType(InteractiveSupportsFeatureService.InteractiveTextBufferSupportsFeatureService))) _
                 .CreateExportProvider()
 
             Using workspace = TestWorkspace.Create(

src/Features/Core/Portable/Completion/Providers/AbstractTypeImportCompletionProvider.cs

@@ -8,12 +8,14 @@
 using System.Threading;
 using System.Threading.Tasks;
 using Microsoft.CodeAnalysis.AddImports;
+using Microsoft.CodeAnalysis.Debugging;
 using Microsoft.CodeAnalysis.Editing;
 using Microsoft.CodeAnalysis.Experiments;
 using Microsoft.CodeAnalysis.Formatting;
 using Microsoft.CodeAnalysis.Internal.Log;
 using Microsoft.CodeAnalysis.LanguageServices;
 using Microsoft.CodeAnalysis.PooledObjects;
+using Microsoft.CodeAnalysis.Shared;
 using Microsoft.CodeAnalysis.Shared.Extensions;
 using Microsoft.CodeAnalysis.Shared.Extensions.ContextQuery;
 using Microsoft.CodeAnalysis.Text;
@@ -179,7 +181,14 @@ internal override async Task<CompletionChange> GetChangeAsync(Document document,
             var containingNamespace = TypeImportCompletionItem.GetContainingNamespace(completionItem);
             Debug.Assert(containingNamespace != null);
 
-            if (document.Project.Solution.Workspace.CanApplyChange(ApplyChangesKind.ChangeDocument))
+            if (ShouldCompleteWithFullyQualifyTypeName(document))
+            {
+                var fullyQualifiedName = $"{containingNamespace}.{completionItem.DisplayText}";
+                var change = new TextChange(completionListSpan, fullyQualifiedName);
+
+                return CompletionChange.Create(change);
+            }
+            else
             {
                 // Find context node so we can use it to decide where to insert using/imports.
                 var tree = await document.GetSyntaxTreeAsync(cancellationToken).ConfigureAwait(false);
@@ -223,14 +232,32 @@ internal override async Task<CompletionChange> GetChangeAsync(Document document,
 
                 return CompletionChange.Create(Utilities.Collapse(newText, builder.ToImmutableAndFree()));
             }
-            else
+
+            static bool ShouldCompleteWithFullyQualifyTypeName(Document document)
             {
-                // For workspace that doesn't support document change, e.g. DebuggerIntellisense
-                // we complete the type name in its fully qualified form instead.
-                var fullyQualifiedName = $"{containingNamespace}.{completionItem.DisplayText}";
-                var change = new TextChange(completionListSpan, fullyQualifiedName);
+                var workspace = document.Project.Solution.Workspace;
 
-                return CompletionChange.Create(change);
+                // Certain types of workspace don't support document change, e.g. DebuggerIntellisense
+                if (!workspace.CanApplyChange(ApplyChangesKind.ChangeDocument))
+                {
+                    return true;
+                }
+
+                // During an EnC session, adding import is not supported.
+                var encService = workspace.Services.GetService<IDebuggingWorkspaceService>()?.EditAndContinueServiceOpt;
+                if (encService?.EditSession != null)
+                {
+                    return true;
+                }
+
+                // Certain documents, e.g. Razor document, don't support adding imports
+                var documentSupportsFeatureService = workspace.Services.GetService<IDocumentSupportsFeatureService>();
+                if (!documentSupportsFeatureService.SupportsRefactorings(document))
+                {
+                    return true;
+                }
+
+                return false;
             }
         }
 

src/Features/Core/Portable/MoveToNamespace/MoveToNamespaceResult.cs

@@ -18,7 +18,7 @@ internal class MoveToNamespaceResult
 
         public MoveToNamespaceResult(
             Solution originalSolution,
-            Solution updatedSolution, 
+            Solution updatedSolution,
             DocumentId updatedDocumentId,
             ImmutableDictionary<string, ISymbol> newNameOriginalSymbolMapping)
         {

src/Features/Core/Portable/Shared/IDocumentSupportsFeatureService.cs

@@ -0,0 +1,33 @@
+// Copyright (c) Microsoft.  All Rights Reserved.  Licensed under the Apache License, Version 2.0.  See License.txt in the project root for license information.
+
+using System.Composition;
+using Microsoft.CodeAnalysis.Host;
+using Microsoft.CodeAnalysis.Host.Mef;
+
+namespace Microsoft.CodeAnalysis.Shared
+{
+    internal interface IDocumentSupportsFeatureService : IWorkspaceService
+    {
+        bool SupportsCodeFixes(Document document);
+        bool SupportsRefactorings(Document document);
+        bool SupportsRename(Document document);
+        bool SupportsNavigationToAnyPosition(Document document);
+    }
+
+
+    [ExportWorkspaceService(typeof(IDocumentSupportsFeatureService), ServiceLayer.Default), Shared]
+    internal class DefaultDocumentSupportsFeatureService : IDocumentSupportsFeatureService
+    {
+        public bool SupportsCodeFixes(Document document)
+            => true;
+
+        public bool SupportsNavigationToAnyPosition(Document document)
+            => true;
+
+        public bool SupportsRefactorings(Document document)
+            => true;
+
+        public bool SupportsRename(Document document)
+            => true;
+    }
+}