New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make WebCore::FormData::EncodingType an enum class #6636
Conversation
EWS run on current version of this PR (hash 0f1e3a7) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM if EWS is happy.
The bar for this PR was that all ports compiled (as there is no change in functionality), so marking for merge queue. |
https://bugs.webkit.org/show_bug.cgi?id=248086 <rdar://102515978> Reviewed by Chris Dumez. Convert FormData::EncodingType to an enum class so its backing type can be changed to uint8_t. The other changes are to fix the namespace for the enum values. * Source/WebCore/loader/FormSubmission.cpp: (WebCore::FormSubmission::create): * Source/WebCore/platform/network/FormData.h: (WebCore::FormData::EncodingType): * Source/WebCore/platform/network/FormDataBuilder.cpp: (WebCore::FormDataBuilder::addKeyValuePairAsFormData): * Source/WebCore/platform/network/FormDataBuilder.h: Canonical link: https://commits.webkit.org/256852@main
0f1e3a7
to
c965864
Compare
Committed 256852@main (c965864): https://commits.webkit.org/256852@main Reviewed commits have been landed. Closing PR #6636 and removing active labels. |
c965864
0f1e3a7
π§ͺ ios-wk2π§ͺ mac-wk1π§ͺ mac-AS-debug-wk2