missing setter for input element value

[grundt]

Any idea why the setter in the overridden input element value property is commented out here (line 18)?

The result is that if any script tries to set the value of an input field, it'll throw an error:
Uncaught TypeError: setting getter-only property "value"
(i.e. by not providing a setter, it breaks existing functionality)

I tried simply removing the comment, and it resolved the error and didn't seem to have any ill effect on Leak Inspector.
So, curious if there's a reason why it's been commented out.

leak-inspector/page_scripts/page_script.js

Lines 9 to 20 in 0842396

	document.addEventListener("modifyInputElementSetterGetter", function (e) {
	let inputElement = getElementByXpath(e.detail);
	let realHTMLInputElement = Object.getOwnPropertyDescriptor(
	HTMLInputElement.prototype,
	"value"
	);
	Object.defineProperty(inputElement, "value", {
	enumerable: true,
	configurable: true,
	// set: realHTMLInputElement.set,
	get: function () {
	const elValue = realHTMLInputElement.get.call(this);

[asumansenol]

Thanks @grundt, I haven't noticed that it may throw an error if we do not set this value. I will correct it by removing the comment.

[asumansenol]

Solved with the commit 5d35b5f70e853d0998d8bed565228a5d1e004a3e.