Ideally we could use just the forward declaration and remove the header but I wanted to inline some code using that and it looks like removing the forward declaration does not work cause the include seems to do nothing.
Created attachment 341902 [details] Patch
I just added this patch that I got from Enrique to fix this issue, I'll r+ myself if he agrees to get it landed.
Yes, please, go ahead.
Comment on attachment 341902 [details] Patch Attachment 341902 [details] did not pass win-ews (win): Output: http://webkit-queues.webkit.org/results/7979296 New failing tests: http/tests/security/contentSecurityPolicy/video-with-https-url-allowed-by-csp-media-src-star.html
Created attachment 341908 [details] Archive of layout-test-results from ews206 for win-future The attached test failures were seen while running run-webkit-tests on the win-ews. Bot: ews206 Port: win-future Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Comment on attachment 341902 [details] Patch Clearing flags on attachment: 341902 Committed r232505: <https://trac.webkit.org/changeset/232505>
All reviewed patches have been landed. Closing bug.
<rdar://problem/40796784>