Decouple Canvas from DisplayList and map Picture and PictureRecorder more directly to their Skia co…

sky/engine/core/painting/PictureRecorder.h

Index: sky/engine/core/painting/PictureRecorder.h
diff --git a/sky/engine/core/painting/PictureRecorder.h b/sky/engine/core/painting/PictureRecorder.h
index 485edd50499bb006a22784d5d68a202816dbd5b6..311aa02b18c646526a2f20d70c25da7c59a795cd 100644
--- a/sky/engine/core/painting/PictureRecorder.h
+++ b/sky/engine/core/painting/PictureRecorder.h
@@ -5,22 +5,41 @@
 #ifndef SKY_ENGINE_CORE_PAINTING_PICTURERECORDER_H_
 #define SKY_ENGINE_CORE_PAINTING_PICTURERECORDER_H_
 
-#include "sky/engine/core/painting/Canvas.h"
+#include "sky/engine/tonic/dart_wrappable.h"
+#include "sky/engine/wtf/PassRefPtr.h"
+#include "sky/engine/wtf/RefCounted.h"
+#include "third_party/skia/include/core/SkPictureRecorder.h"
 
 namespace blink {
 
 class Picture;
+class Canvas;
 
-class PictureRecorder : public Canvas {
+class PictureRecorder : public RefCounted<PictureRecorder>,
+                        public DartWrappable {
     DEFINE_WRAPPERTYPEINFO();
 public:
-    ~PictureRecorder() override;
-    static PassRefPtr<PictureRecorder> create(double width, double height);
+    static PassRefPtr<PictureRecorder> create()
+    {
+        return adoptRef(new PictureRecorder());
+    }
 
+    ~PictureRecorder();
+
+    // PassRefPtr<Canvas> beginRecording(double width, double height);
+    SkCanvas* beginRecording(double width, double height);
     PassRefPtr<Picture> endRecording();
+    bool isRecording();
+
+    void set_canvas(PassRefPtr<Canvas> canvas);
+
+    int foo = 0;

[eseidel, 2015/06/23 00:41:32]

I suspect you don't want this?

[iansf, 2015/06/23 22:46:09]

You may be right... ;-)

 
 private:
-    PictureRecorder(const FloatSize& size);
+	PictureRecorder();
+
+	OwnPtr<SkPictureRecorder> m_pictureRecorder;
+	RefPtr<Canvas> m_canvas;
 };
 
 } // namespace blink