Add TimedCanvasDrawListener

third_party/WebKit/Source/core/html/canvas/CanvasDrawListener.h

Index: third_party/WebKit/Source/core/html/canvas/CanvasDrawListener.h
diff --git a/third_party/WebKit/Source/core/html/canvas/CanvasDrawListener.h b/third_party/WebKit/Source/core/html/canvas/CanvasDrawListener.h
index b1fc834dc496811d193393a7c1e1d6abdbaf61eb..8ce29ad1264f2258f760682d0fc64f8c726ffc54 100644
--- a/third_party/WebKit/Source/core/html/canvas/CanvasDrawListener.h
+++ b/third_party/WebKit/Source/core/html/canvas/CanvasDrawListener.h
@@ -7,6 +7,7 @@
 
 #include "core/CoreExport.h"
 #include "platform/heap/Handle.h"
+#include "public/platform/WebCanvasCaptureHandler.h"
 #include "third_party/skia/include/core/SkImage.h"
 #include "wtf/PassRefPtr.h"
 
@@ -14,9 +15,14 @@ namespace blink {
 
 class CORE_EXPORT CanvasDrawListener : public GarbageCollectedMixin {

[esprehn, 2015/12/15 23:58:01]

I wonder if this should be mixin if it's going to have members?

[emircan, 2015/12/16 15:36:21]

It looks like Mixin is allowed to have members from the example given [0].
Either way it is an OwnPtr to a chrome object and no tracing is needed right?

[0]
https://chromium.googlesource.com/chromium/src/+/master/third_party/WebKit/So...

 public:
-    virtual ~CanvasDrawListener() {}
-    virtual bool needsNewFrame() const = 0;
-    virtual void sendNewFrame(const WTF::PassRefPtr<SkImage>&) = 0;
+    virtual ~CanvasDrawListener();
+    virtual bool needsNewFrame() const;
+    virtual void sendNewFrame(const WTF::PassRefPtr<SkImage>&);
+
+protected:
+    explicit CanvasDrawListener(const PassOwnPtr<WebCanvasCaptureHandler>&);
+
+    OwnPtr<WebCanvasCaptureHandler> m_handler;
 };
 
 } // namespace blink