The RetainPtr is only used for CF platform in this class.
Created attachment 55941 [details] patch
Comment on attachment 55941 [details] patch > #include <errno.h> > #include <stdio.h> > +#if PLATFORM(CF) > #include <wtf/RetainPtr.h> > +#endif > #include <wtf/Threading.h> > #include <wtf/Vector.h> > #include <wtf/text/CString.h> Includes inside #if go in their own paragraph, not sorted in the middle of the main include paragraph. We also normally put #if in headers so we don't have to make includes everywhere conditional. For example, see the #if ENABLE(SVG) in SVGElement.h. Despite those minor problems it seems OK to land this as-is.
Comment on attachment 55941 [details] patch Clearing flags on attachment: 55941 Committed r59482: <http://trac.webkit.org/changeset/59482>
All reviewed patches have been landed. Closing bug.