Merged PR 473246: Always open documents in the Razor LSP

In my previous PR, that added the method I'm removing here, I tried to be a little fancy and not bother opening the Razor document because if we were getting an LSP request for a document, then it must already be open. I completely forgot that this isn't true for the IDynamicFile LSP request.

If a document is open, and you call `openTextDocument`, vscode just returns immediately anyway so there really is no harm in calling this.

Related work items: #1822437

Author: davidwengier

src/razor/src/Document/RazorDocumentManager.ts

@@ -50,7 +50,7 @@ export class RazorDocumentManager implements IRazorDocumentManager {
         // succeed. Without this, even a simple diagnostics request will fail in Roslyn if the user just opens a .razor document
         // and leaves it open past the timeout.
         if (this.razorDocumentGenerationInitialized) {
-            await this.ensureProjectedDocumentsOpen(document);
+            await this.ensureDocumentAndProjectedDocumentsOpen(document);
         }
 
         return document;
@@ -306,10 +306,6 @@ export class RazorDocumentManager implements IRazorDocumentManager {
             await vscode.workspace.openTextDocument(razorUri);
         }
 
-        await this.ensureProjectedDocumentsOpen(document);
-    }
-
-    private async ensureProjectedDocumentsOpen(document: IRazorDocument) {
         await vscode.workspace.openTextDocument(document.csharpDocument.uri);
         await vscode.workspace.openTextDocument(document.htmlDocument.uri);
     }