Closed Bug 967259 Opened 7 years ago Closed 7 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: 7 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.