Cleanup "addCookie" and its sole user addCookie doesn't cleanly fit in with the native cookie storage API, and a lot of dead code was added to support it as well. Refactoring this to be "setCookies" like NSHTTPCookieStorage supports will be better going forward.
Created attachment 304098 [details] Patch
Already reviewed. Waiting for EWS.
Created attachment 304117 [details] Patch
Attempt to fix GTK, and hopefully the rest of the EWS bots will actually be responsive this time.
Created attachment 304120 [details] Patch
Comment on attachment 304120 [details] Patch r=me GTK is still sad.
Created attachment 304160 [details] Patch for landing
(In reply to comment #6) > Comment on attachment 304120 [details] > Patch > > r=me > > GTK is still sad. This is a known issue with our forwarding headers generator. It doesn't scan generated sources, so in this case #include <WebCoreCookie.h> is only added to the generated header DerivedSources/WebKit2/WebCookieManagerMessages.h. WebCookieManager.cpp needs that include, but it doesn't include it explicitly becase it already includes WebCookieManagerMessages.h. Our forwarding header generator doesn't generate the forwarding header in this case. To fix it we only need to make sure the header is included in a non-generated source file, in this case WebCookieManager.cpp. We could probably change the generator to also scan generated sources, ensuring the script is always run after all generated sources during the build. I'm not sure how easy that would be, though.
Comment on attachment 304160 [details] Patch for landing Clearing flags on attachment: 304160 Committed r213759: <http://trac.webkit.org/changeset/213759>
All reviewed patches have been landed. Closing bug.
(In reply to comment #8) > This is a known issue with our forwarding headers generator. It doesn't scan > generated sources, so in this case #include <WebCoreCookie.h> is only added > to the generated header DerivedSources/WebKit2/WebCookieManagerMessages.h. > WebCookieManager.cpp needs that include, but it doesn't include it > explicitly becase it already includes WebCookieManagerMessages.h. Our > forwarding header generator doesn't generate the forwarding header in this > case. To fix it we only need to make sure the header is included in a > non-generated source file, in this case WebCookieManager.cpp. We could > probably change the generator to also scan generated sources, ensuring the > script is always run after all generated sources during the build. I'm not > sure how easy that would be, though. Is there a bug report for this...?
(In reply to comment #9) > Comment on attachment 304160 [details] > Patch for landing > > Clearing flags on attachment: 304160 > > Committed r213759: <http://trac.webkit.org/changeset/213759> It broke the Apple Mac cmake build.
Just to document, the Windows build was fixed in bug169546