Redirect HTML resource bytes directly to parser thread (Blink side CL)

public/platform/WebParserResourceBridge.h

 /*
- * Copyright (C) 2013 Google Inc.  All rights reserved.
+ * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
  * copyright notice, this list of conditions and the following disclaimer
  * in the documentation and/or other materials provided with the
  * distribution.
  *     * Neither the name of Google Inc. nor the names of its
  * contributors may be used to endorse or promote products derived from
  * this software without specific prior written permission.
  *
  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
  * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
  * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef HTMLParserThread_h
-#define HTMLParserThread_h
+#ifndef WebParserResourceBridge_h
+#define WebParserResourceBridge_h
 
-#include "wtf/Functional.h"
-#include "wtf/OwnPtr.h"
-#include "public/platform/WebThread.h"
+namespace blink {
 
-namespace WebCore {
+class WebThread;
 
-class HTMLParserThread {
+class WebParserResourceBridge {

[abarth-chromium, 2013/12/18 18:28:49]

Is this specific to parsers?  It seems like a generic mechanism for receiving
data callbacks on a background thread.  WebThreadedResourceProvider ?

 public:
-    static HTMLParserThread* shared();
-    void postTask(const Closure&);
+    class Peer {

[abarth-chromium, 2013/12/18 18:28:49]

I'd make this a top-level class instead of a nested class.  Perhaps
WebThreadedResourceClient?

+    public:
+        virtual void OnReceivedData(const char* data, size_t length) = 0;

[abarth-chromium, 2013/12/18 18:28:49]

didReceivedData

+    };
 
-private:
-    HTMLParserThread();
-    ~HTMLParserThread();
+    virtual void setPeer(Peer*) = 0;

[abarth-chromium, 2013/12/18 18:28:49]

setClient

+    virtual WebThread* getParserThread() = 0;

[abarth-chromium, 2013/12/18 18:28:49]

If this interface isn't specific to parsing, perhaps "resourceThread" ?

 
-    OwnPtr<blink::WebThread> m_thread;
+    virtual ~WebParserResourceBridge() { }
 };
 
-} // namespace WebCore
+} // namespace blink
 
-#endif // HTMLParserThread_h
+#endif