RESOLVED FIXED 193607
C strings in ClientCertificateAuthenticationXPCConstants.h are duplicated
https://bugs.webkit.org/show_bug.cgi?id=193607
Summary C strings in ClientCertificateAuthenticationXPCConstants.h are duplicated
David Kilzer (:ddkilzer)
Reported 2019-01-18 21:18:40 PST
C strings in ClientCertificateAuthenticationXPCConstants.h are duplicated in each source file that includes this header: $ egrep -l -r 'ClientCertificateAuthenticationXPCConstants.h[">]' Source/WebKit Source/WebKit/UIProcess/Authentication/cocoa/AuthenticationChallengeProxyCocoa.mm Source/WebKit/Shared/Authentication/cocoa/AuthenticationManagerCocoa.mm We should move the string constants to their own source file so they aren't duplicated.
Attachments
Patch v1 (8.86 KB, patch)
2019-01-18 21:35 PST, David Kilzer (:ddkilzer)
no flags
Patch v2 (8.86 KB, patch)
2019-01-18 22:38 PST, David Kilzer (:ddkilzer)
no flags
Patch v3 (14.89 KB, patch)
2019-01-19 09:17 PST, David Kilzer (:ddkilzer)
achristensen: review+
Patch v4 (for landing) (14.82 KB, patch)
2019-01-21 12:32 PST, David Kilzer (:ddkilzer)
no flags
David Kilzer (:ddkilzer)
Comment 1 2019-01-18 21:18:52 PST
David Kilzer (:ddkilzer)
Comment 2 2019-01-18 21:35:14 PST
Created attachment 359585 [details] Patch v1
David Kilzer (:ddkilzer)
Comment 3 2019-01-18 22:38:29 PST
Created attachment 359588 [details] Patch v2
David Kilzer (:ddkilzer)
Comment 4 2019-01-19 09:17:08 PST
Created attachment 359612 [details] Patch v3
Alex Christensen
Comment 5 2019-01-21 08:55:49 PST
Comment on attachment 359612 [details] Patch v3 View in context: https://bugs.webkit.org/attachment.cgi?id=359612&action=review > Source/WebKit/Shared/Authentication/cocoa/ClientCertificateAuthenticationXPCConstants.h:32 > +struct ClientCertificateAuthentication { Isn't this more of a namespace?
David Kilzer (:ddkilzer)
Comment 6 2019-01-21 09:56:49 PST
Comment on attachment 359612 [details] Patch v3 View in context: https://bugs.webkit.org/attachment.cgi?id=359612&action=review >> Source/WebKit/Shared/Authentication/cocoa/ClientCertificateAuthenticationXPCConstants.h:32 >> +struct ClientCertificateAuthentication { > > Isn't this more of a namespace? Sure, a namespace could be used instead. Would you prefer that?
David Kilzer (:ddkilzer)
Comment 7 2019-01-21 12:32:16 PST
Created attachment 359706 [details] Patch v4 (for landing)
David Kilzer (:ddkilzer)
Comment 8 2019-01-21 12:32:39 PST
Comment on attachment 359706 [details] Patch v4 (for landing) Now with namespace.
David Kilzer (:ddkilzer)
Comment 9 2019-01-22 12:50:13 PST
Note You need to log in before you can comment on or make changes to this bug.