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

Source/core/html/canvas/WebGLActiveInfo.h

 /*
  * Copyright (C) 2009 Apple 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
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 16 matching lines @@
 #define WebGLActiveInfo_h
 
 #include "bindings/core/v8/ScriptWrappable.h"
 #include "platform/graphics/GraphicsTypes3D.h"
 #include "wtf/PassRefPtr.h"
 #include "wtf/RefCounted.h"
 #include "wtf/text/WTFString.h"
 
 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)
         , m_type(type)
         , m_size(size)
     {
         ASSERT(name.length());
         ASSERT(type);
         ASSERT(size);
         ScriptWrappable::init(this);
     }
     String m_name;
     GLenum m_type;
     GLint m_size;
 };
 
 }
 
 #endif // WebGLActiveInfo_h