code review 4535056: http: remove finalURL from Client.Get; move to Response

src/pkg/http/request_test.go

 // Copyright 2009 The Go Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style
 // license that can be found in the LICENSE file.
 
 package http_test
 
 import (
         "bytes"
         "fmt"
         . "http"
@@ skipping 144 matching lines @@
                         w.WriteHeader(StatusSeeOther)
                 case "/foo/":
                         fmt.Fprintf(w, "foo")
                 default:
                         w.WriteHeader(StatusBadRequest)
                 }
         }))
         defer ts.Close()
 
         var end = regexp.MustCompile("/foo/$")
-        r, url, err := Get(ts.URL)
+        r, err := Get(ts.URL)
         if err != nil {
                 t.Fatal(err)
         }
         r.Body.Close()
+        url := r.Request.URL.String()
         if r.StatusCode != 200 || !end.MatchString(url) {
                 t.Fatalf("Get got status %d at %q, want 200 matching /foo/$", r.StatusCode, url)
         }
 }
 
 func TestMultipartRequest(t *testing.T) {
         // Test that we can read the values and files of a·
         // multipart request with FormValue and FormFile,
         // and that ParseMultipartForm can be called multiple times.
         req := newTestMultipartRequest(t)
@@ skipping 125 matching lines @@
 --MyBoundary
 Content-Disposition: form-data; name="texta"
 
 ` + textaValue + `
 --MyBoundary
 Content-Disposition: form-data; name="textb"
 
 ` + textbValue + `
 --MyBoundary--
 `