Oilpan: move LiveNodeList collections to the heap.

public/web/WebNode.h

 /*
  * Copyright (C) 2009 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 126 matching lines @@
     {
         T res;
         res.WebNode::assign(*this);
         return res;
     }
 
 #if BLINK_IMPLEMENTATION
     WebNode(const PassRefPtrWillBeRawPtr<WebCore::Node>&);
     WebNode& operator=(const PassRefPtrWillBeRawPtr<WebCore::Node>&);
     operator PassRefPtrWillBeRawPtr<WebCore::Node>() const;
+#if ENABLE(OILPAN)
+    // This constructor enables creation of WebNodes from Members

[tkent, 2014/05/19 07:23:40]

Sounds reasonable.

+    // holding WebCore::Node-derived objects (this shows up in WebVector
+    // assignments, for instance.) It is needed because a RawPtr<T> constructor
+    // from a Member<U> isn't provided, hence the above constructor
+    // won't be usable.

[haraken, 2014/05/19 09:19:50]

Yeah, probably sounds reasonable.

It might be possible to provide a RawPtr<T> constructor from a Member<U>, but it
might confuse overload resolution for toV8() etc. (I'm not sure.)

+    template<typename U>
+    WebNode(const WebCore::Member<U>& other, EnsurePtrConvertibleArgDecl(U, WebCore::Node))
+        : m_private(other.get())
+    {
+    }
+#endif
 #endif
 
 #if BLINK_IMPLEMENTATION
     template<typename T> T* unwrap()
     {
         return static_cast<T*>(m_private.get());
     }
 
     template<typename T> const T* constUnwrap() const
     {
@@ skipping 16 matching lines @@
 }
 
 inline bool operator<(const WebNode& a, const WebNode& b)
 {
     return a.lessThan(b);
 }
 
 } // namespace blink
 
 #endif