1415797 - Fix failures in dom/network/tests/ with comformant Promise handling

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, defect, P2)

Description

[Bevis Tseng[:bevis][:btseng](Exited)]

This is a follow-up of the try result in bug 1193394 comment 58:
(C2 in Android 4.3 API16+ debug build: https://treeherder.mozilla.org/logviewer.html#?job_id=143221100&repo=try&lineNumber=2488)
(C1 in Android 4.3 API16+ opt build)

102 INFO TEST-UNEXPECTED-FAIL | dom/network/tests/test_tcpsocket_client_and_server_basics.html | The client should get a close event when it closes itself. - got "error", expected "close"
174 INFO TEST-UNEXPECTED-FAIL | dom/network/tests/test_tcpsocket_jsm.html | The client should get a close event when it closes itself. - got "error", expected "close"

Comment 1

[Bevis Tseng[:bevis][:btseng](Exited)]

It's failed on all platform instead. Update the summary properly.

Comment 2

[Bevis Tseng[:bevis][:btseng](Exited)]

Take this bug to follow up.

Comment 3

[Bevis Tseng[:bevis][:btseng](Exited)]

Attachment: (v1) Patch: Fix failures in dom/network/tests/ with comformant Promise handling.

I'd like to fix it from the implementation instead of the test case because, from the API perspective, it shall be reasonable to close a socket immediately without an error right after it was opened successfully:
https://searchfox.org/mozilla-central/rev/b0098afaeaad24a6752aa418ca3e86eade5fab17/dom/network/tests/test_tcpsocket_client_and_server_basics.js#311-324

BTW, TCPSocket seems not part of HTML standard and the only use case is only available in ppapi:
https://searchfox.org/mozilla-central/rev/b0098afaeaad24a6752aa418ca3e86eade5fab17/browser/extensions/mortar/host/common/ppapi-runtime.jsm#1200

(Ask for your review of this change because the owner of this module seems no longer available.)

Try result looks fine with/without the new microtask scheduling:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=1e011c7b00e8782af07f421f32bfef144df565ce
https://treeherder.mozilla.org/#/jobs?repo=try&revision=67a348f553a15c822b984b5d8c1f448011fdddb6

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8936700 [details] [diff] [review]
(v1) Patch: Fix failures in dom/network/tests/ with comformant Promise handling.

Interesting. Hmm, did I have this in some of my patches and forgot to upload.

Comment 5

[Bevis Tseng[:bevis][:btseng](Exited)]

Here is what you did in your wip patch:
   // -- Client closes the connection
   clientSocket.close();
   is(clientSocket.readyState, 'closing',
      'client readyState should be losing immediately after calling close');
 
+  is((yield clientQueue.waitForEvent()).type, 'error',
+     'The client should get a error event.');
   is((yield clientQueue.waitForEvent()).type, 'close',
      'The client should get a close event when it closes itself.');

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a0dce3b206fa
Fix failures in dom/network/tests/ with comformant Promise handling. r=smaug

Comment 7

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/a0dce3b206fa