|Summary:||[Curl] Curl Cookie Database File should be configurable using NetworkProcessCreationParameters|
|Product:||WebKit||Reporter:||Christopher Reid <chris.reid>|
|Component:||Platform||Assignee:||Christopher Reid <chris.reid>|
|Severity:||Normal||CC:||achristensen, annulen, beidson, bfulgham, cdumez, chris.reid, commit-queue, darin, don.olmstead, ews-watchlist, galpeter, webkit-bug-importer, youennf|
|Version:||WebKit Nightly Build|
Description Christopher Reid 2018-02-13 16:22:00 PST
In preparation for WinCairo WebKit2, the curl should follow the other ports and allow the cookie file to be configurable through the network process creation parameters.
Comment 2 EWS Watchlist 2018-02-13 16:50:59 PST
Attachment 333746 [details] did not pass style-queue: ERROR: Source/WebKit/NetworkProcess/curl/NetworkProcessCurl.cpp:50: One line control clauses should not use braces. [whitespace/braces]  Total errors found: 1 in 11 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Christopher Reid 2018-02-13 16:56:23 PST
Created attachment 333748 [details] patch Fixing style issues
Comment 4 youenn fablet 2018-02-21 13:22:32 PST
Comment on attachment 333748 [details] patch I do not see anything wrong there related to the 'OWNER' side. I am not very familiar with CURL, it would be good if somebody more familiar could take an additional look. View in context: https://bugs.webkit.org/attachment.cgi?id=333748&action=review > Source/WebKit/ChangeLog:10 > + This is parameter is not used yet, it is added to prepare for WinCairo WebKit support. s/This is/This/
Comment 5 Chris Dumez 2018-02-21 14:20:08 PST
Was there discussion on the mailing list about adding support for Windows to WebKit2?
Comment 6 Christopher Reid 2018-02-21 14:21:23 PST
Created attachment 334413 [details] Patch for landing
Comment 7 Christopher Reid 2018-02-21 14:43:41 PST
(In reply to Chris Dumez from comment #5) > Was there discussion on the mailing list about adding support for Windows to > WebKit2? I don't think there was too much discussion about adding WebKit 2 support for WinCairo on the mailing list yet. Don has mentioned that was something we were working in one of our updates here: https://lists.webkit.org/pipermail/webkit-dev/2017-November/029783.html. We are very much interested in working towards WebKit2 support including getting WebKit2 WinCairo buildbots going. Let us know if you would like to see the discussion opened up on the mailing list some more.
Comment 8 Don Olmstead 2018-02-21 14:49:48 PST
Its actually been longer than that https://lists.webkit.org/pipermail/webkit-dev/2017-August/029329.html
Comment 9 WebKit Commit Bot 2018-02-21 16:10:30 PST
Comment on attachment 334413 [details] Patch for landing Clearing flags on attachment: 334413 Committed r228905: <https://trac.webkit.org/changeset/228905>
Comment 10 WebKit Commit Bot 2018-02-21 16:10:31 PST
All reviewed patches have been landed. Closing bug.