1048422 - dom/inputmethod/mochitest/test_basic.html is perma-failing (The input context type should match. - got text, expected input )

Status: RESOLVED DUPLICATE of bug 1048423

(Core :: DOM: Device Interfaces, defect)

Description

[Ryan VanderMeulen [:RyanVM]]

Looks like this failure started on jgraham's push for bug 1042921. However, it seems highly unlikely that his patch actually caused this failure vs. causing a previously-hidden failure to appear.

So there are two issues here:
1.) The test is perma-failing
2.) The harness isn't failing the run because of it

This bug is on file for fixing the test itself. If a harness fix comes first, I will disable this test for now so that we don't potentially miss other similar failures down the road.

Comment 1

[Ryan VanderMeulen [:RyanVM]]

Bug 1048423 tracks the harness side of this.

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Would have helped if I'd included a log link. The failure is occurring on all platforms.

https://tbpl.mozilla.org/php/getParsedLog.php?id=45164240&tree=Mozilla-Inbound&full=1#error0

03:16:22     INFO -  1609 INFO TEST-START | /tests/dom/inputmethod/mochitest/test_basic.html
03:16:22     INFO -  1610 INFO ++DOMWINDOW == 73 (0x123f76400) [pid = 1348] [serial = 323] [outer = 0x12229b400]
03:16:22     INFO -  System JS : ERROR jar:file:///builds/slave/talos-slave/test/build/application/FirefoxNightlyDebug.app/Contents/MacOS/omni.ja!/components/nsHandlerService.js:891 - NS_ERROR_FAILURE: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIProperties.get]
03:16:22     INFO -  1611 INFO [Child 1354] WARNING: NS_ENSURE_TRUE(mDocShell) failed: file /builds/slave/m-in-osx64-d-00000000000000000/build/embedding/browser/webBrowser/nsWebBrowser.cpp, line 365
03:16:22     INFO -  1612 INFO [Child 1354] WARNING: NS_ENSURE_TRUE(domWindow) failed: file /builds/slave/m-in-osx64-d-00000000000000000/build/embedding/browser/webBrowser/nsDocShellTreeOwner.cpp, line 86
03:16:22     INFO -  1613 INFO [Parent 1348] WARNING: RemoveObserver() called for unregistered observer: file /builds/slave/m-in-osx64-d-00000000000000000/build/hal/Hal.cpp, line 204
03:16:22     INFO -  1614 INFO [Parent 1348] WARNING: RemoveObserver() called for unregistered observer: file /builds/slave/m-in-osx64-d-00000000000000000/build/hal/Hal.cpp, line 204
03:16:22     INFO -  1615 INFO [Parent 1348] WARNING: RemoveObserver() called for unregistered observer: file /builds/slave/m-in-osx64-d-00000000000000000/build/hal/Hal.cpp, line 204
03:16:22     INFO -  1616 INFO [Parent 1348] WARNING: RemoveObserver() called for unregistered observer: file /builds/slave/m-in-osx64-d-00000000000000000/build/hal/Hal.cpp, line 204
03:16:22     INFO -  1617 INFO ###!!! [Child][MessageChannel] Error: Closed channel: cannot send/recv
03:16:22     INFO -  1618 INFO [Child 1354] WARNING: content process _exit()ing: file /builds/slave/m-in-osx64-d-00000000000000000/build/dom/ipc/ContentChild.cpp, line 1482
03:16:22     INFO -  1619 INFO ++DOCSHELL 0x12514a000 == 12 [pid = 1348] [id = 32]
03:16:22     INFO -  1620 INFO ++DOMWINDOW == 74 (0x123f70000) [pid = 1348] [serial = 324] [outer = 0x0]
03:16:22     INFO -  1621 INFO ###################################### forms.js loaded
03:16:22     INFO -  1622 INFO ############################### browserElementPanning.js loaded
03:16:22     INFO -  1623 INFO [Parent 1348] WARNING: Subdocument container has no frame: file /builds/slave/m-in-osx64-d-00000000000000000/build/layout/base/nsDocumentViewer.cpp, line 2515
03:16:22     INFO -  1624 INFO ++DOMWINDOW == 75 (0x124b61c00) [pid = 1348] [serial = 325] [outer = 0x123f70000]
03:16:22     INFO -  1625 INFO [Parent 1348] WARNING: NS_ENSURE_TRUE(mMutable) failed: file /builds/slave/m-in-osx64-d-00000000000000000/build/netwerk/base/src/nsSimpleURI.cpp, line 265
03:16:22     INFO -  1626 INFO ######################## BrowserElementChildPreload.js loaded
03:16:22     INFO -  1627 INFO ++DOMWINDOW == 76 (0x124bd5400) [pid = 1348] [serial = 326] [outer = 0x123f70000]
03:16:22     INFO -  1628 INFO [Parent 1348] WARNING: NS_ENSURE_TRUE(aSelection->GetRangeCount()) failed: file /builds/slave/m-in-osx64-d-00000000000000000/build/editor/libeditor/base/nsEditor.cpp, line 3836
03:16:22     INFO -  1629 INFO [Parent 1348] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file /builds/slave/m-in-osx64-d-00000000000000000/build/editor/libeditor/base/nsEditor.cpp, line 3815
03:16:22     INFO -  1630 INFO [Parent 1348] WARNING: NS_ENSURE_SUCCESS(res, res) failed with result 0x80004005: file /builds/slave/m-in-osx64-d-00000000000000000/build/editor/libeditor/text/nsTextEditRules.cpp, line 441
03:16:22     INFO -  1631 INFO file_test_app.html was loaded.[Parent 1348] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80520012: file /builds/slave/m-in-osx64-d-00000000000000000/build/js/xpconnect/loader/mozJSComponentLoader.cpp, line 966
03:16:22     INFO -  1632 INFO System JS : ERROR resource://gre/modules/XPCOMUtils.jsm:243 - NS_ERROR_FILE_NOT_FOUND: Component returned failure code: 0x80520012 (NS_ERROR_FILE_NOT_FOUND) [nsIXPCComponents_Utils.import]
03:16:22     INFO -  1633 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | inputcontextchange event was fired.
03:16:22     INFO -  1634 INFO TEST-UNEXPECTED-FAIL | /tests/dom/inputmethod/mochitest/test_basic.html | The input context type should match. - got text, expected input
03:16:22     INFO -  TEST-INFO | expected PASS
03:16:22     INFO -  1635 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | The inputType should match.
03:16:22     INFO -  1636 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | The input mode should match.
03:16:22     INFO -  1637 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | The language should match.
03:16:22     INFO -  1638 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | Should get the text around the cursor.
03:16:22     INFO -  1639 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | selectionStart was set successfully.
03:16:22     INFO -  1640 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | selectionEnd was set successfully.
03:16:22     INFO -  1641 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | sendKey should changed the input field correctly.
03:16:23     INFO -  1642 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | deleteSurroundingText finished
03:16:23     INFO -  1643 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | deleteSurroundingText should changed the input field correctly.
03:16:23     INFO -  1644 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | replaceSurroundingText finished
03:16:23     INFO -  1645 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | replaceSurroundingText changed the input field correctly.
03:16:23     INFO -  1646 INFO System JS : WARNING resource://gre/components/nsInputListAutoComplete.js:26 - assignment to undeclared variable entries
03:16:23     INFO -  1647 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | setComposition finished
03:16:23     INFO -  1648 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | endComposition changed the input field correctly.
03:16:23     INFO -  1649 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | onselectionchange fired
03:16:23     INFO -  1650 INFO TEST-PASS | /tests/dom/inputmethod/mochitest/test_basic.html | onsurroundingtextchange fired
03:16:23     INFO -  1651 INFO TEST-OK | /tests/dom/inputmethod/mochitest/test_basic.html | took 1057ms

Comment 3

[Ryan VanderMeulen [:RyanVM]]

Apparently we can in fact can get an intermittent orange out of this. Setting the bug flags so we can star it from TBPL.

Comment 4

[Legacy TBPL/Treeherder Robot]

KWierso
https://tbpl.mozilla.org/php/getParsedLog.php?id=45196021&tree=Mozilla-Inbound
Windows 7 32-bit mozilla-inbound debug test mochitest-3 on 2014-08-04 13:38:27
revision: 8ca82f7e965c
slave: t-w732-ix-065

117 INFO TEST-UNEXPECTED-FAIL | /tests/dom/inputmethod/mochitest/test_basic.html | The input context type should match. - got text, expected input
Return code: 1

Comment 5

[Legacy TBPL/Treeherder Robot]

KWierso
https://tbpl.mozilla.org/php/getParsedLog.php?id=45202249&tree=B2g-Inbound
Ubuntu VM 12.04 b2g-inbound debug test mochitest-3 on 2014-08-04 15:34:34
revision: 7f81be7db528
slave: tst-linux32-spot-1093

2598 INFO TEST-UNEXPECTED-FAIL | /tests/dom/inputmethod/mochitest/test_basic.html | The input context type should match. - got text, expected input
4367 INFO TEST-UNEXPECTED-FAIL | /tests/dom/media/tests/identity/test_peerConnection_peerIdentity.html | Unexpected callback with = 'Peer Identity mismatch, expected: [e-mail]' at: ["PCW_setRemoteDescription@http://mochi.test:8888/tests/dom/media/tests/mochitest/pc.js:1708:1","PCT_setRemoteDescription@http://mochi.test:8888/tests/dom/media/tests/mochitest/pc.js:804:3","commandsPeerConnection<@http://mochi.test:8888/tests/dom/media/tests/mochitest/templates.js:281:7","_executeNex... [exceeded max length]

Comment 6

[Legacy TBPL/Treeherder Robot]

KWierso
https://tbpl.mozilla.org/php/getParsedLog.php?id=45204747&tree=Mozilla-Inbound
Windows 7 32-bit mozilla-inbound debug test mochitest-3 on 2014-08-04 16:34:19
revision: 34df51f6221e
slave: t-w732-ix-070

117 INFO TEST-UNEXPECTED-FAIL | /tests/dom/inputmethod/mochitest/test_basic.html | The input context type should match. - got text, expected input
Return code: 1

Comment 7

[Legacy TBPL/Treeherder Robot]

bz
https://tbpl.mozilla.org/php/getParsedLog.php?id=45219203&tree=Mozilla-Inbound
Ubuntu VM 12.04 mozilla-inbound debug test mochitest-3 on 2014-08-04 20:34:06
revision: 3d59b6f2a929
slave: tst-linux32-spot-073

2598 INFO TEST-UNEXPECTED-FAIL | /tests/dom/inputmethod/mochitest/test_basic.html | The input context type should match. - got text, expected input
4379 INFO TEST-UNEXPECTED-FAIL | /tests/dom/media/tests/identity/test_peerConnection_peerIdentity.html | Unexpected callback with = 'Peer Identity mismatch, expected: [e-mail]' at: ["PCW_setRemoteDescription@http://mochi.test:8888/tests/dom/media/tests/mochitest/pc.js:1708:1","PCT_setRemoteDescription@http://mochi.test:8888/tests/dom/media/tests/mochitest/pc.js:804:3","commandsPeerConnection<@http://mochi.test:8888/tests/dom/media/tests/mochitest/templates.js:281:7","_executeNex... [exceeded max length]

Comment 8

[Jan Jongboom [:janjongboom] (inactive)]

Let's see if I can repro this locally.

Comment 9

[Legacy TBPL/Treeherder Robot]

:nigelb
https://tbpl.mozilla.org/php/getParsedLog.php?id=45220436&tree=B2g-Inbound
b2g_ubuntu64_vm b2g-inbound opt test mochitest-1 on 2014-08-04 20:32:19
revision: 574bead88b99
slave: tst-linux64-spot-1137

3377 INFO TEST-UNEXPECTED-FAIL | /tests/dom/datastore/tests/test_oop.html | Test timed out.
3398 INFO TEST-UNEXPECTED-FAIL | /tests/dom/datastore/tests/test_oop_events.html | Message from app: KO getDataStores('foo') returns 1 element
3414 INFO TEST-UNEXPECTED-FAIL | /tests/dom/datastore/tests/test_oop_events.html | Test timed out.
3435 INFO TEST-UNEXPECTED-FAIL | /tests/dom/datastore/tests/test_readonly.html | Message from app: KO getDataStores('bar') returns 1 element
3646 INFO TEST-UNEXPECTED-FAIL | /tests/dom/events/test/test_bug489671.html | monitorConsole | number of messages - got 1, expected 3
4283 INFO TEST-UNEXPECTED-FAIL | /tests/dom/inputmethod/mochitest/test_basic.html | The input context type should match. - got text, expected input
19981 INFO NeckoParent::AllocPHttpChannelParent: FATAL error: App does not have permission: KILLING CHILD PROCESS
TEST-UNEXPECTED-TIMEOUT | Shutdown | application timed out after 330 seconds with no output
TEST-UNEXPECTED-FAIL | Shutdown | application terminated with exit code 6
PROCESS-CRASH | Shutdown | application crashed [@ libpthread-2.15.so + 0xbd84]
Return code: 1

Comment 10

[Yuan Xulei [:yxl] (Leave from May 18)]

I think the cause was found in Bug 1048423 Comment 2.

This is a fallout of bug 1042921. We have a TODO reporting "The input context type should match. - got text, expected input", which is recognized as a perma-failing mistakenly.

:chmanchester is solving this with bug 1048423.

Comment 11

[Legacy TBPL/Treeherder Robot]

edmorley
https://tbpl.mozilla.org/php/getParsedLog.php?id=45233299&tree=B2g-Inbound
Ubuntu VM 12.04 x64 b2g-inbound debug test mochitest-3 on 2014-08-05 01:25:28
revision: 72ab55cf78fe
slave: tst-linux64-spot-635

2618 INFO TEST-UNEXPECTED-FAIL | /tests/dom/inputmethod/mochitest/test_basic.html | The input context type should match. - got text, expected input
7329 INFO Assertion failure: mCanRunMessagesSynchronously, at /builds/slave/b2g-in-l64-d-00000000000000000/build/content/media/MediaStreamGraph.cpp:1869
8021 INFO TEST-UNEXPECTED-ERROR | /tests/dom/media/tests/mochitest/test_dataChannel_basicAudio.html | This test left crash dumps behind, but we weren't expecting it to!
PROCESS-CRASH | Main app process exited normally | application crashed [@ mozilla::MediaStreamGraphImpl::AppendMessage(mozilla::ControlMessage*)]
Return code: 1

Comment 12

[Jan Jongboom [:janjongboom] (inactive)]

Attachment: Patch v1

So I don't know why this would ever fail, this test is even called like:

  todo_is(gContext.type, 'input', 'The input context type should match.');

so it should just ignore it, like it does in my b2g desktop mochitest run (adds a TODO: 1). Anyway, the valid values for inputContext.type are 'contenteditable', 'textarea' or 'type' (as seen in https://github.com/mozilla/gecko-dev/blob/master/dom/inputmethod/MozKeyboard.js#L319), so let's just fix the test.

Comment 13

[Jan Jongboom [:janjongboom] (inactive)]

Try on just one platform, as it fails everywhere: https://tbpl.mozilla.org/?tree=Try&rev=31e9ec9cdc10

Comment 14

[Ed Morley [:emorley]]

(In reply to Jan Jongboom [:janjongboom] (Telenor) from comment #12)
> So I don't know why this would ever fail, this test is even called like:
> 
>   todo_is(gContext.type, 'input', 'The input context type should match.');

Recent harness changes mean that this todo_is is incorrectly identified - bug 1048423 has a fix to the harness (see comment 10).

Comment 15

[Yuan Xulei [:yxl] (Leave from May 18)]

Comment on attachment 8467666 [details] [diff] [review]
Patch v1

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

::: dom/inputmethod/mochitest/test_basic.html
@@ +35,5 @@
>        inputmethod_cleanup();
>        return;
>      }
>  
> +    is(gContext.type, 'text', 'The input context type should match.');

Please add a TODO comment here.

Comment 16

[Yuan Xulei [:yxl] (Leave from May 18)]

Jan, we don't need to fix this bug any more as :edmorley's bug 1048423, which is landing, will fix this.

Comment 18

[Jan Jongboom [:janjongboom] (inactive)]

I still don't get why this is a todo though. Are we going to be changing gContext.type at some point?

Comment 19

[Yuan Xulei [:yxl] (Leave from May 18)]

(In reply to Jan Jongboom [:janjongboom] (Telenor) from comment #18)
> I still don't get why this is a todo though. Are we going to be changing
> gContext.type at some point?

Yes, we want to change gContext.type as the following:

...

interface InputContext: EventTarget {
   // The tag name of input field, which is enum of "input", "textarea", or "contenteditable"
   readonly DOMString type;
...

see https://wiki.mozilla.org/WebAPI/KeboardIME#Proposed_API for details.