The function findAll() of QWebElement returns a QList of QWebElement. This return type does not give any flexibility to upgrade the API later, (like adding filtering on the list, or lazy population). To add some flexibility, a d-pointer is needed in the return type. The type QWebElementCollection should be reintroduced.
Created attachment 41858 [details] Path to replace QList<QWebElement> by QWebElementCollection This patch basically revert the commit 42994 (e4ed760783109673896551892c029dc043c70b71 on git). The changes from the original patch are: -QWebElement operator[] do not return a const QWebElement anymore to be coherent with the previous use of QList -The new tests had to be adapted. If the idea is accepted and we reintroduce QWebElementCollection, a new bug should be created in the tree 29843 to review the API of QWebElementCollection.
LGTM, feel free to create an API-spinoff
Comment on attachment 41858 [details] Path to replace QList<QWebElement> by QWebElementCollection Clearing flags on attachment: 41858 Committed r50058: <http://trac.webkit.org/changeset/50058>
All reviewed patches have been landed. Closing bug.
I guess we need to add a follow up bug report reviewing all the API of this new class.