internal/frontend: remove two unnecessary type conversions

Found by unconvert.

Change-Id: Ia8d8f9f6d12ef53262782dadaf0c7f505fa8cb59
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/238045
Reviewed-by: Julie Qiu <julie@golang.org>
diff --git a/internal/frontend/doc_test.go b/internal/frontend/doc_test.go
index 6add15a..fde74fb 100644
--- a/internal/frontend/doc_test.go
+++ b/internal/frontend/doc_test.go
@@ -71,7 +71,7 @@
 	}
 
 	for _, test := range tests {
-		if got := string(hackUpDocumentation(test.body)); got != test.want {
+		if got := hackUpDocumentation(test.body); got != test.want {
 			t.Errorf("hackUpDocumentation(%s) = %s, want %s", test.body, got, test.want)
 		}
 	}
diff --git a/internal/frontend/overview.go b/internal/frontend/overview.go
index 16f047f..8dbe541 100644
--- a/internal/frontend/overview.go
+++ b/internal/frontend/overview.go
@@ -111,7 +111,7 @@
 		return ""
 	}
 	if !isMarkdown(readme.Filepath) {
-		return template.HTML(fmt.Sprintf(`<pre class="readme">%s</pre>`, html.EscapeString(string(readme.Contents))))
+		return template.HTML(fmt.Sprintf(`<pre class="readme">%s</pre>`, html.EscapeString(readme.Contents)))
 	}
 
 	// bluemonday.UGCPolicy allows a broad selection of HTML elements and