1089184 - Reenable test_user_select.html

Status: NEW

(Core :: DOM: Selection, defect, P5)

Description

[poiru]

Bug 946065 causes test_user_select.html to start failing on Android 4.0 after being moved from mochitest-1 to mochitest-2.

14:35:43     INFO -  227 INFO TEST-START | /tests/dom/base/test/test_moz_user_select.html
...
/test/test_moz_user_select.html | test9: range[0].endContainer
14:35:43     INFO -  320 INFO TEST-PASS | /tests/dom/base/test/test_moz_user_select.html | test9: range[0].endOffset
14:35:43     INFO -  321 INFO TEST-PASS | /tests/dom/base/test/test_moz_user_select.html | test2: range[0].toString()
14:35:43     INFO -  322 INFO TEST-PASS | /tests/dom/base/test/test_moz_user_select.html | test2: selected text
14:35:43     INFO -  323 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test2: range[0].toString() - got aaaaaaa, expected bbbbbbbbcc
14:35:43     INFO -  324 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test2: selected text - got , expected bbbbbbbbcc
14:35:43     INFO -  325 INFO TEST-PASS | /tests/dom/base/test/test_moz_user_select.html | test2: Selection range count
14:35:43     INFO -  326 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test2: range[0].startContainer - got [object HTMLElement], expected [object HTMLDivElement]
14:35:43     INFO -  327 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test2: range[0].startOffset - got 0, expected 1
14:35:43     INFO -  328 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test2: range[0].endContainer - got [object HTMLElement], expected [object Text]
14:35:43     INFO -  329 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test2: range[0].endOffset - got 1, expected 10
14:35:43     INFO -  330 INFO TEST-PASS | /tests/dom/base/test/test_moz_user_select.html | test3: range[0].toString()
14:35:43     INFO -  331 INFO TEST-PASS | /tests/dom/base/test/test_moz_user_select.html | test3: selected text
14:35:43     INFO -  332 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test3: range[0].toString() - got ccccccc, expected aaaaaabbbbbbbb
14:35:43     INFO -  333 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test3: selected text - got , expected aaaaaabbbbbbbb
14:35:43     INFO -  334 INFO TEST-PASS | /tests/dom/base/test/test_moz_user_select.html | test3: Selection range count
14:35:43     INFO -  335 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test3: range[0].startContainer - got [object HTMLElement], expected [object Text]
14:35:43     INFO -  336 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test3: range[0].startOffset - got 0, expected 1
14:35:43     INFO -  337 INFO TEST-UNEXPECTED-FAIL | /tests/dom/base/test/test_moz_user_select.html | test3: range[0].endContainer - got [object HTMLElement], expected [object HTMLDivElement]
14:35:43     INFO -  338 INFO TEST-OK | /tests/dom/base/test/test_moz_user_select.html | took 3636ms

See: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=ec212fb7da7c

Comment 1

[poiru]

https://hg.mozilla.org/integration/mozilla-inbound/rev/8894caf6427a

I took the liberty of disabling this test on Android since it was blocking bug 946065. Waiting for this bug to be resolved would have probably resulted in a new set of flaky tests popping up.

Mats, let me know if have any idea why these failures might occur.

Also, the try push in comment 0 contained some additional testing changes to test_user_select.html (like renaming it to test_moz_user_select.html). Here is a try push without those changes: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=b306b3c93a15

Comment 2

[Mats Palmgren (inactive)]

Weird, it passes most of the tests, but starts failing at the Shift+click ones:
http://mxr.mozilla.org/mozilla-aurora/source/content/base/test/test_user_select.html?force=1#196
and it only runs the "test2" and "test3" and then bails?

If I remove all other tests in dom/base/test/mochitest.ini then it's green (in M1 this time):
https://tbpl.mozilla.org/?tree=Try&rev=a36ddbe301ec

I'm guessing one of the tests in dom/base/test/mochitest.ini that is running before
test_user_select.html is causing this failure somehow.  If someone else does the bisecting
to figure out which one(s) I can take a look again.

Comment 3

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/8894caf6427a

Comment 4

[BugBot [:suhaib / :marco]]

The leave-open keyword is there and there is no activity for 6 months.
:hsinyi, maybe it's time to close this bug?

Comment 5

[Hsin-Yi Tsai (she/her) [:hsinyi]]

The test is still disabled on android, so the bug is still valid.

Comment 6

[BugBot [:suhaib / :marco]]

The leave-open keyword is there and there is no activity for 6 months.
:hsinyi, maybe it's time to close this bug?