Make WebThreadedDataReceiver wrap a Blink-managed object instead.

Source/core/html/parser/ThreadedDataReceiver.h

-// Copyright 2014 The Chromium Authors. All rights reserved.
+// Copyright (c) 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 WebThreadedDataReceiver_h
-#define WebThreadedDataReceiver_h
+#ifndef ThreadedDataReceiver_h
+#define ThreadedDataReceiver_h
+
+#include "platform/heap/Handle.h"
 
 namespace blink {
 
 class WebThread;
 
-class WebThreadedDataReceiver {
+class ThreadedDataReceiver : public WillBeGarbageCollectedMixin {
 public:
+    virtual ~ThreadedDataReceiver() { }
+
+#if !ENABLE(OILPAN)
+    virtual void ref() = 0;
+    virtual void deref() = 0;
+#endif

[haraken, 2015/01/26 01:47:04]

I wonder why you need to define ref() and deref() here (and thus need to
override ref() and deref() in ParserDataReceiver).

[sof, 2015/01/26 06:14:20]

WebThreadedDataReceiver::WebPrivatePtr<> wraps a ThreadedDataReceiver, so it
needs to have the refcounting pair of methods available non-Oilpan.

I don't know the planned set of implementations of this interface (besides this
one), so that seemed tidiest. If ParserDataReceiver is the only one, we can
simplify and just have the WebThreadedDataReceiver delegate calls to it &
shorten out this interface (and its mixin + ref-forwarding trickery.)

[haraken, 2015/01/26 06:25:30]

Thanks, makes sense.

+
     virtual void acceptData(const char* data, int dataLength) = 0;
     virtual WebThread* backgroundThread() = 0;
 
     virtual bool needsMainthreadDataCopy() = 0;
     virtual void acceptMainthreadDataNotification(const char* data, int dataLength, int encodedDataLength) = 0;
-
-    virtual ~WebThreadedDataReceiver() { }
 };
 
 } // namespace blink
 
-#endif
+#endif // ThreadedDataReceiver_h