[Webkit-unassigned] [Bug 70117] Expose HTMLCanvasElement supportsContext

[bugzilla-daemon at webkit.org]

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





--- Comment #66 from Kenneth Russell <kbr at google.com>  2013-06-06 13:29:57 PST ---
(In reply to comment #64)
> (In reply to comment #63)
> > (In reply to comment #62)
> > > (From update of attachment 203895 [details] [details] [details])
> > > View in context: https://bugs.webkit.org/attachment.cgi?id=203895&action=review
> > > 
> > > This is WIP patch (I need to add and polish the tests for testing well-formed and malformed attributes).
> > 
> > The right thing to do in that case is not to set either r? or r-. r- is used by a reviewer to indicate the patch has been rejected.
> 
> Is there a way to upload a patch without setting the r field?
> 
> Also, how should supportsContext('webkit-3d', {alpha: 'bob'}) be handled? Should supportsContext be checking that the values of the dictionary are really booleans (before JS conversion)?

This behavior is defined by the Web IDL and ECMAScript specs. See in particular http://dev.w3.org/2006/webapi/WebIDL/#es-boolean . I don't think any exception should be thrown in this case; the String should be coerced into a boolean using ECMAScript's ToBoolean abstract operation.

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