Closed Bug 587182 Opened 15 years ago Closed 15 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+
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: