Closed
Bug 895288
Opened 11 years ago
Closed 11 years ago
Intermittent browser_aboutHome.js | Test timed out | Found a tab after previous test timed out: about:blank
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
FIXED
Firefox 25
People
(Reporter: ttaubert, Assigned: mak)
References
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
1.42 KB,
patch
|
ttaubert
:
review+
|
Details | Diff | Splinter Review |
https://tbpl.mozilla.org/php/getParsedLog.php?id=25411881&tree=Mozilla-Central TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_aboutHome.js | Test timed out TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/browser/base/content/test/browser_aboutHome.js | Found a tab after previous test timed out: about:blank :(
Assignee | ||
Comment 1•11 years ago
|
||
It is over the 60 seconds timeout, could be the box was strangely slow and it took more time, we may bump up the timeout to 90s and see. INFO TEST-END | chrome://mochitests/content/browser/browser/base/content/test/browser_aboutHome.js | finished in 80285ms The timeout fired 2 seconds earlier and the test was doing stuff just before the timeout.
Assignee: nobody → mak77
No longer blocks: 890409
Assignee | ||
Comment 2•11 years ago
|
||
Actually the long time looks due to loading the google search page, it prints a very high amount of console log due to css errors. after 30s we start the health report test, and it alone takes almost 1 minute. We should try to block loading earlier.
Assignee | ||
Comment 3•11 years ago
|
||
also, our timeout handling in b-c should better notify us. In this case it was clear the test was not hanging, cause it just printed a TEST-INFO. It's good to be notified of tests taking too much, but it should not appear as a common timeout. Instead of "timed out" it should tell us "Interrupted due to taking too long time to complete". I think this is feasible, we can track when the last output happened at least.
Comment 4•11 years ago
|
||
(In reply to Marco Bonardo [:mak] from comment #2) > Actually the long time looks due to loading the google search page, it > prints a very high amount of console log due to css errors. Is that possibly related to bug 845205 comment 36? Maybe this failure was triggered only while that patch was in the tree?
Assignee | ||
Comment 5•11 years ago
|
||
No, I don't think so, it is a bunch of css errors in the console, not assertions. Stuff like "invalid background... bla bla bla". Also, health report takes quite some time to do its job, not sure why.
Assignee | ||
Comment 6•11 years ago
|
||
this should improve the time taken by the test, by stopping the load even earlier.
Attachment #778159 -
Flags: review?(ttaubert)
Reporter | ||
Comment 7•11 years ago
|
||
The last statement before the long pause is: 19:10:34 | TEST-PASS | chrome://mochitests/content/browser/browser/base/content/test/browser_aboutHome.js | Searches provider is available. The next one after the pause is: 19:11:27 | TEST-PASS | chrome://mochitests/content/browser/browser/base/content/test/browser_aboutHome.js | One more search recorded. To me this seems like this took almost a minute to complete: provider.getMeasurement("counts", 2).getValues().then(data => { /* do stuff */ }); I don't think this would be fixed by moving gBrowser.stop() around, would it? I have no idea how the healthReporter is implemented but I wonder if maybe something went wrong in the backend?
Assignee | ||
Comment 8•11 years ago
|
||
First of all I want to remove those hundreds very long css warnings from the console, those are also slowing down everything. Locally I don't see any issue in getMeasurement, but I also don't see those css warnings.
Reporter | ||
Comment 9•11 years ago
|
||
Comment on attachment 778159 [details] [diff] [review] home.diff Review of attachment 778159 [details] [diff] [review]: ----------------------------------------------------------------- Ok, fair enough.
Attachment #778159 -
Flags: review?(ttaubert) → review+
Assignee | ||
Comment 10•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/7fb8ea5cd2fc I will take a look at the next logs and check time differences, if still unexpectedly high, I will file a bug to investigate why HR takes so much time.
Target Milestone: --- → Firefox 25
Comment 11•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/7fb8ea5cd2fc
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Comment 12•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/fc5259bc0785
status-firefox24:
--- → fixed
status-firefox25:
--- → fixed
Comment hidden (Legacy TBPL/Treeherder Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•