Closed
Bug 931116
Opened 11 years ago
Closed 9 years ago
[Tracking] b2g desktop mochitest failures
Categories
(Testing :: Mochitest, defect)
Tracking
(firefox28 fixed, b2g-v1.2 fixed)
RESOLVED
WONTFIX
People
(Reporter: ahal, Unassigned)
References
Details
Attachments
(2 files, 1 obsolete file)
33.52 KB,
patch
|
jgriffin
:
review+
|
Details | Diff | Splinter Review |
60.90 KB,
patch
|
ahal
:
review+
|
Details | Diff | Splinter Review |
There are ~100 failing mochitest failures specific to b2g desktop when I run them locally. I haven't had much luck getting people to help investigate so I'm going to disable them so we can at least get the rest rolled out asap.
Reporter | ||
Comment 1•11 years ago
|
||
Actually turned out to only be 17 tests after multiple failures from single tests were taken into account.
Reporter | ||
Comment 2•11 years ago
|
||
We'll see how it does on Cedar: https://tbpl.mozilla.org/?tree=Cedar&rev=edb9156ddb0a
Reporter | ||
Comment 3•11 years ago
|
||
Morphing this into a tracking bug. Looks like there were many more failures than 100, they were just hidden. Also lots of intermittents that look like they aren't related to any particular test.
Reporter | ||
Comment 4•11 years ago
|
||
This patch green's up Cedar (about 24 in a row now without failures). This patch is a little overzealous since some of the tests were disabled due to bug 924681 which is now fixed. I don't know which tests these were, so I'll file a follow up bug to look into re-enabling them.
Attachment #822453 -
Attachment is obsolete: true
Attachment #826748 -
Flags: review?(jgriffin)
Reporter | ||
Updated•11 years ago
|
Whiteboard: [leave-open]
Comment 5•11 years ago
|
||
Comment on attachment 826748 [details] [diff] [review]
Patch 1.0 - disable initial failing b2g desktop mochitests
Review of attachment 826748 [details] [diff] [review]:
-----------------------------------------------------------------
\o/
Attachment #826748 -
Flags: review?(jgriffin) → review+
Reporter | ||
Comment 6•11 years ago
|
||
Comment 7•11 years ago
|
||
Reporter | ||
Updated•11 years ago
|
Blocks: b2g-mochitests
Reporter | ||
Comment 8•11 years ago
|
||
This is the same set of tests disabled as the last patch, except based off mozilla-b2g26, so carrying r+ forward.
Attachment #828694 -
Flags: review+
Reporter | ||
Comment 9•11 years ago
|
||
status-b2g-v1.2:
--- → fixed
status-firefox28:
--- → fixed
Reporter | ||
Updated•11 years ago
|
Assignee: ahalberstadt → nobody
Status: ASSIGNED → NEW
I kind of ran into a problem that might affect a lot of tests running in b2g desktop. I thought I'd post it here in case it helps anyone.
Lots of tests do synthesizeKeyEvent(key). When that code runs in a content process, the key event is handled normally. When everything runs in one process, the key event usually gets thrown away here:
http://mxr.mozilla.org/mozilla-central/source/b2g/chrome/content/shell.js?rev=8db31fe79a2d#433
B2g desktop uses one process for everything by default, so I suspect that a lot of tests are failing because of this problem.
I ran into this accidentally while fixing some e10s stuff and I haven't looked into it much on the b2g side, so I might have misdiagnosed the issue. But if someone is interested in enabling these tests, it's worth investigating.
All of the indexedDB tests are failing because the test page runs with a principal of "mochi.test" but then the parent window has a principal of the testing app. This looks like a third-party window and indexedDB refuses to load in third-party windows.
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Whiteboard: [leave-open]
You need to log in
before you can comment on or make changes to this bug.
Description
•