Oilpan: move custom element objects to the heap.

Source/core/dom/custom/CustomElementMicrotaskResolutionStep.h

 /*
  * 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
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 15 matching lines @@
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (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 CustomElementMicrotaskResolutionStep_h
 #define CustomElementMicrotaskResolutionStep_h
 
 #include "core/dom/custom/CustomElementDescriptor.h"
 #include "core/dom/custom/CustomElementMicrotaskStep.h"
+#include "platform/heap/Handle.h"
 #include "wtf/PassOwnPtr.h"
 #include "wtf/PassRefPtr.h"
 #include "wtf/RefPtr.h"
 
 namespace WebCore {
 
 class CustomElementRegistrationContext;
 class Element;
 
 class CustomElementMicrotaskResolutionStep : public CustomElementMicrotaskStep {
     WTF_MAKE_NONCOPYABLE(CustomElementMicrotaskResolutionStep);
 public:
-    static PassOwnPtr<CustomElementMicrotaskResolutionStep> create(PassRefPtr<CustomElementRegistrationContext>, PassRefPtr<Element>, const CustomElementDescriptor&);
+    static PassOwnPtrWillBeRawPtr<CustomElementMicrotaskResolutionStep> create(PassRefPtrWillBeRawPtr<CustomElementRegistrationContext>, PassRefPtrWillBeRawPtr<Element>, const CustomElementDescriptor&);
 
     virtual ~CustomElementMicrotaskResolutionStep();
 
+    virtual void trace(Visitor*) OVERRIDE;
+
 private:
-    CustomElementMicrotaskResolutionStep(PassRefPtr<CustomElementRegistrationContext>, PassRefPtr<Element>, const CustomElementDescriptor&);
+    CustomElementMicrotaskResolutionStep(PassRefPtrWillBeRawPtr<CustomElementRegistrationContext>, PassRefPtrWillBeRawPtr<Element>, const CustomElementDescriptor&);
 
     virtual Result process() OVERRIDE;
     virtual bool needsProcessOrStop() const OVERRIDE;
 
 #if !defined(NDEBUG)
     virtual void show(unsigned indent) OVERRIDE;
 #endif
 
-    RefPtr<CustomElementRegistrationContext> m_context;
-    RefPtr<Element> m_element;
+    RefPtrWillBeMember<CustomElementRegistrationContext> m_context;
+    RefPtrWillBeMember<Element> m_element;
     CustomElementDescriptor m_descriptor;

[haraken, 2014/05/26 15:40:23]

You need to trace this one, or you can simply drop ALLOW_ONLY_INLINE_ALLOCATION
from  CustomElementDescriptor since CustomElementDescriptor has no members to be
traced.

[sof, 2014/05/26 16:41:08]

If the object doesn't have any Members (CustomElementDescriptor doesn't), it
doesn't need a trace() method. Separately adding ALLOW_ONLY_INLINE_ALOCATION()
is useful to constrain use, I think.

 };
 
 }
 
 #endif // CustomElementMicrotaskResolutionStep_h