upvar staticDepth limit missing from BindNameToSlot

RESOLVED FIXED in mozilla1.9.1

Status

()

Core
JavaScript Engine
P1
normal
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: brendan, Assigned: brendan)

Tracking

Trunk
mozilla1.9.1
Points:
---
Bug Flags:
in-testsuite -
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Created attachment 335723 [details] [diff] [review]
patch

Fix attached, based on tracemonkey repo.

/be
Attachment #335723 - Flags: review?(igor)
(Assignee)

Updated

10 years ago
Status: NEW → ASSIGNED
Priority: -- → P1

Comment 1

10 years ago
Comment on attachment 335723 [details] [diff] [review]
patch

Sigh, this mean one more iteration with the bug 448828, but whatever.
Attachment #335723 - Flags: review?(igor) → review+
(Assignee)

Comment 2

10 years ago
Fixed on m-c:

http://hg.mozilla.org/mozilla-central/index.cgi/rev/9ecf699d4047

/be
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Backed out at Brendan's request, to see if it fixes linux mochitest failures.
* Failing tinderbox log:
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1219955155.1219959439.15309.gz#err2
* Backout:
http://hg.mozilla.org/mozilla-central/rev/fe61440b3e42
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Looks like the orange was sporadic -- the next build on that tinderbox was green, and it was using the same code as the orange build. (It started building before my backout)

So, this should be safe to re-land.
Re-landed as http://hg.mozilla.org/mozilla-central/rev/018b8ac99a4c
Status: REOPENED → RESOLVED
Last Resolved: 10 years ago10 years ago
Resolution: --- → FIXED

Updated

10 years ago
Flags: in-testsuite-
Flags: in-litmus-
You need to log in before you can comment on or make changes to this bug.