FrameLoader: Convert three little loops to using Vector<Ref<Frame>>.
Created attachment 210666 [details] Patch
Comment on attachment 210666 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=210666&action=review > Source/WebCore/loader/FrameLoader.cpp:2323 > for (Frame* child = m_frame.tree().lastChild(); child; child = child->tree().previousSibling()) These look bit last season. Wonder if we should add FrameIterator?
(In reply to comment #2) > (From update of attachment 210666 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=210666&action=review > > > Source/WebCore/loader/FrameLoader.cpp:2323 > > for (Frame* child = m_frame.tree().lastChild(); child; child = child->tree().previousSibling()) > > These look bit last season. Wonder if we should add FrameIterator? That sounds like fun.
Comment on attachment 210666 [details] Patch Clearing flags on attachment: 210666 Committed r155168: <http://trac.webkit.org/changeset/155168>
All reviewed patches have been landed. Closing bug.