Use newImageSnapshot() to get an image from a Canvas

Source/platform/graphics/Pattern.h

 /*
  * Copyright (C) 2006, 2007, 2008 Apple Computer, Inc.  All rights reserved.
  * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2007-2008 Torch Mobile, Inc.
  * 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:
  * 1. Redistributions of source code must retain the above copyright
@@ skipping 14 matching lines @@
  * (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 Pattern_h
 #define Pattern_h
 
 #include "SkShader.h"
 #include "platform/PlatformExport.h"
 #include "platform/graphics/Image.h"
+#include "platform/graphics/StaticBitmapImage.h"

[Justin Novosad, 2014/07/29 14:56:52]

No need for include.

[Rémi Piotaix, 2014/07/29 17:18:23]

Done.

 #include "platform/transforms/AffineTransform.h"
 
 #include "wtf/PassRefPtr.h"
 #include "wtf/RefCounted.h"
 #include "wtf/RefPtr.h"
 
 namespace blink {
 
-class AffineTransform;
-
 class PLATFORM_EXPORT Pattern : public RefCounted<Pattern> {
 public:
-    static PassRefPtr<Pattern> create(PassRefPtr<Image> tileImage, bool repeatX, bool repeatY)
-    {
-        return adoptRef(new Pattern(tileImage, repeatX, repeatY));
-    }
-    ~Pattern();
+    static PassRefPtr<Pattern> create(PassRefPtr<Image> tileImage, bool repeatX, bool repeatY);
+
+    virtual ~Pattern();

[Justin Novosad, 2014/07/29 14:56:52]

The pattern destructor should be responsible for for notifying v8 that
externalMemoryAllocated is going to 0.

[Rémi Piotaix, 2014/07/29 17:18:23]

Already in Pattern.cpp ?

 
     SkShader* shader();
 
     void setPatternSpaceTransform(const AffineTransform& patternSpaceTransformation);
-    const AffineTransform& getPatternSpaceTransform() { return m_patternSpaceTransformation; };
+    const AffineTransform& getPatternSpaceTransform() { return m_patternSpaceTransformation; }
 
     bool repeatX() const { return m_repeatX; }
     bool repeatY() const { return m_repeatY; }
+protected:
+    virtual PassRefPtr<SkShader> createShader() = 0;
 
-private:
-    Pattern(PassRefPtr<Image>, bool repeatX, bool repeatY);
+    Pattern(bool repeatX, bool repeatY)
+        : m_repeatX(repeatX)
+        , m_repeatY(repeatY)
+        , m_externalMemoryAllocated(0) { }
 
-    RefPtr<NativeImageSkia> m_tileImage;
     bool m_repeatX;
     bool m_repeatY;
     AffineTransform m_patternSpaceTransformation;
+    int m_externalMemoryAllocated;

[Justin Novosad, 2014/07/29 14:56:52]

should probably be intptr_t.  What type does v8 expect?  Make it private.

[Rémi Piotaix, 2014/07/30 18:59:12]

Done.

+
+private:
     RefPtr<SkShader> m_pattern;
-    int m_externalMemoryAllocated;
 };
 
-} //namespace
+} // namespace
 
 #endif