The default bug view has changed. See this FAQ.

"ASSERTION: We forgot to Morph a slim wrapper!" with Components.lookupMethod

RESOLVED FIXED in Firefox 17

Status

()

Core
XPConnect
--
critical
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: Jesse Ruderman, Unassigned)

Tracking

(Blocks: 1 bug, {assertion, regression, testcase})

Trunk
mozilla17
x86_64
Mac OS X
assertion, regression, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox17+ verified)

Details

Attachments

(3 attachments)

(Reporter)

Description

5 years ago
Created attachment 644734 [details]
testcase (asserts fatally when loaded)

###!!! ASSERTION: We forgot to Morph a slim wrapper!: 'clasp->ext.innerObject || IS_WN_WRAPPER_OBJECT(obj)', file js/xpconnect/wrappers/WrapperFactory.cpp, line 283

Assertion failure: (DebugCheckWrapperClass(obj) && !js::GetReservedSlot(obj, 0).isDouble()), at js/xpconnect/wrappers/XrayWrapper.cpp:440

Might be a regression from bug 774245.
(Reporter)

Comment 1

5 years ago
Created attachment 644735 [details]
stack traces
Blocks: 774245
tracking-firefox17: --- → ?
Duplicate of this bug: 776328
Created attachment 644916 [details] [diff] [review]
Morph slim wrappers in LookupMethod. v1

We're not going through PrepareForUnwrapping, so we need to manually morph slim wrappers.
Attachment #644916 - Flags: review?(mrbkap)
tracking-firefox17: ? → +

Updated

5 years ago
Attachment #644916 - Flags: review?(mrbkap) → review+
pushed to m-i:
http://hg.mozilla.org/integration/mozilla-inbound/rev/8eaf364324ce
https://hg.mozilla.org/mozilla-central/rev/8eaf364324ce
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17

Updated

5 years ago
status-firefox17: --- → fixed
Keywords: verifyme
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:17.0) Gecko/17.0 Firefox/17.0

Verified the fix on latest Firefox 17 beta debug build on Mac Os X 10.7.5 and doesn't assert.
Marking verified for Firefox 17.
status-firefox17: fixed → verified
mass remove verifyme requests greater than 4 months old
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.