[Webkit-unassigned] [Bug 79096] [GTK] fast/mediastream/peerconnection-argument-types.html fails

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Feb 21 19:43:00 PST 2012


https://bugs.webkit.org/show_bug.cgi?id=79096


Yanbin <yanbin.zhang at intel.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |abarth at webkit.org




--- Comment #2 from Yanbin <yanbin.zhang at intel.com>  2012-02-21 19:43:00 PST ---
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

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list