Stop using SpawnedTestServer in DownloadContentTest.*

content/browser/download/download_manager_impl.cc

Index: content/browser/download/download_manager_impl.cc
diff --git a/content/browser/download/download_manager_impl.cc b/content/browser/download/download_manager_impl.cc
index 7562c0f3619b30e5e4e6ea0aa4dfc9c50570710f..8ec699359f8d31b233600473b4068587ea103e13 100644
--- a/content/browser/download/download_manager_impl.cc
+++ b/content/browser/download/download_manager_impl.cc
@@ -101,7 +101,7 @@ void BeginDownload(scoped_ptr<DownloadUrlParameters> params,
                                            true);
     }
     if (has_etag) {
-      request->SetExtraRequestHeaderByName("If-Match", params->etag(), true);
+      request->SetExtraRequestHeaderByName("If-Range", params->etag(), true);

[Randy Smith (Not in Mondays), 2015/11/12 00:57:57]

a-r nit: Should this be in this CL?  It looks like a real bug if this needed to
be changed.

[asanka, 2015/11/13 21:40:01]

I'll make another CL and make this CL depend on that for the change from
If-Match to If-Range. This isn't a correctness bug, but rather an efficiency
improvement.

If-Match results in a compliant server sending an HTTP 412 in the case of an
ETag mismatch. Then the downloads logic needs to send an unconditionalized
request to fetch the entire resource.

If-Range, on the other hand, will respond with a 200 along with the entire
entity if there's an ETag mismatch. It's just one request shorter.

     }
   }