Allow BlobDataHandles to be copied, and have their UUIDs read, on any thread.

content/browser/loader/upload_data_stream_builder_unittest.cc

Index: content/browser/loader/upload_data_stream_builder_unittest.cc
diff --git a/content/browser/loader/upload_data_stream_builder_unittest.cc b/content/browser/loader/upload_data_stream_builder_unittest.cc
index dba3dff4f0f234c4c6af59bfd984f3b075d614b5..f28d9b8c8ebf1e24c6a68841231eb8009b408c66 100644
--- a/content/browser/loader/upload_data_stream_builder_unittest.cc
+++ b/content/browser/loader/upload_data_stream_builder_unittest.cc
@@ -10,6 +10,7 @@
 #include "base/files/file_path.h"
 #include "base/message_loop/message_loop.h"
 #include "base/message_loop/message_loop_proxy.h"
+#include "base/run_loop.h"
 #include "base/time/time.h"
 #include "content/common/resource_request_body.h"
 #include "net/base/upload_bytes_element_reader.h"
@@ -257,6 +258,12 @@ TEST(UploadDataStreamBuilderTest, ResolveBlobAndCreateUploadDataStream) {
   EXPECT_TRUE(AreElementsEqual(*upload->element_readers()[5], blob_element1));
   EXPECT_TRUE(AreElementsEqual(*upload->element_readers()[6], blob_element2));
   EXPECT_TRUE(AreElementsEqual(*upload->element_readers()[7], upload_element2));
+
+  // Clean up for ASAN.
+  handle1.reset();
+  handle2.reset();

[michaeln, 2014/05/01 18:50:48]

bah... also need to reset request_body and maybe upload, might be cleaner and
more maintainable to push the majority of the method body down a scope...

void ResolveBlobAndCreateUploadDataStream() {
  base::MessageLoop message_loop;
  {
    stuff
  }
  base::RunLoop().RunUntilIdle();
}

+  base::RunLoop run_loop;
+  run_loop.RunUntilIdle();

[michaeln, 2014/05/01 18:50:48]

i've seen this more consisely written as...
base::RunLoop().RunUntilIdle();
...in other tests.

 }
 
 }  // namespace content