Closed Bug 967259 Opened 11 years ago Closed 11 years ago

Intermittent Jetpack addon-page/main.test that add-on page has no chrome | chrome is not visible for addon page - false == true

Categories

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

x86
Linux
defect
Not set
normal

Tracking

(firefox28 unaffected, firefox29 affected, firefox30 fixed, firefox-esr24 unaffected)

RESOLVED FIXED
Tracking Status
firefox28 --- unaffected
firefox29 --- affected
firefox30 --- fixed
firefox-esr24 --- unaffected

People

(Reporter: RyanVM, Assigned: evold)

References

Details

(Keywords: intermittent-failure)

https://tbpl.mozilla.org/php/getParsedLog.php?id=34022681&tree=Mozilla-Aurora Ubuntu VM 12.04 mozilla-aurora pgo test jetpack on 2014-02-03 13:35:27 PST for push afad6b72ed8d slave: tst-linux32-spot-170 TEST-START | addon-page/main.test add-on page deprecation message TEST-PASS | addon-page/main.test add-on page deprecation message | only one error is dispatched TEST-PASS | addon-page/main.test add-on page deprecation message | the console message is an error TEST-PASS | addon-page/main.test add-on page deprecation message | The message is deprecation message TEST-END | addon-page/main.test add-on page deprecation message TEST-START | addon-page/main.test that add-on page has no chrome TEST-PASS | addon-page/main.test that add-on page has no chrome | chrome is visible for non addon page TEST-UNEXPECTED-FAIL | addon-page/main.test that add-on page has no chrome | chrome is not visible for addon page - false == true TEST-INFO | Traceback (most recent call last): File "resource://gre/modules/commonjs/sdk/timers.js", line 40, in notify callback.apply(null, args); File "resource://test-addon-page-at-jetpack/addon-page/lib/main.js", line 62, in exports["test that add-on page has no chrome"]/< 'chrome is not visible for addon page'); File "resource://gre/modules/commonjs/sdk/test/assert.js", line 124, in equal operator: "==" File "resource://gre/modules/commonjs/sdk/test/assert.js", line 88, in fail this._log.fail(message); File "resource://gre/modules/commonjs/sdk/deprecated/unit-test.js", line 83, in fail this.console.testMessage(false, false, this.test.name, message); File "resource://gre/modules/commonjs/sdk/test/harness.js", line 521, in testMessage this.trace(); TEST-PASS | addon-page/main.test that add-on page has no chrome | chrome is visible again TEST-PASS | addon-page/main.test that add-on page has no chrome | add-on page tab is closed on unload TEST-END | addon-page/main.test that add-on page has no chrome
The Fennec tests fail for the same reason.
Assignee: nobody → evold
Depends on: 967294
(In reply to Erik Vold [:erikvold] [:ztatic] from comment #1) > The Fennec tests fail for the same reason. I should have said the test fails on Fennec for the same reason.
Depends on: 968621
This should be resolved now.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
The change I made in bug 967294 that should resolve this is to wait for the addon page tab to close before assuming it was closed.
Can we get this fix uplifted to 29 too?
You need to log in before you can comment on or make changes to this bug.