Pathc for better support for text and various optimizations about flashcanvas HOT 10 CLOSED

Thank you for your contribution. I'll check your patch later.

Some comments:

- TextMetrics
Should we implement the attribute "height"? It's not in the specification.

- server side script
It's better to add the following header, isn't it?
header('Content-Disposition: attachment; filename="canvas.png"');

- getScriptUrl()
No need to modify. See http://d.hatena.ne.jp/shogo4405/20070206/1170741017 for
example. The page is written in Japanese, but you'll understand what I want to 
say if
you look at the code.

- TextMetrics
I added this because I never understood why this isn't in the specs, but if it 
isn't
in the spec, it isn't in the spec...

- server side script
Yep, it would even be better to add this :

$ext = substr($mime, strrpos($mime, "/")+1);
header('Content-Disposition: attachment; filename="image.'.$ext.'"');

- getScriptUrl()
Very clever ! I didn't know this

I saw that FlashCanvas pro supported text rotation, did you make it work with
rotationZ (but Flash10 only) ? It's really sad the result is so bad compared to
embedded fonts, there must be a way to make it better with another method.

I'll have to add the relative font size with "em" and percents, that's not so 
hard.
Multiple font families support too (only the first one will be used here, and 
if not
found, it will always fall back to a serif font)

Committed some patches. The remaining patches will follow.

> - custom menu with two options : "Save image as" and "Go to project page" 
(FxCanvas)
> - added a few data types as they are in the spec for future use 
(CanvasPixelArray,
ImageData and TextMetrics)
> - added a server side script taking as POST parameter a data URI scheme for 
image
download
> - added support for JPEG for the toDataUrl method

I removed the code for height attribute from TextMetrics.as, because it is not
supported also in other browsers.

As for text rotation, I didn't use rotationZ. If you need more information, 
please
E-mail me.

Fabien, your patch contains the modification:

> // If the browser does not support HTML5 Canvas (the CanvasRenderingContext2D
global object is 
> // not defined in Safari 3, whereas is supports Canvas)
> if (!window["CanvasRenderingContext2D"] &&
!document.createElement("canvas").getContext) {

I propose an alternative since I don't want to use createElement("canvas") here.
So what do you think?

// If the browser is IE and does not support HTML5 Canvas
if (window.ActiveXObject && !window["CanvasRenderingContext2D"]) {

Hello, thank for applying some parts of my match !
I suggest something even better : 

if (false /*@cc_on || !window["CanvasRenderingContext2D"]*/) {

The second part won't even be interpreted by non-IE browsers in the case the 
script tag 
is not between conditionnal comments.

Unfortunately, Google Closure Compiler does not (and will not) support 
conditional
comments. It's sad...
http://code.google.com/p/closure-compiler/issues/detail?id=47

Committed the remaining patches with little modification.
If you do not have any objection, I'll close this issue in a few days.

That's OK for me !
I'll fix the font size ASAP.

This issue was closed.
If you make additional patches, please report it as an other issue.