Better workaround for extension browser test problems on win debug bots

chrome/browser/extensions/extension_apitest.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.
 
 #include "chrome/browser/extensions/extension_apitest.h"
 
 #include "base/string_util.h"
 #include "base/stringprintf.h"
 #include "chrome/browser/extensions/api/test/test_api.h"
 #include "chrome/browser/extensions/extension_service.h"
@@ skipping 148 matching lines @@
                                            const std::string& page_url) {
   return RunExtensionSubtest(extension_name, page_url, kFlagEnableFileAccess);
 }
 
 bool ExtensionApiTest::RunExtensionSubtest(const char* extension_name,
                                            const std::string& page_url,
                                            int flags) {
   DCHECK(!page_url.empty()) << "Argument page_url is required.";
   // See http://crbug.com/177163 for details.
 #if defined(OS_WIN) && !defined(NDEBUG)
-  LOG(WARNING) << "Workaround for 177163, prematurely stopping test";
-  // Following is necessary for sharding scripts to think the test passed.
-  fprintf(stderr, "[       OK ] X (1000ms total)\r\n");
-  fflush(stderr);
-  ExitProcess(0);
+  LOG(WARNING) << "Workaround for 177163, prematurely returning";
+  return true;
 #endif
   return RunExtensionTestImpl(extension_name, page_url, flags);
 }
 
 
 bool ExtensionApiTest::RunPageTest(const std::string& page_url) {
   return RunExtensionSubtest("", page_url);
 }
 
 bool ExtensionApiTest::RunPageTest(const std::string& page_url,
@@ skipping 136 matching lines @@
   test_config_->SetInteger(kTestWebSocketPort,
                            websocket_server_->host_port_pair().port());
 
   return true;
 }
 
 void ExtensionApiTest::SetUpCommandLine(CommandLine* command_line) {
   ExtensionBrowserTest::SetUpCommandLine(command_line);
   test_data_dir_ = test_data_dir_.AppendASCII("api_test");
 }