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

sky/engine/core/painting/PaintingContext.cpp

Index: sky/engine/core/painting/PaintingContext.cpp
diff --git a/sky/engine/core/painting/PaintingContext.cpp b/sky/engine/core/painting/PaintingContext.cpp
index 1ab9198e5dca7eb9f0cd9a176c947ea8c9ea0132..96fab9038c824e411e032611d29a68416b303a38 100644
--- a/sky/engine/core/painting/PaintingContext.cpp
+++ b/sky/engine/core/painting/PaintingContext.cpp
@@ -17,8 +17,11 @@ PassRefPtr<PaintingContext> PaintingContext::create(PassRefPtr<Element> element,
     return adoptRef(new PaintingContext(element, size));
 }
 
+// TODO(iansf): Get rid of PaintingContext, which is only relevant to DOM-based
+//              Sky apps, which are now deprecated.  For now, make the compiler
+//              happy by constructing Canvas with a nullptr.
 PaintingContext::PaintingContext(PassRefPtr<Element> element, const FloatSize& size)
-    : Canvas(size)
+    : Canvas(nullptr)
     , m_element(element)
 {
 }
@@ -29,9 +32,6 @@ PaintingContext::~PaintingContext()
 
 void PaintingContext::commit()
 {
-    if (!isRecording())
-        return;
-    PaintingTasks::enqueueCommit(m_element, finishRecording());
     m_element->document().scheduleVisualUpdate();
 }