Closed
Bug 812146
Opened 12 years ago
Closed 12 years ago
Javascript doesn't honor values set via Foo.prototype.bar
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 783396
People
(Reporter: i-bugtracker-mozilla-org-39ztgfdskjhf, Unassigned)
Details
(Keywords: regression, testcase)
Attachments
(1 file)
395 bytes,
text/html
|
Details |
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/20100101 Firefox/16.0
Build ID: 20121024073032
Steps to reproduce:
Execute the script
Actual results:
hhh.align should have the value 'left'.
Expected results:
Property has undefined value.
Comment 1•12 years ago
|
||
http://software.hixie.ch/utilities/js/live-dom-viewer/saved/1914
Uncommenting the this.__proto__ line fixes the issue.
Assignee: nobody → general
Status: UNCONFIRMED → NEW
Component: Untriaged → JavaScript Engine
Ever confirmed: true
Product: Firefox → Core
Attachment #681950 -
Attachment mime type: text/plain → text/html
Comment 2•12 years ago
|
||
Ms2ger, is this a regression? Regression range?
m-c
good=2012-06-05
bad=2012-06-06
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=a7a905fd70d5&tochange=6338a8988917
m-i
good=2012-06-04
bad=2012-06-05
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=778ba119ded6&tochange=f56e2197d9cd
Comment 4•12 years ago
|
||
Possibly a regression from bug 759978?
tracking-firefox17:
--- → ?
tracking-firefox18:
--- → ?
tracking-firefox19:
--- → ?
Keywords: regressionwindow-wanted
Comment 5•12 years ago
|
||
Hmm. My 2012-10-28 nightly shows correct behavior. So does the 2012-11-14 nightly. I do see the problem in Firefox 16.
Is this fixed on trunk?
Yes, it's fixed in the latest Nightly too (Win 7). So it's likely a dupe. I should have to test the latest nightly... :/
Comment 7•12 years ago
|
||
The test is also misleading in that it hard codes the 'undefined':
alert("Expected 'left'; get 'undefined' in FF 16; result="+hhh.align);
Anyways, this was fixed by bug 767961 (this particular case was reported in bug 783396). Is it a matter of backporting the patch?
Comment 8•12 years ago
|
||
Nope. That bug is fixed in 17, so unless we plan to backport to a 16 chemspill we're all set. Thanks!
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
Updated•12 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•