Update the <canvas> IDLs wrt 'unrestricted'

Update the <canvas> IDLs wrt 'unrestricted'

With the basic support for 'unrestricted' from
https://codereview.chromium.org/257783003 its possible to turn 'float'
into 'unrestricted float' where the spec has the attribute/parameter
marked that way.
Also apply 'unrestricted' to all the non-standard APIs.
When the actual support has landed it should be possible to remove any
explicit throwing as well, so mark such locations with FIXMEs.

BUG=367648
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=172733

[fs, 2014-04-25 13:54:40]

Lots of flaky tests ATM, but this Cl should not even modify the generated
code... so PTAL.

[Justin Novosad, 2014-04-25 14:44:28]

https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
File Source/core/html/canvas/CanvasRenderingContext2D.cpp (right):

https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
Source/core/html/canvas/CanvasRenderingContext2D.cpp:1651: // FIXME: These
exceptions will be thrown by generated bindings code once crbug.com/354298 is
fixed.
Shouldn't we keep these comments until we are ready to remove the checks?

https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
File Source/core/html/canvas/CanvasRenderingContext2D.idl (right):

https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
Source/core/html/canvas/CanvasRenderingContext2D.idl:99: void
strokeText(DOMString text, float x, float y, optional float maxWidth);
fillText and strokeText take unrestricteds according to the spec.

[fs, 2014-04-25 15:50:15]

https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
File Source/core/html/canvas/CanvasRenderingContext2D.cpp (right):

https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
Source/core/html/canvas/CanvasRenderingContext2D.cpp:1651: // FIXME: These
exceptions will be thrown by generated bindings code once crbug.com/354298 is
fixed.
On 2014/04/25 14:44:29, junov wrote:
> Shouldn't we keep these comments until we are ready to remove the checks?

Yes? "Full" (==actual) support for 'unrestricted' is required before that
though. I suspect it'll be enabled initially by some TypeChecking=Foo attribute.

https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
File Source/core/html/canvas/CanvasRenderingContext2D.idl (right):

https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
Source/core/html/canvas/CanvasRenderingContext2D.idl:99: void
strokeText(DOMString text, float x, float y, optional float maxWidth);
On 2014/04/25 14:44:29, junov wrote:
> fillText and strokeText take unrestricteds according to the spec.

Rebase fail... Fixed.

[Justin Novosad, 2014-04-25 16:30:24]

On 2014/04/25 15:50:15, fs wrote:
>
https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
> File Source/core/html/canvas/CanvasRenderingContext2D.cpp (right):
> 
>
https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
> Source/core/html/canvas/CanvasRenderingContext2D.cpp:1651: // FIXME: These
> exceptions will be thrown by generated bindings code once crbug.com/354298 is
> fixed.
> On 2014/04/25 14:44:29, junov wrote:
> > Shouldn't we keep these comments until we are ready to remove the checks?
> 
> Yes? "Full" (==actual) support for 'unrestricted' is required before that
> though. I suspect it'll be enabled initially by some TypeChecking=Foo
attribute.
> 
>
https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
> File Source/core/html/canvas/CanvasRenderingContext2D.idl (right):
> 
>
https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
> Source/core/html/canvas/CanvasRenderingContext2D.idl:99: void
> strokeText(DOMString text, float x, float y, optional float maxWidth);
> On 2014/04/25 14:44:29, junov wrote:
> > fillText and strokeText take unrestricteds according to the spec.
> 
> Rebase fail... Fixed.

lgtm

Please file a bug for removing the finite checks when the time is right. Just so
we don't forget about it.

[fs, 2014-04-28 07:53:01]

On 2014/04/25 16:30:24, junov wrote:
> On 2014/04/25 15:50:15, fs wrote:
> >
>
https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
> > File Source/core/html/canvas/CanvasRenderingContext2D.cpp (right):
> > 
> >
>
https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
> > Source/core/html/canvas/CanvasRenderingContext2D.cpp:1651: // FIXME: These
> > exceptions will be thrown by generated bindings code once crbug.com/354298
is
> > fixed.
> > On 2014/04/25 14:44:29, junov wrote:
> > > Shouldn't we keep these comments until we are ready to remove the checks?
> > 
> > Yes? "Full" (==actual) support for 'unrestricted' is required before that
> > though. I suspect it'll be enabled initially by some TypeChecking=Foo
> attribute.
> > 
> >
>
https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
> > File Source/core/html/canvas/CanvasRenderingContext2D.idl (right):
> > 
> >
>
https://codereview.chromium.org/257853002/diff/1/Source/core/html/canvas/Canv...
> > Source/core/html/canvas/CanvasRenderingContext2D.idl:99: void
> > strokeText(DOMString text, float x, float y, optional float maxWidth);
> > On 2014/04/25 14:44:29, junov wrote:
> > > fillText and strokeText take unrestricteds according to the spec.
> > 
> > Rebase fail... Fixed.
> 
> lgtm
> 
> Please file a bug for removing the finite checks when the time is right. Just
so
> we don't forget about it.

Done (crbug.com/367648)

[fs, 2014-04-28 07:53:43]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2014-04-28 07:54:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fs@opera.com/257853002/40001

[commit-bot: I haz the power, 2014-04-28 08:29:52]

Change committed as 172733