Revert "net/http/internal/http2: prevent alloc when writing status code for responses"

This reverts commit e67d773.

Reason for revert: CL 769100 reverts CL 762040. Therefore, CL 762140
needs to be reverted too.

For #78833

Change-Id: I5f836e0c0950a596c88a5d53d6d705656a6a6964
Reviewed-on: https://go-review.googlesource.com/c/go/+/769201
LUCI-TryBot-Result: golang-scoped@luci-project-accounts.iam.gserviceaccount.com <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Nicholas Husin <husin@google.com>
Reviewed-by: Damien Neil <dneil@google.com>

Author: nicholashusin

src/net/http/internal/http2/http2.go

@@ -21,6 +21,7 @@ import (
 	"net"
 	"os"
 	"slices"
+	"strconv"
 	"strings"
 	"sync"
 	"time"
@@ -214,6 +215,17 @@ func validWireHeaderFieldName(v string) bool {
 	return true
 }
 
+// TODO: avoid alloc when code is neither 200 nor 404.
+func httpCodeString(code int) string {
+	switch code {
+	case 200:
+		return "200"
+	case 404:
+		return "404"
+	}
+	return strconv.Itoa(code)
+}
+
 // A closeWaiter is like a sync.WaitGroup but only goes 1 to 0 (open to closed).
 type closeWaiter chan struct{}
 

src/net/http/internal/http2/write.go

@@ -10,8 +10,6 @@ import (
 	"log"
 	"net/http/internal/httpcommon"
 	"net/url"
-	"strconv"
-	"strings"
 
 	"golang.org/x/net/http/httpguts"
 	"golang.org/x/net/http2/hpack"
@@ -202,7 +200,7 @@ type writeResHeaders struct {
 
 func encKV(enc *hpack.Encoder, k, v string) {
 	if VerboseLogs {
-		log.Printf("http2: server encoding header %q = %q", strings.Clone(k), strings.Clone(v))
+		log.Printf("http2: server encoding header %q = %q", k, v)
 	}
 	enc.WriteField(hpack.HeaderField{Name: k, Value: v})
 }
@@ -223,8 +221,7 @@ func (w *writeResHeaders) writeFrame(ctx writeContext) error {
 	buf.Reset()
 
 	if w.httpResCode != 0 {
-		codeBuf := strconv.AppendInt(make([]byte, 0, 3), int64(w.httpResCode), 10)
-		encKV(enc, ":status", string(codeBuf))
+		encKV(enc, ":status", httpCodeString(w.httpResCode))
 	}
 
 	encodeHeaders(enc, w.h, w.trailers)