Closed Bug 762654 Opened 9 years ago Closed 9 years ago

Switch the 2d canvas context to new DOM bindings

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla16

People

(Reporter: peterv, Assigned: peterv)

References

Details

Attachments

(1 file, 1 obsolete file)

No description provided.
Depends on: 762657
Depends on: 761772
Attached patch v1 (obsolete) — Splinter Review
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?
Blocks: 746773
FWIW we're getting pretty close to being able to remove nsCanvasRenderingContext2D, if that helps here.
Attached patch v1.1Splinter Review
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.
Attachment #631113 - Attachment is obsolete: true
Attachment #636489 - Flags: review?(bzbarsky)
Comment on attachment 636489 [details] [diff] [review]
v1.1

r=me
Attachment #636489 - Flags: review?(bzbarsky) → review+
Depends on: 769080
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
Target Milestone: mozilla16 → ---
https://hg.mozilla.org/mozilla-central/rev/9feb8257666c
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Depends on: 776242
Depends on: 778128
No longer depends on: 778128
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.