Oilpan: move 2D Canvas and WebGL objects to the heap.

Source/core/html/canvas/WebGLActiveInfo.h

Index: Source/core/html/canvas/WebGLActiveInfo.h
diff --git a/Source/core/html/canvas/WebGLActiveInfo.h b/Source/core/html/canvas/WebGLActiveInfo.h
index b02e0c78e35efff710f2c6bd02bbf58ea2180003..193bd33901db3c6a2e763899264d9d5151f2983d 100644
--- a/Source/core/html/canvas/WebGLActiveInfo.h
+++ b/Source/core/html/canvas/WebGLActiveInfo.h
@@ -34,16 +34,18 @@
 
 namespace WebCore {
 
-class WebGLActiveInfo : public RefCounted<WebGLActiveInfo>, public ScriptWrappable {
+class WebGLActiveInfo FINAL : public RefCountedWillBeGarbageCollectedFinalized<WebGLActiveInfo>, public ScriptWrappable {

[bajones, 2014/07/18 20:31:29]

This doesn't need to be Finalized, does it?

[sof, 2014/07/18 21:39:29]

String gets in the way of that; it having a RefPtr<> to its Impl that needs to
be finalized.

(Classes derived from the non-finalized GarbageCollected<> are checked by the
Oilpan clang plugin, and will flag an error if you tried to use it here.)

 public:
-    static PassRefPtr<WebGLActiveInfo> create(const String& name, GLenum type, GLint size)
+    static PassRefPtrWillBeRawPtr<WebGLActiveInfo> create(const String& name, GLenum type, GLint size)
     {
-        return adoptRef(new WebGLActiveInfo(name, type, size));
+        return adoptRefWillBeNoop(new WebGLActiveInfo(name, type, size));
     }
     String name() const { return m_name; }
     GLenum type() const { return m_type; }
     GLint size() const { return m_size; }
 
+    void trace(Visitor*) { }
+
 private:
     WebGLActiveInfo(const String& name, GLenum type, GLint size)
         : m_name(name)