Since checkin in r108299. Consistently. Diff: --- /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/fast/mediastream/peerconnection-argument-types-expected.txt +++ /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/fast/mediastream/peerconnection-argument-types-actual.txt @@ -3,7 +3,7 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE". -PASS typeof webkitPeerConnection== 'function' is true +FAIL typeof webkitPeerConnection== 'function' should be true. Was false. PASS new webkitPeerConnection() threw exception TypeError: Not enough arguments. PASS new webkitPeerConnection("STUN 203.0.113.2:2478") threw exception TypeError: Not enough arguments. PASS new webkitPeerConnection("STUN relay.example.net:3478") threw exception TypeError: Not enough arguments. @@ -23,8 +23,8 @@ PASS new webkitPeerConnection("STUNS NONE") threw exception TypeError: Not enough arguments. PASS new webkitPeerConnection("undefined") threw exception TypeError: Not enough arguments. PASS new webkitPeerConnection("null") threw exception TypeError: Not enough arguments. -PASS new webkitPeerConnection({})) threw exception SyntaxError: Unexpected token ). -PASS new webkitPeerConnection(42)) threw exception SyntaxError: Unexpected token ). +FAIL new webkitPeerConnection({})) should throw SyntaxError: Unexpected token ). Threw exception SyntaxError: Unexpected token ')'. +FAIL new webkitPeerConnection(42)) should throw SyntaxError: Unexpected token ). Threw exception SyntaxError: Unexpected token ')'. PASS new webkitPeerConnection(Infinity) threw exception TypeError: Not enough arguments. PASS new webkitPeerConnection(-Infinity) threw exception TypeError: Not enough arguments. PASS new webkitPeerConnection(emptyFunction) threw exception TypeError: Not enough arguments. @@ -45,7 +45,7 @@ PASS new webkitPeerConnection("STUN NONE",emptyFunction) did not throw exception. PASS new webkitPeerConnection("STUNS NONE",emptyFunction) did not throw exception. PASS new webkitPeerConnection("TURN NONE",undefined) threw exception Error: TYPE_MISMATCH_ERR: DOM Exception 17. -PASS new webkitPeerConnection("TURNS NONE",{}) did not throw exception. +FAIL new webkitPeerConnection("TURNS NONE",{}) should not throw exception. Threw exception Error: TYPE_MISMATCH_ERR: DOM Exception 17 PASS new webkitPeerConnection("STUN NONE",42) threw exception Error: TYPE_MISMATCH_ERR: DOM Exception 17. PASS new webkitPeerConnection("STUNS NONE",Infinity) threw exception Error: TYPE_MISMATCH_ERR: DOM Exception 17. PASS new webkitPeerConnection("STUNS NONE",-Infinity) threw exception Error: TYPE_MISMATCH_ERR: DOM Exception 17.
I haven't checked, but could the current test results be V8-specific?
Those test cases are defined according to http://www.w3.org/TR/webrtc/ For failed test cases: new webkitPeerConnection("TURNS NONE",{}) I looked into this failure and think it is a bug for V8. Test case should fail explicitly by throwing an exception, otherwise the code should not run correctly in any case. According to the WebIDL spec for “Callback types” (http://dev.w3.org/2006/webapi/WebIDL/#es-callback): If V is not a Function object, then throw a TypeError. Hi, Adam, Do we need to report a bug entry for this issue for V8 and modify test case? And, I am very sorry to make a slight mistake for following two test cases definition: PASS new webkitPeerConnection({})) threw exception SyntaxError: Unexpected token ). PASS new webkitPeerConnection(42)) threw exception SyntaxError: Unexpected token ) It should be PASS new webkitPeerConnection({}) threw exception TypeError: Not enough arguments. PASS new webkitPeerConnection(42) threw exception TypeError: Not enough arguments. I will upload a new patch to fix them. While, the test result should not failed for original definition. I am not sure why those test cases were failed. Philippe, I guess you should look into why test case typeof webkitPeerConnection== 'function' is failed. Thanks Yanbin
Created attachment 128131 [details] patch
Comment on attachment 128131 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=128131&action=review > LayoutTests/ChangeLog:9 > + [GTK] fast/mediastream/peerconnection-argument-types.html fails > + https://bugs.webkit.org/show_bug.cgi?id=79096 > + > + Reviewed by NOBODY (OOPS!). > + > + * fast/mediastream/peerconnection-argument-types-expected.txt: > + * fast/mediastream/script-tests/argument-types.js: Can you explain why you're making this change?
There is a mistake for following two test cases definition: PASS new webkitPeerConnection({})) threw exception SyntaxError: Unexpected token ). PASS new webkitPeerConnection(42)) threw exception SyntaxError: Unexpected token ) It should be PASS new webkitPeerConnection({}) threw exception TypeError: Not enough arguments. PASS new webkitPeerConnection(42) threw exception TypeError: Not enough arguments. This patch is to fix them.
Comment on attachment 128131 [details] patch Ok. In general, it's a good idea to explain why you're making a change in the ChangeLog.
You didn't unskip the test either.
Comment on attachment 128131 [details] patch Clearing flags on attachment: 128131 Committed r108460: <http://trac.webkit.org/changeset/108460>
All reviewed patches have been landed. Closing bug.