Make SharedMemoryDataConsumerHandle thread-safe.

content/child/shared_memory_data_consumer_handle.h

 // 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.
 
 #ifndef CONTENT_CHILD_SHARED_MEMORY_DATA_CONSUMER_HANDLE_H_
 #define CONTENT_CHILD_SHARED_MEMORY_DATA_CONSUMER_HANDLE_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "content/common/content_export.h"
 #include "content/public/child/request_peer.h"
 #include "third_party/WebKit/public/platform/WebDataConsumerHandle.h"
 
 namespace content {
 
 // This class is a WebDataConsumerHandle that accepts RequestPeer::ReceivedData.
-// TODO(yhirano): Currently this only works on a single thread. Make this class
-// work with a multi thread environment.
 class CONTENT_EXPORT SharedMemoryDataConsumerHandle final
     : public NON_EXPORTED_BASE(blink::WebDataConsumerHandle) {
  private:
   class Context;
 
  public:
   enum BackpressureMode {
     kApplyBackpressure,
     kDoNotApplyBackpressure,
   };
@@ skipping 13 matching lines @@
   };
 
   SharedMemoryDataConsumerHandle(BackpressureMode mode,
                                  scoped_ptr<Writer>* writer);
   virtual ~SharedMemoryDataConsumerHandle();
 
   virtual Result read(void* data, size_t size, Flags flags, size_t* readSize);
   virtual Result beginRead(const void** buffer, Flags flags, size_t* available);
   virtual Result endRead(size_t readSize);
   virtual void registerClient(Client* client);
   virtual void unregisterClient();

[tyoshino (SeeGerritForStatus), 2015/05/28 11:03:58]

registerClient/unregisterClient should happen in the same thread. Right?

[yhirano, 2015/05/28 11:24:41]

Yes, as written in WebDataConsumerHandle. Is there any problem?

 
  private:
   scoped_refptr<Context> context_;
 
   DISALLOW_COPY_AND_ASSIGN(SharedMemoryDataConsumerHandle);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_CHILD_SHARED_MEMORY_DATA_CONSUMER_HANDLE_H_