chromium_build_stats: remove should-upload handler

This is not used now.

Bug: 1286201
Change-Id: I6f73e4a95e9c9269977cfda232d39bce3395dd7e
Reviewed-on: https://chromium-review.googlesource.com/c/infra/infra/+/3414809
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Auto-Submit: Takuto Ikuta <tikuta@chromium.org>
Reviewed-by: Fumitoshi Ukai <ukai@google.com>
Commit-Queue: Fumitoshi Ukai <ukai@google.com>
Cr-Commit-Position: refs/heads/main@{#46592}
diff --git a/go/src/infra/appengine/chromium_build_stats/app/app.go b/go/src/infra/appengine/chromium_build_stats/app/app.go
index 27243bd..553b22d 100644
--- a/go/src/infra/appengine/chromium_build_stats/app/app.go
+++ b/go/src/infra/appengine/chromium_build_stats/app/app.go
@@ -37,31 +37,9 @@
 var ninjaLogRE = regexp.MustCompile(`gs://chrome-goma-log/.*/ninja_log.*\.gz`)
 
 func init() {
-	// This is handler to check whether ninja log can be uploaded or not from
-	// depot_tools user.
-	http.HandleFunc("/should-upload", shouldUploadHandler)
-
 	http.HandleFunc("/", handler)
 }
 
-// This code is copied from https://chromium.googlesource.com/infra/infra/+/b74c3b27b1fb8f60c969e9f158c86d49b56f3d4b/go/src/infra/appengine/depot_tools_metrics/metrics/app.go#52
-// shouldUploadHandler handles the '/should-upload' endpoint, which is used by
-// depot_tools to check whether it should collect and upload metrics.
-// It returns 200 if the request in coming from a corp machine, and thus a
-// Googler, and 403 otherwise.
-func shouldUploadHandler(w http.ResponseWriter, r *http.Request) {
-	if r.URL.Path != "/should-upload" {
-		http.NotFound(w, r)
-		return
-	}
-	// TRUSTED_IP_REQUEST=1 means the request is coming from a corp machine.
-	if r.Header.Get("X-AppEngine-Trusted-IP-Request") != "1" {
-		http.Error(w, "Access Denied: You're not on corp.", http.StatusForbidden)
-		return
-	}
-	fmt.Fprintf(w, "Success")
-}
-
 func handler(w http.ResponseWriter, req *http.Request) {
 	loc := req.FormValue("loc")
 	if loc != "" {