Add custom V8 binding for Document.getCSSCanvasContext

[Mike Belshe]

Made this function custom to deal with the difference between DOMObject in the IDL and CanvasRenderingContext2D.

[Mike Belshe]

Created attachment 28557 [details]
patch

[Dimitri Glazkov (Google)]

Comment on attachment 28557 [details]
patch

Looks good, aside for helper functions.

Also, don't forget to put bug URL in the ChangeLog. I usually do:

URL
Description.

(standard stuff prepareChangelog puts in)

> +    String contextId = ToWebCoreString(args[0]);

toWebCoreString

> +    String name = ToWebCoreString(args[1]);

Ditto.

> +    int width = ToInt32(args[2]);

toInt32
> +    int height = ToInt32(args[3]);
Ditto.
> +    CanvasRenderingContext2D* result = imp->getCSSCanvasContext(contextId, name, width, height);
> +    return V8Proxy::ToV8Object(V8ClassIndex::CANVASRENDERINGCONTEXT2D, result);
> +}
> +
>  } // namespace WebCore

[Mike Belshe]

Created attachment 28563 [details]
updated patch

[Dimitri Glazkov (Google)]

Landed as http://trac.webkit.org/changeset/41691.