Easier upload progress browser test for Mac buildbots

content/browser/loader/async_resource_handler_browsertest.cc

 // Copyright 2015 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.
 
 #include "content/browser/loader/async_resource_handler.h"
 
 #include <string>
 
 #include "base/format_macros.h"
 #include "base/strings/string_util.h"
@@ skipping 10 matching lines @@
 
 namespace content {
 
 namespace {
 
 const char kPostPath[] = "/post";
 const char kRedirectPostPath[] = "/redirect";
 
 // ThreadSanitizer is too slow to perform the full upload, so tests
 // using that build get an easier test which might not show two distinct
 // progress events. See crbug.com/526985.

[mmenke, 2015/10/30 15:35:37]

Add comment about OSX being slow, too, and a bug link.

-#if defined(THREAD_SANITIZER)
+#if defined(THREAD_SANITIZER) || defined(OS_MACOSX)
 const size_t kPayloadSize = 1062882;  // 2*3^12
 #else
 const size_t kPayloadSize = 28697814;  // 2*3^15
 #endif
 
 scoped_ptr<net::test_server::HttpResponse> HandlePostAndRedirectURLs(
   const std::string& request_path,
   const net::test_server::HttpRequest& request) {
   scoped_ptr<net::test_server::BasicHttpResponse> http_response(
       new net::test_server::BasicHttpResponse());
@@ skipping 52 matching lines @@
   EXPECT_TRUE(ExecuteScriptAndExtractString(
       shell()->web_contents(),
       base::StringPrintf("WaitForAsyncXHR('%s', %" PRIuS ")",
                          kRedirectPostPath,
                          kPayloadSize),
       &js_result));
   EXPECT_EQ(js_result, "success");
 }
 
 }  // namespace content