Summary: | Remove inefficient use of HashMap from FormController along with a bit of other streamlining and modernizing | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Cameron McCormack (:heycam) <heycam> | ||||
Component: | Forms | Assignee: | Darin Adler <darin> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cdumez, changseok, darin, esprehn+autocc, ews-watchlist, gyuyoung.kim, mifenton, webkit-bug-importer, wenson_hsieh | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Local Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Cameron McCormack (:heycam)
2021-08-05 19:24:18 PDT
I’ll make those changes myself. Let me know if you’d prefer to do them or already started. I’m a bit concerned that there may not be enough test coverage for form saving and restoring. I have not started so feel free! Created attachment 436053 [details]
Patch
Passing all the tests and ready for review Comment on attachment 436053 [details]
Patch
r=me
Committed r281505 (240878@main): <https://commits.webkit.org/240878@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 436053 [details]. |