Refactor in components/script/dom/htmlanchorelement.rs to

[shuky19]

Thank you for contributing to Servo! Please replace each [ ] by [X] when the step is complete, and replace __ with appropriate data:

• ./mach build -d does not report any errors
• ./mach test-tidy --faster does not report any errors
• These changes fix #11280 (github issue number if applicable).

Either:

• There are tests for these changes OR
• These changes do not require tests because test are already exists and passed

Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process.

Refactor in components/script/dom/htmlanchorelement.rs to make update_href accept &Url argument

---

This change is

[highfive]

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @Ms2ger (or someone else) soon.

[highfive]

Heads up! This PR modifies the following files:

• @KiChjang: components/script/dom/htmlanchorelement.rs

[highfive]

Warning

• These commits modify script code, but no tests are modified. Please consider adding a test!

[KiChjang]

I'm sorry, but there isn't any consensus on whether this PR is what we need yet.

[metajack]

Was there discussion of this somewhere? If so, that discussion should be summarized here.

[KiChjang]

The issue referenced (#11280) has discussions that are waiting an answer from @jdm.

[metajack]

Sorry, I should have followed the reference. Thanks for clarifying.

[jdm]

Sorry, we're going to need to sit on these changes until we resolve the confusion in #11281.

[jdm]

Nobody ever replied in #11281. I don't think there's any point in leaving this open until there's some movement there, unfortunately.