Introduce CompositeDataConsumerHandle.

Source/modules/fetch/CompositeDataConsumerHandle.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 CompositeDataConsumerHandle_h
+#define CompositeDataConsumerHandle_h
+
+#include "public/platform/WebDataConsumerHandle.h"
+#include "wtf/OwnPtr.h"
+#include "wtf/PassOwnPtr.h"
+#include "wtf/RefPtr.h"
+
+namespace blink {
+
+// This is a utility class to construct a composite data consumer handle. It
+// owns a web data consumer handle and delegates methods. A user can update
+// the handle by using |update| method.
+class CompositeDataConsumerHandle final : public WebDataConsumerHandle {
+public:
+    // |handle| must not be locked.

[hiroshige, 2015/06/09 05:37:16]

Also |handle| must be non-null (according to the current impl)?
If so, please add comments and assertions.

[yhirano, 2015/06/09 06:30:52]

Done, though I'm not sure if we should always write whether we accept a null.

+    static PassOwnPtr<CompositeDataConsumerHandle> create(PassOwnPtr<WebDataConsumerHandle> handle)
+    {
+        return adoptPtr(new CompositeDataConsumerHandle(handle));
+    }
+    ~CompositeDataConsumerHandle() override;
+
+    // This function should be called on the thread on which this object
+    // was created. |handle| must not be locked.

[hiroshige, 2015/06/09 05:37:16]

ditto.

[yhirano, 2015/06/09 06:30:52]

Done.

+    void update(PassOwnPtr<WebDataConsumerHandle> /* handle */);
+
+    // Utility static methods each of which provides a "basic block" of
+    // a CompositeDataConsumerHandle. For example, a user can create
+    // a CompositeDataConsumerHandle with a "waiting" handle to provide
+    // WebDataConsumerHandle APIs while waiting for another handle. When that
+    // handle arrives, the user will use |update| to replace the "waiting"
+    // handle with the arrived one.
+    //
+    // Returns a handle that returns ShouldWait for read / beginRead / endRead

[hiroshige, 2015/06/09 05:37:16]

Actually this returns UnexpectedError for endRead.

[yhirano, 2015/06/09 06:30:52]

Done.

+    // operations.
+    static PassOwnPtr<WebDataConsumerHandle> createWaitingHandle();
+
+    // Returns a handle that returns Done for read / beginRead / endRead

[hiroshige, 2015/06/09 05:37:16]

ditto.

[yhirano, 2015/06/09 06:30:52]

Done.

+    // operations.
+    static PassOwnPtr<WebDataConsumerHandle> createDoneHandle();
+
+private:
+    class Context;
+    class ReaderImpl;
+    Reader* obtainReaderInternal(Client*) override;
+
+    explicit CompositeDataConsumerHandle(PassOwnPtr<WebDataConsumerHandle>);

[hiroshige, 2015/06/09 05:37:16]

The handle must be non-null?

[yhirano, 2015/06/09 06:30:52]

I don't want to write comments because this is a private function.

+
+    RefPtr<Context> m_context;
+};
+
+} // namespace blink
+
+#endif // CompositeDataConsumerHandle_h