More aggressive IO asserts in content_browsertests.

content/browser/tracing/tracing_controller_browsertest.cc

 // Copyright 2013 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/public/browser/tracing_controller.h"
 
 #include <stdint.h>
 #include <utility>
 
 #include "base/files/file_util.h"
 #include "base/memory/ref_counted_memory.h"
 #include "base/run_loop.h"
 #include "base/strings/pattern.h"
+#include "base/threading/thread_restrictions.h"
 #include "build/build_config.h"
 #include "content/browser/tracing/tracing_controller_impl.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/trace_uploader.h"
 #include "content/public/test/browser_test_utils.h"
 #include "content/public/test/content_browser_test.h"
 #include "content/public/test/content_browser_test_utils.h"
 #include "content/shell/browser/shell.h"
 #include "content/test/test_content_browser_client.h"
 
@@ skipping 121 matching lines @@
     disable_recording_done_callback_count_++;
     last_metadata_.reset(metadata.release());
     last_data_ = data->data();
     EXPECT_TRUE(data->size() > 0);
     quit_callback.Run();
   }
 
   void StopTracingFileDoneCallbackTest(base::Closure quit_callback,
                                             const base::FilePath& file_path) {
     disable_recording_done_callback_count_++;
-    EXPECT_TRUE(PathExists(file_path));
-    int64_t file_size;
-    base::GetFileSize(file_path, &file_size);
-    EXPECT_TRUE(file_size > 0);
+    {
+      base::ThreadRestrictions::ScopedAllowIO allow_io_for_test_verifications;
+      EXPECT_TRUE(PathExists(file_path));
+      int64_t file_size;
+      base::GetFileSize(file_path, &file_size);
+      EXPECT_GT(file_size, 0);
+    }
     quit_callback.Run();
     last_actual_recording_file_path_ = file_path;
   }
 
     int get_categories_done_callback_count() const {
     return get_categories_done_callback_count_;
   }
 
   int enable_recording_done_callback_count() const {
     return enable_recording_done_callback_count_;
@@ skipping 243 matching lines @@
   EXPECT_TRUE(last_data().find("os-name") != std::string::npos);
   EXPECT_TRUE(last_data().find("user-agent") != std::string::npos);
 
   EXPECT_TRUE(last_data().find("not-whitelisted") != std::string::npos);
   EXPECT_TRUE(last_data().find("this_not_found") == std::string::npos);
 }
 
 IN_PROC_BROWSER_TEST_F(TracingControllerTest,
                        EnableAndStopTracingWithFilePath) {
   base::FilePath file_path;
-  base::CreateTemporaryFile(&file_path);
+  {
+    base::ThreadRestrictions::ScopedAllowIO allow_io_for_creating_test_file;
+    base::CreateTemporaryFile(&file_path);
+  }
   TestStartAndStopTracingFile(file_path);
   EXPECT_EQ(file_path.value(), last_actual_recording_file_path().value());
 }
 
 IN_PROC_BROWSER_TEST_F(TracingControllerTest,
                        EnableAndStopTracingWithCompression) {
   TestStartAndStopTracingCompressed();
 }
 
 IN_PROC_BROWSER_TEST_F(TracingControllerTest,
                        EnableAndStopTracingWithEmptyFileAndNullCallback) {
   Navigate(shell());
 
   TracingController* controller = TracingController::GetInstance();
   EXPECT_TRUE(controller->StartTracing(
       TraceConfig(),
       TracingController::StartTracingDoneCallback()));
   EXPECT_TRUE(controller->StopTracing(NULL));
   base::RunLoop().RunUntilIdle();
 }
 
 }  // namespace content