Bug 182751

Summary: [Curl] Curl Cookie Database File should be configurable using NetworkProcessCreationParameters
Product: WebKit Reporter: Christopher Reid <chris.reid>
Component: PlatformAssignee: 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
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch for landing none

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 1 Christopher Reid 2018-02-13 16:49:24 PST
Created attachment 333746 [details]
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] [4]
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]

Fixing style issues
Comment 4 youenn fablet 2018-02-21 13:22:32 PST
Comment on attachment 333748 [details]

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.
Comment 11 Radar WebKit Bug Importer 2018-02-21 16:11:21 PST