Closed
Bug 1237807
Opened 9 years ago
Closed 9 years ago
Intermittent e10s test_websocket.html | Test timed out.
Categories
(Core :: DOM: Workers, defect)
Core
DOM: Workers
Tracking
()
RESOLVED
FIXED
mozilla47
People
(Reporter: KWierso, Assigned: baku)
References
(Blocks 1 open bug)
Details
(Keywords: intermittent-failure)
Attachments
(5 files)
11.82 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
11.45 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
8.41 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
10.88 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
48.85 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
Assignee | ||
Comment 2•9 years ago
|
||
It happened just once. And that is a big test, maybe something else was slowing down the mochitests.
I want to see some of these before working on it.
Flags: needinfo?(amarchesini)
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
Comment 4•9 years ago
|
||
Intermittent failures on opt builds really can't be a duplicate of the bug which disabled the test on debug builds, and baku isn't likely to see any more of them if it's resolved as a duplicate, and thus hidden away in the junk suggestions.
https://treeherder.mozilla.org/logviewer.html#?job_id=3720144&repo=b2g-inbound
There, there's another.
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Updated•9 years ago
|
Summary: Intermittent test_websocket.html | Test timed out. → Intermittent e10s test_websocket.html | Test timed out.
Version: 45 Branch → Trunk
Comment hidden (Intermittent Failures Robot) |
Updated•9 years ago
|
Blocks: e10s-tests
tracking-e10s:
--- → +
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 12•9 years ago
|
||
Looks like OSX is also affected by this bug. Can we try giving this the rr chaos-mode treatment and see if anything turns up?
Flags: needinfo?(amarchesini)
Comment 13•9 years ago
|
||
Also leaks when it fails on OSX:
TEST-UNEXPECTED-FAIL | leakcheck | tab process: 6760 bytes leaked (ChannelEvent, ChannelEventQueue, ListenerAndContextContainer, LoadInfo, Mutex, ...)
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → amarchesini
Flags: needinfo?(amarchesini)
Assignee | ||
Comment 15•9 years ago
|
||
The current test_websocket.html is very complex: it contains 47 tests, often running together. What I propose here is to split that test in multiple mochitests (10 tests per mochitest), using a promise chain to manage them.
Attachment #8726762 -
Flags: review?(ehsan)
Assignee | ||
Comment 16•9 years ago
|
||
Attachment #8726764 -
Flags: review?(ehsan)
Assignee | ||
Comment 17•9 years ago
|
||
Attachment #8726765 -
Flags: review?(ehsan)
Assignee | ||
Comment 18•9 years ago
|
||
Attachment #8726766 -
Flags: review?(ehsan)
Assignee | ||
Comment 19•9 years ago
|
||
Attachment #8726767 -
Flags: review?(ehsan)
Updated•9 years ago
|
Attachment #8726762 -
Flags: review?(ehsan) → review+
Updated•9 years ago
|
Attachment #8726764 -
Flags: review?(ehsan) → review+
Updated•9 years ago
|
Attachment #8726765 -
Flags: review?(ehsan) → review+
Updated•9 years ago
|
Attachment #8726766 -
Flags: review?(ehsan) → review+
Updated•9 years ago
|
Attachment #8726767 -
Flags: review?(ehsan) → review+
Comment 20•9 years ago
|
||
Comment on attachment 8726762 [details] [diff] [review]
ws1.patch
Review of attachment 8726762 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/base/test/websocket_helpers.js
@@ +33,5 @@
> + ok(false, "onerror called on test " + e.target._testNumber + "!");
> + }
> +
> + ws.addEventListener("close", function(e) {
> + ws._receivedCloseEvent = true;
Seems like this flag is not being used after your patches any more...
Comment 21•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/27bf11a167ff
https://hg.mozilla.org/integration/mozilla-inbound/rev/56741b76ccf7
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a294e361e63
https://hg.mozilla.org/integration/mozilla-inbound/rev/a06ac8e3220a
https://hg.mozilla.org/integration/mozilla-inbound/rev/b5c2b685c260
Comment 22•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/27bf11a167ff
https://hg.mozilla.org/mozilla-central/rev/56741b76ccf7
https://hg.mozilla.org/mozilla-central/rev/9a294e361e63
https://hg.mozilla.org/mozilla-central/rev/a06ac8e3220a
https://hg.mozilla.org/mozilla-central/rev/b5c2b685c260
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•