Change BuildNamespace extension to target CodeRenderingContext

Because `CodeRenderingContext` owns the `CodeWriter` instance used for code gen, and the `BuildNamespace` extension method targets `CodeWriter` and also takes a `CodeRenderingContext`, we can change it  can target `CodeRenderingContext`.

Author: DustinCampbell

src/Compiler/Microsoft.CodeAnalysis.Razor.Compiler/src/Language/CodeGeneration/CodeRenderingContextExtensions.cs

@@ -0,0 +1,36 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+
+using System;
+using static Microsoft.AspNetCore.Razor.Language.CodeGeneration.CodeWriterExtensions;
+
+namespace Microsoft.AspNetCore.Razor.Language.CodeGeneration;
+
+internal static class CodeRenderingContextExtensions
+{
+    public static CSharpCodeWritingScope BuildNamespace(this CodeRenderingContext context, string? name, SourceSpan? span)
+    {
+        var writer = context.CodeWriter;
+
+        if (name.IsNullOrEmpty())
+        {
+            return new CSharpCodeWritingScope(writer, writeBraces: false);
+        }
+
+        writer.Write("namespace ");
+
+        if (context.Options.DesignTime || span is null)
+        {
+            writer.WriteLine(name);
+        }
+        else
+        {
+            writer.WriteLine();
+            using (writer.BuildEnhancedLinePragma(span, context))
+            {
+                writer.WriteLine(name);
+            }
+        }
+        return new CSharpCodeWritingScope(writer);
+    }
+}

src/Compiler/Microsoft.CodeAnalysis.Razor.Compiler/src/Language/CodeGeneration/CodeWriterExtensions.cs

@@ -474,31 +474,6 @@ private static CSharpCodeWritingScope BuildLambda(CodeWriter writer, bool async,
         return scope;
     }
 
-#nullable enable
-    public static CSharpCodeWritingScope BuildNamespace(this CodeWriter writer, string? name, SourceSpan? span, CodeRenderingContext context)
-    {
-        if (name.IsNullOrEmpty())
-        {
-            return new CSharpCodeWritingScope(writer, writeBraces: false);
-        }
-
-        writer.Write("namespace ");
-        if (context.Options.DesignTime || span is null)
-        {
-            writer.WriteLine(name);
-        }
-        else
-        {
-            writer.WriteLine();
-            using (writer.BuildEnhancedLinePragma(span, context))
-            {
-                writer.WriteLine(name);
-            }
-        }
-        return new CSharpCodeWritingScope(writer);
-    }
-#nullable disable
-
     public static CSharpCodeWritingScope BuildClassDeclaration(
         this CodeWriter writer,
         ImmutableArray<string> modifiers,

src/Compiler/Microsoft.CodeAnalysis.Razor.Compiler/src/Language/CodeGeneration/DefaultDocumentWriter.cs

@@ -109,21 +109,21 @@ public override void VisitUsingDirective(UsingDirectiveIntermediateNode node)
 
         public override void VisitNamespaceDeclaration(NamespaceDeclarationIntermediateNode node)
         {
-            var codeWriter = CodeWriter;
-
-            using (codeWriter.BuildNamespace(node.Content, node.Source, _context))
+            using (_context.BuildNamespace(node.Content, node.Source))
             {
+                var writer = CodeWriter;
+
                 if (node.Children.OfType<UsingDirectiveIntermediateNode>().Any())
                 {
                     // Tooling needs at least one line directive before using directives, otherwise Roslyn will
                     // not offer to create a new one. The last using in the group will output a hidden line
                     // directive after itself.
-                    codeWriter.WriteLine("#line default");
+                    writer.WriteLine("#line default");
                 }
                 else
                 {
                     // If there are no using directives, we output the hidden directive here.
-                    codeWriter.WriteLine("#line hidden");
+                    writer.WriteLine("#line hidden");
                 }
 
                 VisitDefault(node);