Closed Bug 751595 Opened 12 years ago Closed 12 years ago

"can't access dead object" exception in api-utils/content/symbiont.js:onStart()

Categories

(Add-on SDK Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

Attachments

(1 file)

We are seing many such exception during tests:
error: An exception occurred.
Traceback (most recent call last):
  File "resource://19947864-b189-492b-9845-7f917eaa2485-at-jetpack/api-utils/lib/system/events.js", line 58, in 
    data: data
  File "resource://19947864-b189-492b-9845-7f917eaa2485-at-jetpack/api-utils/lib/observer-service.js", line 35, in listener
    callback.call(target || callback, subject, data);
  File "resource://19947864-b189-492b-9845-7f917eaa2485-at-jetpack/api-utils/lib/content/symbiont.js", line 140, in onStart
    if (window && window == frame.contentWindow) {
TypeError: can't access dead object

It happens to be related to just one test badly written:
  test-content-symbiont:test:constructing symbiont && validating API

A frame was used but never unregistered from the created Symbiont.
Attached file Pull request 426 (deleted) —
Assignee: nobody → poirot.alex
Attachment #620734 - Flags: review?(rFobic)
Blocks: 749526
Attachment #620734 - Flags: review?(rFobic) → review+
Commits pushed to master at https://github.com/mozilla/addon-sdk

https://github.com/mozilla/addon-sdk/commit/af6fdbe1e71e1a3348077dbb9a29bf88fbb79e04
Bug 751595: Simplify badly written test (test-content-symbiont) that ended up throwing many "dead object exception".

https://github.com/mozilla/addon-sdk/commit/35cf82a91adfde61fc2481fcde0b0a206d5a3c3f
Merge pull request #426 from ochameau/bug/751595-fix-test-content-symbiont-dead-object

Bug 751595: Simplify badly written test (test-content-symbiont) r=@gozala
Status: NEW → RESOLVED
Closed: 12 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: