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

content/browser/loader/upload_data_stream_builder_unittest.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/browser/loader/upload_data_stream_builder.h"
 
 #include <algorithm>
 
 #include "base/file_util.h"
 #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"
 #include "net/base/upload_data_stream.h"
 #include "net/base/upload_file_element_reader.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 #include "webkit/browser/blob/blob_storage_context.h"
 
 using webkit_blob::BlobData;
@@ skipping 227 matching lines @@
                                      base::MessageLoopProxy::current().get());
   ASSERT_EQ(8U, upload->element_readers().size());
   EXPECT_TRUE(AreElementsEqual(*upload->element_readers()[0], blob_element1));
   EXPECT_TRUE(AreElementsEqual(*upload->element_readers()[1], blob_element2));
   EXPECT_TRUE(AreElementsEqual(*upload->element_readers()[2], upload_element1));
   EXPECT_TRUE(AreElementsEqual(*upload->element_readers()[3], blob_element1));
   EXPECT_TRUE(AreElementsEqual(*upload->element_readers()[4], blob_element2));
   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