Oilpan: Prepare to support garbage-collected Node in WebNode hierarchy.

Source/web/WebFormElement.cpp

 /*
  * 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 62 matching lines @@
 void WebFormElement::submit()
 {
     unwrap<HTMLFormElement>()->submit();
 }
 
 void WebFormElement::getNamedElements(const WebString& name,
                                       WebVector<WebNode>& result)
 {
     Vector<RefPtr<Element> > tempVector;
     unwrap<HTMLFormElement>()->getNamedElements(name, tempVector);
+#if !ENABLE(OILPAN)

[haraken, 2014/05/08 06:22:37]

#if ENABLE(OILPAN)
#else
#endif

would be better.

Also add a FIXME that this should be removed once we change |tempVector| to
WillBeHeapVector<RefPtrWillBeMember<Element>>.

[tkent, 2014/05/08 07:39:30]

Done.

     result.assign(tempVector);
+#else
+    Vector<WebNode> tempVector2;

[wibling-chromium, 2014/05/08 07:07:01]

Just to make sure i understand. This extra copy is only needed until we change
HTMLFormElement's getNamedLElements to return a Vector<RefPtrWillBeRawPtr<Node>?
If so could we add a FIXME?

[tkent, 2014/05/08 07:39:30]

Right.  We can remove the extra copy later.

+    tempVector2.reserveCapacity(tempVector.size());
+    for (size_t i = 0; i < tempVector.size(); ++i)
+        tempVector2.append(WebNode(tempVector[i].get()));
+    result.assign(tempVector2);
+#endif
 }
 
 void WebFormElement::getFormControlElements(WebVector<WebFormControlElement>& result) const
 {
     const HTMLFormElement* form = constUnwrap<HTMLFormElement>();
-    Vector<RefPtr<HTMLFormControlElement> > formControlElements;
+    Vector<WebFormControlElement> formControlElements;
 
     const Vector<FormAssociatedElement*>& associatedElements = form->associatedElements();
     for (Vector<FormAssociatedElement*>::const_iterator it = associatedElements.begin(); it != associatedElements.end(); ++it) {
         if ((*it)->isFormControlElement())
             formControlElements.append(toHTMLFormControlElement(*it));
     }
     result.assign(formControlElements);
 }
 
 bool WebFormElement::checkValidity()
 {
     return unwrap<HTMLFormElement>()->checkValidity();
 }
 
 void WebFormElement::finishRequestAutocomplete(WebFormElement::AutocompleteResult result)
 {
     unwrap<HTMLFormElement>()->finishRequestAutocomplete(static_cast<HTMLFormElement::AutocompleteResult>(result));
 }
 
-WebFormElement::WebFormElement(const PassRefPtr<HTMLFormElement>& e)
+WebFormElement::WebFormElement(const PassRefPtrWillBeRawPtr<HTMLFormElement>& e)
     : WebElement(e)
 {
 }
 
-WebFormElement& WebFormElement::operator=(const PassRefPtr<HTMLFormElement>& e)
+WebFormElement& WebFormElement::operator=(const PassRefPtrWillBeRawPtr<HTMLFormElement>& e)
 {
     m_private = e;
     return *this;
 }
 
-WebFormElement::operator PassRefPtr<HTMLFormElement>() const
+WebFormElement::operator PassRefPtrWillBeRawPtr<HTMLFormElement>() const
 {
     return toHTMLFormElement(m_private.get());
 }
 
 } // namespace blink