Closed Bug 1178709 Opened 10 years ago Closed 9 years ago

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

Categories

(Firefox :: Bookmarks & History, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 43
Tracking Status
firefox41 --- disabled
firefox42 --- disabled
firefox43 --- disabled
firefox-esr38 --- unaffected

People

(Reporter: cbook, Assigned: mak)

References

(Depends on 1 open bug, )

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

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
Attached patch focus patch v1Splinter Review
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]
Status: NEW → RESOLVED
Closed: 9 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.

Attachment

General

Created:
Updated:
Size: