This includes testapi, testRegExp, WebCoreTestSupport, TestWebKitAPI, gtest-md, record-memory-win, TestNetscapePlugin, WebKitLauncherWin, ImageDiff, ImageDiffLauncher, DumpRenderTree, DumpRenderTree Launcher
<rdar://problem/13025341>
Created attachment 187199 [details] DRT and related projects This include DumpRenderTree and launcer, ImageDiff and launcher, WebCoreTestSupport and TestNetscapePlugin
Comment on attachment 187199 [details] DRT and related projects View in context: https://bugs.webkit.org/attachment.cgi?id=187199&action=review Looks great! I think you want to lose the *.users (and probably elsewhere in all the VS2010 stuff we've been playing with), Also, I had a question about a file you deleted, that I hoped you could clarify in your ChangeLog entry. Thanks! > Source/WebCore/ChangeLog:10 > + * WebCore.vcxproj/WebCoreTestSupport.vcxproj.user: Added. I'm not sure we want to be checking in *.user files. I think it only contains user-specific stuff like break points and other stuff. We should probably have them added to svn:ignore. > Source/WebCore/WebCore.vcxproj/WebCoreTestSupport.vcxproj:52 > + <DisableSpecificWarnings Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">4065;4273;4565;4701;4702;%(DisableSpecificWarnings)</DisableSpecificWarnings> All of these targets use the same disabled warnings: to the props file, please! > Source/WebCore/WebCore.vcxproj/WebCoreTestSupport.vcxproj.user:3 > +</Project> I don't think we ever want to add these to source control. > Tools/ChangeLog:17 > + * DumpRenderTree/DumpRenderTree.vcxproj/DumpRenderTree/DumpRenderTreeLauncher.vcxproj.user: Added. Ditto my user comments from elsewhere. > Tools/ChangeLog:30 > + * DumpRenderTree/DumpRenderTree.vcxproj/ImageDiff/ImageDiffLauncher.vcxproj.user: Added. Ditto > Tools/ChangeLog:42 > + * DumpRenderTree/DumpRenderTree.vcxproj/TestNetscapePlugin/TestNetscapePlugin.vcxproj.user: Added. Ditto. > Tools/ChangeLog:48 > + * DumpRenderTree/TestNetscapePlugIn/win/CallJSThatDestroysPlugin.cpp: Removed. Why is this being deleted? Could you add a note, like maybe it's not used anymore or something? > Tools/DumpRenderTree/DumpRenderTree.vcxproj/DumpRenderTree/DumpRenderTree.vcxproj.user:3 > +</Project> Delete please. > Tools/DumpRenderTree/DumpRenderTree.vcxproj/DumpRenderTree/DumpRenderTreeLauncher.vcxproj.user:4 > + <LocalDebuggerCommandArguments>C:\cygwin\home\Roger\WebKit\OpenSource\LayoutTests\fast\js\kde</LocalDebuggerCommandArguments> See! I don't want that for my build! :) Delete from checking, please! > Tools/DumpRenderTree/DumpRenderTree.vcxproj/ImageDiff/ImageDiff.vcxproj.user:3 > +</Project> Delete this. > Tools/DumpRenderTree/DumpRenderTree.vcxproj/ImageDiff/ImageDiffLauncher.vcxproj.user:3 > +</Project> Delete please. > Tools/DumpRenderTree/DumpRenderTree.vcxproj/TestNetscapePlugin/TestNetscapePlugin.vcxproj.user:3 > +</Project> Deletr
Ok, I'll do a mass svn delete of all the user files. Also the file in question is a TestNetscapeAPI test that was in the wrong folder. I jsut moved it to the tests folder. I'll have to fix the VS2005 build when I land this.
Committed with changes: http://trac.webkit.org/changeset/142303 I committed removing the .user files in a separate patch.
I still have to do WebKitAPI, g-testmd, record-memory-win and WebKitLaunherWin
Created attachment 187987 [details] Last remaining test projects gtest-md, testwebkitapi and record-memory Also updates WebKit solution
Created attachment 188147 [details] Remaining test projects
Comment on attachment 188147 [details] Remaining test projects Looks good to me.
committed: http://trac.webkit.org/changeset/142785