Peter Van der Beken [:peterv] Assignee
	Description • 12 years ago

      No description provided.

	:Ms2ger (he/him; ⌚ UTC+1/+2)
	Comment 2 • 12 years ago

Comment on attachment 631113 [details] [diff] [review]
v1

Review of attachment 631113 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/base/Makefile.in
@@ +128,5 @@
>  
>  LOCAL_INCLUDES += \
>  		-I$(topsrcdir)/js/xpconnect/src \
>  		-I$(topsrcdir)/js/xpconnect/wrappers \
> +		-I$(topsrcdir)/content/canvas/src \

Why's that?

	Robert O'Callahan (:roc) (email my personal email if necessary)
	Comment 3 • 12 years ago

FWIW we're getting pretty close to being able to remove nsCanvasRenderingContext2D, if that helps here.

	Peter Van der Beken [:peterv] Assignee
	Comment 4 • 12 years ago

This makes the pref for Azure canvas static, so requiring a restart after changing it.
I had to make nsIDOMCanvasRenderingContext2D_GetImageData a custom quickstub because ImageData is a non-wrappercached object with a new binding.

	Boris Zbarsky [:bzbarsky]
	Comment 5 • 12 years ago

Comment on attachment 636489 [details] [diff] [review]
v1.1

r=me

	Peter Van der Beken [:peterv] Assignee
	Comment 6 • 12 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/d6f341e77b68

	Matt Brubeck (:mbrubeck)
	Comment 7 • 12 years ago

Sorry, I backed this out on inbound along with bug 762652 because Windows PGO builds were crashing in gfxContext::FillAzure:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a2cc88619ba

https://tbpl.mozilla.org/php/getParsedLog.php?id=13051005&tree=Mozilla-Inbound

	Peter Van der Beken [:peterv] Assignee
	Comment 8 • 12 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/9feb8257666c

	Ryan VanderMeulen [:RyanVM]
	Comment 9 • 12 years ago

https://hg.mozilla.org/mozilla-central/rev/9feb8257666c