Fix integration tests in src/chrome and src/extensions so that we can turn on IO thread checks wi...

extensions/browser/api/printer_provider/printer_provider_apitest.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 <stddef.h>
 
 #include <utility>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/files/file.h"
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/json/json_string_value_serializer.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted_memory.h"
 #include "base/run_loop.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
+#include "base/threading/thread_restrictions.h"
 #include "device/usb/mock_usb_device.h"
 #include "device/usb/mock_usb_service.h"
 #include "extensions/browser/api/printer_provider/printer_provider_api.h"
 #include "extensions/browser/api/printer_provider/printer_provider_api_factory.h"
 #include "extensions/browser/api/printer_provider/printer_provider_print_job.h"
 #include "extensions/browser/api/usb/usb_guid_map.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/value_builder.h"
 #include "extensions/shell/test/shell_apitest.h"
@@ skipping 61 matching lines @@
 class PrinterProviderApiTest : public ShellApiTest {
  public:
   enum PrintRequestDataType {
     PRINT_REQUEST_DATA_TYPE_NOT_SET,
     PRINT_REQUEST_DATA_TYPE_FILE,
     PRINT_REQUEST_DATA_TYPE_FILE_DELETED,
     PRINT_REQUEST_DATA_TYPE_BYTES
   };
 
   PrinterProviderApiTest() {}
-  ~PrinterProviderApiTest() override {}
+  ~PrinterProviderApiTest() override {
+    base::ThreadRestrictions::ScopedAllowIO allow_io;
+    ignore_result(data_dir_.Delete());
+  }
 
   void StartGetPrintersRequest(
       const PrinterProviderAPI::GetPrintersCallback& callback) {
     PrinterProviderAPIFactory::GetInstance()
         ->GetForBrowserContext(browser_context())
         ->DispatchGetPrintersRequested(callback);
   }
 
   void StartGetUsbPrinterInfoRequest(
       const std::string& extension_id,
@@ skipping 118 matching lines @@
         base::Bind(&RecordPrintResultAndRunCallback, &success, &print_status,
                    run_loop.QuitClosure());
 
     switch (data_type) {
       case PRINT_REQUEST_DATA_TYPE_NOT_SET:
         StartPrintRequestWithNoData(extension_id, callback);
         break;
       case PRINT_REQUEST_DATA_TYPE_FILE:
         ASSERT_TRUE(StartPrintRequestUsingFileInfo(extension_id, callback));
         break;
-      case PRINT_REQUEST_DATA_TYPE_FILE_DELETED:
+      case PRINT_REQUEST_DATA_TYPE_FILE_DELETED: {
         ASSERT_TRUE(StartPrintRequestUsingFileInfo(extension_id, callback));
+        base::ThreadRestrictions::ScopedAllowIO allow_io;
         ASSERT_TRUE(data_dir_.Delete());
         break;
+      }
       case PRINT_REQUEST_DATA_TYPE_BYTES:
         StartPrintRequestUsingDocumentBytes(extension_id, callback);
         break;
     }
 
     if (data_type != PRINT_REQUEST_DATA_TYPE_NOT_SET)
       ASSERT_TRUE(catcher.GetNextResult()) << catcher.message();
 
     run_loop.Run();
     EXPECT_EQ(expected_result, print_status);
@@ skipping 87 matching lines @@
  protected:
   device::MockUsbService usb_service_;
 
  private:
   // Initializes |data_dir_| if needed and creates a file in it containing
   // provided data.
   bool CreateTempFileWithContents(const char* data,
                                   int size,
                                   base::FilePath* path,
                                   base::File::Info* file_info) {
+    base::ThreadRestrictions::ScopedAllowIO allow_io;
     if (!data_dir_.IsValid() && !data_dir_.CreateUniqueTempDir())
       return false;
 
     *path = data_dir_.GetPath().AppendASCII("data.pwg");
     int written = base::WriteFile(*path, data, size);
     if (written != size)
       return false;
     if (!base::GetFileInfo(*path, file_info))
       return false;
     return true;
@@ skipping 495 matching lines @@
   RunUsbPrinterInfoRequestTest("EMPTY_RESPONSE");
 }
 
 IN_PROC_BROWSER_TEST_F(PrinterProviderApiTest, GetUsbPrinterInfoNoListener) {
   RunUsbPrinterInfoRequestTest("NO_LISTENER");
 }
 
 }  // namespace
 
 }  // namespace extensions