cmd/coordinator: display LUCI build results on build dashboards (part 2)

This makes the new view on by default. As before, it remains
possible to opt-out via a 'legacyonly=1' URL query parameter.

For golang/go#65913.

Change-Id: I52b0a7e206a1d8bdf4645d4e1f91cadebb446c0e
Reviewed-on: https://go-review.googlesource.com/c/build/+/568195
Reviewed-by: Michael Knyszek <[email protected]>
Reviewed-by: Dmitri Shuralyov <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>
Auto-Submit: Dmitri Shuralyov <[email protected]>

Author: dmitshur

cmd/coordinator/internal/dashboard/handler.go

@@ -63,7 +63,7 @@ type Handler struct {
 }
 
 func (d *Handler) ServeHTTP(w http.ResponseWriter, req *http.Request) {
-	showLUCI, _ := strconv.ParseBool(req.URL.Query().Get("showluci"))
+	var showLUCI = true
 	if legacyOnly, _ := strconv.ParseBool(req.URL.Query().Get("legacyonly")); legacyOnly {
 		showLUCI = false
 	}

cmd/coordinator/internal/legacydash/ui.go

@@ -102,7 +102,7 @@ func viewForRequest(r *http.Request) (dashboardView, error) {
 	case "json":
 		return jsonView{}, nil
 	case "":
-		showLUCI, _ := strconv.ParseBool(r.URL.Query().Get("showluci"))
+		var showLUCI = true
 		if legacyOnly, _ := strconv.ParseBool(r.URL.Query().Get("legacyonly")); legacyOnly {
 			showLUCI = false
 		}