Closed Bug 1009340 Opened 11 years ago Closed 7 years ago

TEST-UNEXPECTED-FAIL | chrome/main.testChromeInPanel | Test timed out

Categories

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

x86_64
Linux
defect

Tracking

(firefox35 wontfix, firefox36 wontfix, firefox37 fixed, firefox-esr31 wontfix)

RESOLVED INCOMPLETE
mozilla37
Tracking Status
firefox35 --- wontfix
firefox36 --- wontfix
firefox37 --- fixed
firefox-esr31 --- wontfix

People

(Reporter: KWierso, Assigned: evold)

References

Details

(Keywords: intermittent-failure)

Attachments

(2 files)

https://tbpl.mozilla.org/php/getParsedLog.php?id=39508554&tree=Fx-Team Log Ubuntu VM 12.04 fx-team opt test jetpack on 2014-05-12 13:15:20 PDT for push 5fd2e583adc0 slave: tst-linux32-spot-372 TEST-START | chrome/main.testChromeInPanel System JS : ERROR chrome://browser/content/browser.js:14416 - TypeError: value is not a non-null object System JS : ERROR chrome://browser/content/content-sessionStore.js:228 - TypeError: docShell.QueryInterface(...).sessionHistory is null JavaScript error: chrome://browser/content/urlbarBindings.xml, line 672: aUrl is undefined TEST-INFO | [JavaScript Error: "Error adding value null to histogram PLACES_ANNOS_PAGES_SIZE_KB: Error: Not a number" {file: "resource://gre/modules/PlacesDBUtils.jsm" line: 978}] TEST-UNEXPECTED-FAIL | chrome/main.testChromeInPanel | Timed out 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://gre/modules/commonjs/sdk/deprecated/unit-test.js", line 450, in tiredOfWaiting self.console.testMessage(false, false, self.test.name, "Timed out"); File "resource://gre/modules/commonjs/sdk/test/harness.js", line 527, in testMessage this.trace(); TEST-END | chrome/main.testChromeInPanel TEST-START | chrome/main.testChromeLocale
Priority: -- → P1
Blocks: 1020473
Assignee: nobody → evold
Attachment #8491042 - Flags: review?(jsantell) → review+
Commits pushed to master at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/a8ab30119e0288f417b035dafb68715b06fc025a Bug 1009340 logging more information in the chrome test addon https://github.com/mozilla/addon-sdk/commit/267b928313a520c23657106c7161a40a1cf8996e Merge pull request #1623 from erikvold/1009340 Bug 1009340 logging more information in the chrome test addon r=@jsantell
My patch landed, we'll see how that goes..
Assignee: evold → nobody
Summary: Intermittent chrome/main.testChromeInPanel | Timed out → TEST-UNEXPECTED-FAIL | chrome/main.testChromeInPanel | Test timed out
Assignee: nobody → evold
Since my previous patch bug 1083391 was changed, which I think makes this test more reliable, so I am going to revert the change which added the unnecessary setTimeout now.
Depends on: 1083391
Commit pushed to master at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/38796432dad57ee48ef73402d48712a326874a41 Bug 1009340 removing the panel.hide() tests for a panel using chrome b/c it intermittently times out a=me
I was able to reproduce this about 50% of the time on linux with travis.. I just removed the "hide" related code because the test wasn't really meant to test the panel module's hide handle (it apparently works most of the time), but rather that the panel module can show a chrome uri. This seems to work now.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
This is still an issue
Assignee: evold → nobody
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
No longer blocks: 1114752
Assignee: nobody → evold
Attachment #8614205 - Flags: review?(jsantell)
Attachment #8614205 - Flags: review?(jsantell) → review+
Commits pushed to master at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/c8ada29be6a0e99a35c27124b92001fe3343bdd0 Bug 1009340 use contentScriptWhen: end for testChromeInPanel https://github.com/mozilla/addon-sdk/commit/1b982124373bc96da91881aa225b5ded2332d374 Merge pull request #1992 from erikvold/1009340 Bug 1009340 use contentScriptWhen: end for testChromeInPanel r=jsantell
Status: REOPENED → RESOLVED
Closed: 10 years ago7 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: