Why should Windows be left out? Time to get access to the new Web Inspector!
<rdar://problem/14159549>
<rdar://problem/14791568>
Created attachment 209243 [details] Patch just copies files around.
Comment on attachment 209243 [details] Patch just copies files around. The patch looks big and scary, but it's basically just copying files around. Visual Studio is not compact in its representation of things.
Created attachment 209244 [details] Screenshot of New Inspector on Windows
This looks good to me! I'd prefer someone with Windows experience to look over the Windows project/build related bits.
Comment on attachment 209243 [details] Patch just copies files around. I just want to make sure. As we add new files to WebInspectorUI, will we need to add entries to the vcxproj and vcxproj.filters files? I hope not, that is a burden we would like to avoid. Is there a way for VisualStudio to just include a directory without enumerating all the contents?
(In reply to comment #7) > (From update of attachment 209243 [details]) > I just want to make sure. As we add new files to WebInspectorUI, will we need to add entries to the vcxproj and vcxproj.filters files? I hope not, that is a burden we would like to avoid. Is there a way for VisualStudio to just include a directory without enumerating all the contents? I'll look into it, but I think it's unlikely. The project file really only exists to make it easy to find/edit the css/js/html files. The "build" phase is done purely with scripts/wildcards, so forgetting to add a file in the project will NOT prevent the new code from being integrated and used with the build.
This feature assumes that ENABLE(CSS_IMAGE_SET) is true. For some reason, this was never activated on Windows (even though all the code seems to be in place).
Created attachment 209280 [details] Screenshot after enabling CSS_IMAGE_SET
Comment on attachment 209243 [details] Patch just copies files around. r=me on code changes, rs=me on project changes.
Committed r154395: <http://trac.webkit.org/changeset/154395>