Handle negative arguments to drawImage

VERIFIED FIXED

Status

()

Core
Canvas: 2D
VERIFIED FIXED
6 years ago
3 years ago

People

(Reporter: christian, Assigned: jrmuizel)

Tracking

(Blocks: 1 bug, 4 keywords)

Trunk
dev-doc-complete, html5, verified-aurora, verified-beta
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox5+ fixed, firefox6 fixed, firefox7+ fixed)

Details

(Whiteboard: [qa!], URL)

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
+++ This bug was initially created as a clone of Bug #629875 +++

http://www.w3c-test.org/html/tests/submission/PhilipTaylor/canvas/2d.drawImage.negativedest.html
http://www.w3c-test.org/html/tests/submission/PhilipTaylor/canvas/2d.drawImage.negativedir.html
http://www.w3c-test.org/html/tests/submission/PhilipTaylor/canvas/2d.drawImage.negativesource.html

We need to figure out if we want the backout on Aurora as we didn't transplant anything in the source migration (didn't transplant cleanly)
(Reporter)

Updated

6 years ago
tracking-firefox7: --- → +
So, should we get attachment 533887 [details] [diff] [review] reviewed and landed?
Keywords: dev-doc-complete → dev-doc-needed

Comment 2

6 years ago
JP, can you help find the right people to look at this?

Comment 3

6 years ago
Jeff probably has the most context.
Assignee: nobody → jmuizelaar
(Assignee)

Comment 4

6 years ago
Yes, I believe we should land the backout on Aurora. I'll take a closer look at this today.
Created attachment 546030 [details] [diff] [review]
Switch Azure to old behavior

This patch will switch Azure to the old behavior.
I pushed this and the reversion of bug 629875 to aurora:

http://hg.mozilla.org/releases/mozilla-aurora/rev/20e76c984d82

and to inbound:

http://hg.mozilla.org/integration/mozilla-inbound/rev/b9ad0b9119b7
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
status-firefox7: --- → fixed
http://hg.mozilla.org/mozilla-central/rev/b9ad0b9119b7
This was documented a while ago.
Keywords: dev-doc-needed → dev-doc-complete
qa+ for verification in Firefox 7
Whiteboard: [qa+]
Verified using the links from the description in Win XP, Win 7, Mac OS X 10.6 and Ubuntu 11.04 on Firefox 7RC, Aurora and the latest Nightly. On all of the 3 links the results are Failed on a red Background, this is the intended behavior as this is a backout.

Mozilla/5.0 (Windows NT 5.1; rv:7.0) Gecko/20100101 Firefox/7.0
Mozilla/5.0 (Windows NT 6.1; rv:7.0) Gecko/20100101 Firefox/7.0
Mozilla/5.0 (X11; Linux x86_64; rv:7.0) Gecko/20100101 Firefox/7.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:7.0) Gecko/20100101 Firefox/7.0

Mozilla/5.0 (Windows NT 5.1; rv:8.0a2) Gecko/20110921 Firefox/8.0a2
Mozilla/5.0 (Windows NT 6.1; rv:8.0a2) Gecko/20110921 Firefox/8.0a2
Mozilla/5.0 (X11; Linux x86_64; rv:8.0a2) Gecko/20110921 Firefox/8.0a2
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:8.0a2) Gecko/20110921 Firefox/8.0a2

Mozilla/5.0 (Windows NT 5.1; rv:9.0a1) Gecko/20110922 Firefox/9.0a1
Mozilla/5.0 (Windows NT 6.1; rv:9.0a1) Gecko/20110922 Firefox/9.0a1
Mozilla/5.0 (X11; Linux x86_64; rv:9.0a1) Gecko/20110922 Firefox/9.0a1
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:9.0a1) Gecko/20110923 Firefox/9.0a1
Status: RESOLVED → VERIFIED
Whiteboard: [qa+] → [qa!]

Updated

6 years ago
Keywords: verified-aurora, verified-beta
You need to log in before you can comment on or make changes to this bug.