Intermittent browser_bookmarksProperties.js | application timed out after 330 seconds with no output

RESOLVED FIXED in Firefox 43

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: cbook, Assigned: mak)

Tracking

(Depends on 1 bug, {intermittent-failure})

unspecified
Firefox 43
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox41 disabled, firefox42 disabled, firefox43 disabled, firefox-esr38 unaffected)

Details

()

Attachments

(1 attachment)

https://treeherder.mozilla.org/logviewer.html#?job_id=3595617&repo=fx-team

13:45:58 WARNING - TEST-UNEXPECTED-TIMEOUT | browser/components/places/tests/browser/browser_bookmarksProperties.js | application timed out after 330 seconds with no output
Another Start Screen failure.
The recent spike was due to bug 1192831, now backed out.
Mak, I think you made it mad :(
Flags: needinfo?(mak77)
Summary: Intermittent /browser_bookmarksProperties.js | application timed out after 330 seconds with no output → Intermittent browser_bookmarksProperties.js | application timed out after 330 seconds with no output
I'd suggest to disable the test on Windows for now, we'd keep testing on the other platforms.
Then I can split out the other tests to better identify the issue.
Flags: needinfo?(mak77) → needinfo?(ryanvm)
I think it's only windows 8 so far. I added some tracing at https://hg.mozilla.org/try/rev/07a12a5c37a6 (and disabled couple of tests to increase rate(, during failure log has:

{"status": "PASS", "thread": "", "subtest": "We have a places node selected: Unsorted Bookmarks", "pid": null, "source": "mochitest", "test": "browser/components/places/tests/browser/browser_bookmarksProperties.js", "js_source": "TestRunner", "time": 1440574590593, "action": "test_status"}
{"status": "PASS", "thread": "", "subtest": " command 'placesCmd_show:info' on current selected node is enabled", "pid": null, "source": "mochitest", "test": "browser/components/places/tests/browser/browser_bookmarksProperties.js", "js_source": "TestRunner", "time": 1440574590593, "action": "test_status"}
{"status": "PASS", "thread": "", "subtest": "windowObserver: domwindowopened", "pid": null, "source": "mochitest", "test": "browser/components/places/tests/browser/browser_bookmarksProperties.js", "js_source": "TestRunner", "time": 1440574590596, "action": "test_status"}
{"status": "PASS", "thread": "", "subtest": "isFocused (before): false", "pid": null, "source": "mochitest", "test": "browser/components/places/tests/browser/browser_bookmarksProperties.js", "js_source": "TestRunner", "time": 1440574590596, "action": "test_status"}
{"thread": "", "level": "INFO", "pid": null, "source": "mochitest", "js_source": "TestRunner", "time": 1440574634490, "action": "log", "exc_info": false, "message": "Console message: [JavaScript Error: \"1440574634452\tToolkit.Telemetry\tERROR\tTelemetryStorage::loadAbortedSessionPing - error removing ping: PingReadError JS Stack trace: PingReadError@TelemetryStorage.jsm:80:15 < TelemetryStorageImpl.loadPingFile<@TelemetryStorage.jsm:1448:13 < T

That means the focus event is never generated after domwindowopened at http://mxr.mozilla.org/mozilla-central/source/browser/components/places/tests/browser/browser_bookmarksProperties.js#408
Blocks: 1192831
Depends on: 1160326
(In reply to Yury Delendik (:yury) from comment #149)
> That means the focus event is never generated after domwindowopened at
> http://mxr.mozilla.org/mozilla-central/source/browser/components/places/
> tests/browser/browser_bookmarksProperties.js#408

Good catch, this should likely use waitForFocus, cause the focus event could have been sent already, the new tests I splitted out do that.
I may try changing this detail and see how it behaves on Try.
Assignee: nobody → mak77
so far this seems successful on Try, some retriggers are still running though.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=27c2cd85a2a0
Attachment #8653055 - Flags: review?(gijskruitbosch+bugs)
Comment on attachment 8653055 [details] [diff] [review]
focus patch v1

Review of attachment 8653055 [details] [diff] [review]:
-----------------------------------------------------------------

rs=me if retriggers continue to confirm this works.
Attachment #8653055 - Flags: review?(gijskruitbosch+bugs) → review+
all the retriggers were green, or failed on other intermittents. let's see.
Whiteboard: [test disabled on Win8][leave open]
https://hg.mozilla.org/mozilla-central/rev/286c0d72ae55
Status: NEW → RESOLVED
Closed: 4 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
You need to log in before you can comment on or make changes to this bug.