Closed Bug 587182 Opened 11 years ago Closed 11 years ago

Implicit native wrapper in content code assertion

Categories

(Core :: XPConnect, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fennec 2.0a1+ ---

People

(Reporter: azakai, Assigned: mrbkap)

References

Details

Attachments

(1 file)

The patch for bug 550936 now runs into

###!!! ASSERTION: Implicit native wrapper in content code: 'Error', file /builds/slave/tryserver-linux-debug/build/js/src/xpconnect/src/XPCNativeWrapper.cpp, line 388
EnsureLegalActivity [js/src/xpconnect/src/XPCNativeWrapper.cpp:389]

In crashtests and jsreftests (debug builds), all OSes. Perhaps related to bug 582100? Seems to have started recently though.
tracking-fennec: --- → 2.0a1+
Attached patch "Fix"Splinter Review
The assertion is wrong, but it's hard to convince of that. Given that all of the wrappers are getting replaced Real Soon Now (TM), I think we should just weaken this assertion and live with it.
Assignee: nobody → mrbkap
Status: NEW → ASSIGNED
Attachment #465921 - Flags: review?(gal)
Comment on attachment 465921 [details] [diff] [review]
"Fix"

Another week and the code will be gone ...
Attachment #465921 - Flags: review?(gal) → review+
push to e10s.  will land on mc shortly:


http://hg.mozilla.org/projects/electrolysis/rev/0346acf59bc6
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.