Skip to content
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

Extract <input> form data creation into a method on HTMLInputElement #7851

Closed
jdm opened this issue Oct 4, 2015 · 0 comments
Closed

Extract <input> form data creation into a method on HTMLInputElement #7851

jdm opened this issue Oct 4, 2015 · 0 comments

Comments

@jdm
Copy link
Member

@jdm jdm commented Oct 4, 2015

get_form_dataset in HTMLFormElement.rs is long method that's doing a lot of things. Part of the problem is the code for dealing with <input>, which really belongs in a get_form_datum method on HTMLInputElement instead.

Code: components/script/dom/htmlformelement.rs, components/script/dom/htmlinputelement.rs

@jdm jdm added the C-assigned label Oct 6, 2015
bors-servo pushed a commit that referenced this issue Oct 7, 2015
Refactor html form dataset collection

Factor out FormDatum collection for `<input>`
Improve early return logic for getting the FormDatum from an `<input>`
Condense element type patterns

Proposed to close #7851

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7892)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

1 participant
You can’t perform that action at this time.