[Downloads] Fix copy and move operators for TestDownloadRequestHandler.

content/browser/download/download_browsertest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file contains download browser tests that are known to be runnable
 // in a pure content context.  Over time tests should be migrated here.
 
 #include <stddef.h>
 #include <stdint.h>
 #include <utility>
@@ skipping 1053 matching lines @@
   EXPECT_EQ(parameters.etag, value);
 
   ASSERT_TRUE(requests[1].request_headers.GetHeader(
       net::HttpRequestHeaders::kRange, &value));
   EXPECT_EQ(base::StringPrintf("bytes=%" PRId64 "-", interruption.offset),
             value);
 }
 
 // Resumption should only attempt to contact the final URL if the download has a
 // URL chain.
-// Fails on Win x64 GN (dbg).  https://crbug.com/598360
-#if defined(OS_WIN)
-#define MAYBE_RedirectBeforeResume DISABLED_RedirectBeforeResume
-#else
-#define MAYBE_RedirectBeforeResume RedirectBeforeResume
-#endif
-IN_PROC_BROWSER_TEST_F(DownloadContentTest, MAYBE_RedirectBeforeResume) {
+IN_PROC_BROWSER_TEST_F(DownloadContentTest, RedirectBeforeResume) {
   TestDownloadRequestHandler request_handler_1(
       GURL("http://example.com/first-url"));
   request_handler_1.StartServingStaticResponse(
       "HTTP/1.1 302 Redirect\r\n"
       "Location: http://example.com/second-url\r\n"
       "\r\n");
 
   TestDownloadRequestHandler request_handler_2(
       GURL("http://example.com/second-url"));
   request_handler_2.StartServingStaticResponse(
@@ skipping 33 matching lines @@
   download->Resume();
   WaitForCompletion(download);
 
   ASSERT_NO_FATAL_FAILURE(ReadAndVerifyFileContents(
       parameters.pattern_generator_seed, parameters.size,
       download->GetTargetFilePath()));
 }
 
 // If a resumption request results in a redirect, the response should be ignored
 // and the download should be marked as interrupted again.
-// Fails on Win x64 GN (dbg).  https://crbug.com/598360
-#if defined(OS_WIN)
-#define MAYBE_RedirectWhileResume DISABLED_RedirectWhileResume
-#else
-#define MAYBE_RedirectWhileResume RedirectWhileResume
-#endif
-IN_PROC_BROWSER_TEST_F(DownloadContentTest, MAYBE_RedirectWhileResume) {
+IN_PROC_BROWSER_TEST_F(DownloadContentTest, RedirectWhileResume) {
   TestDownloadRequestHandler request_handler(
       GURL("http://example.com/first-url"));
   TestDownloadRequestHandler::Parameters parameters =
       TestDownloadRequestHandler::Parameters::WithSingleInterruption();
   ++parameters.pattern_generator_seed;
   request_handler.StartServing(parameters);
 
   // We should never send a request to the decoy. If we do, the request will
   // always succeed, which results in behavior that diverges from what we want,
   // which is for the download to return to being interrupted.
@@ skipping 41 matching lines @@
   // redirect response shows up as a response with 0 bytes transferred.
   EXPECT_GT(parameters.size, requests[0].transferred_byte_count);
   EXPECT_EQ(0, requests[1].transferred_byte_count);
   EXPECT_GT(parameters.size, requests[2].transferred_byte_count);
 }
 
 // If the server response for the resumption request specifies a bad range (i.e.
 // not the range that was requested or an invalid or missing Content-Range
 // header), then the download should be marked as interrupted again without
 // discarding the partial state.
-// Fails on Win x64 GN (dbg).  https://crbug.com/598360
-#if defined(OS_WIN)
-#define MAYBE_BadRangeHeader DISABLED_BadRangeHeader
-#else
-#define MAYBE_BadRangeHeader BadRangeHeader
-#endif
-IN_PROC_BROWSER_TEST_F(DownloadContentTest, MAYBE_BadRangeHeader) {
+IN_PROC_BROWSER_TEST_F(DownloadContentTest, BadRangeHeader) {
   TestDownloadRequestHandler request_handler;
   TestDownloadRequestHandler::Parameters parameters =
       TestDownloadRequestHandler::Parameters::WithSingleInterruption();
   request_handler.StartServing(parameters);
 
   DownloadItem* download =
       StartDownloadAndReturnItem(shell(), request_handler.url());
   WaitForInterrupt(download);
 
   // Upon resumption, the server starts responding with a bad range header.
@@ skipping 1253 matching lines @@
 
   std::vector<DownloadItem*> downloads;
   DownloadManagerForShell(shell())->GetAllDownloads(&downloads);
   ASSERT_EQ(1u, downloads.size());
 
   EXPECT_EQ(FILE_PATH_LITERAL("Jumboshrimp.txt"),
             downloads[0]->GetTargetFilePath().BaseName().value());
 }
 
 }  // namespace content