[Cocoa] Consolidate duplicate copies of WebArchiveDumpSupport in DRT and WKTR into WebCoreTestSupport
Created attachment 298241 [details] Patch
Created attachment 298243 [details] Patch
Comment on attachment 298243 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=298243&action=review > Source/WebCore/PlatformMac.cmake:747 > + bindings/cocoa/WebArchiveDumpSupport.mm bindings?
(In reply to comment #3) > Comment on attachment 298243 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=298243&action=review > > > Source/WebCore/PlatformMac.cmake:747 > > + bindings/cocoa/WebArchiveDumpSupport.mm > > bindings? Oops! Will fix.
Alex reviewed in person.
Created attachment 298248 [details] Patch
Comment on attachment 298248 [details] Patch Clearing flags on attachment: 298248 Committed r210467: <http://trac.webkit.org/changeset/210467>
All reviewed patches have been landed. Closing bug.
View in context: https://bugs.webkit.org/attachment.cgi?id=298243&action=review > Source/WebCore/platform/spi/cf/CFNetworkSPI.h:186 > +CFTypeID CFURLResponseGetTypeID(); No code actually uses this. Do we expect any code to use it? > Source/WebCore/platform/spi/cf/CFNetworkSPI.h:188 > +void CFURLResponseSetExpectedContentLength(CFURLResponseRef, SInt64 length); No code actually uses this. Do we expect any code to use it?
Why not put the files in Tools/TestRunnerShared/ ?
(In reply to comment #7) > Comment on attachment 298248 [details] > Patch > > Clearing flags on attachment: 298248 > > Committed r210467: <http://trac.webkit.org/changeset/210467> It broke the Apple Mac cmake build. https://build.webkit.org/builders/Apple%20El%20Capitan%20CMake%20Debug%20%28Build%29/builds/11115