[whatwg] Canvas resetClip() API

[Ian Hickson]

On Tue, 8 Oct 2013, Simon Fraser wrote:
> 
> I don’t like the resetClip() API, and don’t think it should be added to 
> canvas.
> 
> The reason resetClip() is a bad API is that it breaks the ability to 
> robustly modularize drawing code. For example, say that you have the 
> following code:
> 
> context.save();
> // … set up a path to clip to
> context.clip();
> someJSLibrary.drawSomething();
> context.restore();
> 
> In the absence of resetClip(), you are guaranteed that 
> someJSLibrary.drawSomething() can’t draw outside of the clip you 
> specified. With resetClip(), that JS library can draw wherever the heck 
> it wants, which may totally break your canvas drawing.

That's true anyway; for example, the JS library could call 
context.restore() to escape your clip(), or could call save() more times 
than it calls restore(). At the end of the day, you have to have a 
contract with your library, and in that there might be "you obey my clip" 
or "you obey my transforms" or "you obey my transforms but it's ok if you 
unclip" or whatever.


> If you really need resetClip() in your own drawing code, you’re probably 
> doing it wrong.

Why is it more wrong to need resetClip() than to need resetTransform()?

-- 
Ian Hickson               U+1047E                )\._.,--....,'``.    fL
http://ln.hixie.ch/       U+263A                /,   _.. \   _\  ;`._ ,.
Things that are impossible just take longer.   `._.-(,_..'--(,_..'`-.;.'