<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - REGRESSION(182495): Error's "column" property should not be readonly"
href="https://bugs.webkit.org/show_bug.cgi?id=146047#c4">Comment # 4</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - REGRESSION(182495): Error's "column" property should not be readonly"
href="https://bugs.webkit.org/show_bug.cgi?id=146047">bug 146047</a>
from <span class="vcard"><a class="email" href="mailto:joepeck@webkit.org" title="Joseph Pecoraro <joepeck@webkit.org>"> <span class="fn">Joseph Pecoraro</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=146047#c3">comment #3</a>)
<span class="quote">> > > Object.getOwnPropertyDescriptor(error, "lineNumber")
> > < {value: 1, writable: true, enumerable: true, configurable: true} = $3
>
> Ahh, this is misleading since no such property already existed, this was
> just added by me.
>
> I think the relevant part of the original change was:
>
> + obj->putDirect(vm, vm.propertyNames->line, jsNumber(line), ReadOnly
> | DontDelete);
> + obj->putDirect(vm, vm.propertyNames->column, jsNumber(column),
> ReadOnly | DontDelete);</span >
Actually the original code had something very similar. So I don't actually see how this was writable before... but there is certainly some visible change here.
Other browsers have different property names and descriptor attributes for the error properties. There is no standardization at all with Errors. That said, making the property writable seems reasonable.
Firefox:
js> Object.getOwnPropertyNames(new Error)
Array [ "fileName", "lineNumber", "columnNumber" ]
js> Object.getOwnPropertyDescriptor(new Error, "lineNumber")
Object { value: 1, writable: true, enumerable: false, configurable: true }
Chrome:
js> Object.getOwnPropertyNames(new Error)
["stack"]
js> Object.getOwnPropertyDescriptor(new Error, "stack")
Object {enumerable: false, configurable: true}</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>