Remove code allowing implicit conversion from WebElementCollection to WebNodeList.

Source/web/WebNodeList.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 13 matching lines @@
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * 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.
  */
 
 #include "config.h"
 #include "WebNodeList.h"
 
-#include "core/dom/Element.h"
+#include "WebNode.h"
 #include "core/dom/Node.h"
 #include "core/dom/NodeList.h"
-#include "core/html/HTMLCollection.h"
 #include "wtf/PassRefPtr.h"
 
-#include "WebNode.h"
-
 using namespace WebCore;
 
 namespace blink {
 
-// FIXME(crbug.com/235008): Remove once chromium has been updated to stop using
-// WebCollection as a WebNodeList.
-class NodeListWithInternalCollection FINAL : public NodeList {
-public:
-    static PassRefPtr<NodeListWithInternalCollection> create(HTMLCollection* collection)
-    {
-        return adoptRef(new NodeListWithInternalCollection(collection));
-    }
-
-    // We have null checks in the following methods because WebNodeList does not have a isNull() method
-    // and callers need to be moved to WebElementCollection and need to handle null using
-    // WebElementCollection::isNull().
-    virtual unsigned length() const OVERRIDE { return m_collection ? m_collection->length() : 0; }
-    virtual Node* item(unsigned index) const OVERRIDE { return m_collection ? m_collection->item(index) : 0; }
-
-private:
-    explicit NodeListWithInternalCollection(HTMLCollection* collection)
-        : m_collection(collection)
-    {
-    }
-
-    RefPtr<HTMLCollection> m_collection;
-};
-
-WebNodeList::WebNodeList(const WebElementCollection& n)
-    : m_private(0)
-{
-    assign(NodeListWithInternalCollection::create(n.m_private).leakRef());
-}
-
 void WebNodeList::reset()
 {
     assign(0);
 }
 
 void WebNodeList::assign(const WebNodeList& other)
 {
     NodeList* p = const_cast<NodeList*>(other.m_private);
     if (p)
         p->ref();
@@ skipping 17 matching lines @@
 {
     return m_private->length();
 }
 
 WebNode WebNodeList::item(size_t index) const
 {
     return WebNode(m_private->item(index));
 }
 
 } // namespace blink