It caused valgrind issues.
Created attachment 153497 [details] Patch
Created attachment 153501 [details] Patch
Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.
Comment on attachment 153501 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=153501&action=review > Tools/DumpRenderTree/chromium/DumpRenderTree.cpp:85 > + OwnPtr<MockWebKitPlatformSupport> mockPlatform; mockPlatform -> m_mockPlatform > Tools/DumpRenderTree/chromium/MockWebKitPlatformSupport.cpp:38 > -Platform* MockWebKitPlatformSupport::create() > +MockWebKitPlatformSupport* MockWebKitPlatformSupport::create() Should this return a PassOwnPtr? > Tools/DumpRenderTree/chromium/MockWebKitPlatformSupport.h:39 > + ~MockWebKitPlatformSupport() { } Please move this declaration out of line.
Created attachment 153750 [details] Patch
Comment on attachment 153501 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=153501&action=review >> Tools/DumpRenderTree/chromium/DumpRenderTree.cpp:85 >> + OwnPtr<MockWebKitPlatformSupport> mockPlatform; > > mockPlatform -> m_mockPlatform Done. >> Tools/DumpRenderTree/chromium/MockWebKitPlatformSupport.cpp:38 >> +MockWebKitPlatformSupport* MockWebKitPlatformSupport::create() > > Should this return a PassOwnPtr? Yes, done. >> Tools/DumpRenderTree/chromium/MockWebKitPlatformSupport.h:39 >> + ~MockWebKitPlatformSupport() { } > > Please move this declaration out of line. Done.
Comment on attachment 153750 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=153750&action=review > Tools/DumpRenderTree/chromium/MockWebKitPlatformSupport.cpp:40 > - return new MockWebKitPlatformSupport(); > + return WTF::adoptPtr(new MockWebKitPlatformSupport()); I think the WTF:: here is optional.
Comment on attachment 153750 [details] Patch Clearing flags on attachment: 153750 Committed r123345: <http://trac.webkit.org/changeset/123345>
All reviewed patches have been landed. Closing bug.