Bug 185662 - Convert CertificateInfo into Credential in UI Process instead of Networking Process
Summary: Convert CertificateInfo into Credential in UI Process instead of Networking P...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jiewen Tan
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-05-15 16:02 PDT by Jiewen Tan
Modified: 2018-05-17 12:26 PDT (History)
11 users (show)

See Also:

Attachments
Patch (31.56 KB, patch)
2018-05-15 16:52 PDT, Jiewen Tan 		no flags Details | Formatted Diff | Diff
Patch (28.43 KB, patch)
2018-05-15 17:12 PDT, Jiewen Tan 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jiewen Tan 2018-05-15 16:02:53 PDT 
Right now we convert CertificateInfo into Credential in the very last stage of client certificate authentication process when it reaches Networking Process. We could have done that earlier in UI Process such that we don't have to pass both Credential and CertificateInfo to Networking Process.
Comment 1 Radar WebKit Bug Importer 2018-05-15 16:03:40 PDT 
<rdar://problem/40275561>
Comment 2 Jiewen Tan 2018-05-15 16:52:02 PDT 
Created attachment 340447 [details]
Patch
Comment 3 Jiewen Tan 2018-05-15 17:12:22 PDT 
Created attachment 340450 [details]
Patch
Comment 4 Jiewen Tan 2018-05-17 11:59:30 PDT 
Comment on attachment 340450 [details]
Patch

Thanks Alex for r+ this patch.
Comment 5 WebKit Commit Bot 2018-05-17 12:26:57 PDT 
Comment on attachment 340450 [details]
Patch

Clearing flags on attachment: 340450

Committed r231916: <https://trac.webkit.org/changeset/231916>
Comment 6 WebKit Commit Bot 2018-05-17 12:26:59 PDT 
All reviewed patches have been landed.  Closing bug.