Closed Bug 662898 Opened 13 years ago Closed 13 years ago

Canvas regression: Cannot draw image into canvas

Categories

(Core :: Graphics: Canvas2D, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla7
Tracking Status
firefox5 - affected
firefox6 - fixed

People

(Reporter: exien.ow, Assigned: roc)

References

()

Details

(Keywords: regression)

Attachments

(2 files, 1 obsolete file)

User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:5.0) Gecko/20100101 Firefox/5.0 Build Identifier: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:5.0) Gecko/20100101 Firefox/5.0 Cannot get canvas from canvas A, and draw it as an image into canvas B. Reproducible: Always Steps to Reproduce: 1. Visit http://radikalfx.com/files/collage/demo.html 2. Click search button on the bottom left 3. Click on an image Actual Results: Image is not loaded into the canvas, instead there is an empty rectangle Expected Results: Image is loaded into the canvas Does not work in Firefox 5, or Firefox 7.0a1. Works in Firefox 4.0.1, Safari, and Google Chrome
Regression window: Works: http://hg.mozilla.org/mozilla-central/rev/0798b52bb40d Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.2a1pre) Gecko/20110324 Firefox/4.2a1pre ID:20110324144234 Fails: http://hg.mozilla.org/mozilla-central/rev/6a95db826e08 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.2a1pre) Gecko/20110324 Firefox/4.2a1pre ID:20110324145508 Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=0798b52bb40d&tochange=6a95db826e08
Blocks: 639689
Status: UNCONFIRMED → NEW
Component: General → Canvas: 2D
Ever confirmed: true
Keywords: regression
OS: Mac OS X → All
Product: Firefox → Core
QA Contact: general → canvas.2d
Hardware: x86_64 → All
FYI, Toggle shadow off, then the image appears,
Bisected in local build, Triggered by: a34fe4ee349a Robert O'Callahan — Bug 639689. Part 3: Remove unnecessary full context save/restore, and redundant SetPattern call. r=joe
Attached patch fix (obsolete) (deleted) — Splinter Review
Assignee: nobody → roc
Attachment #538172 - Flags: review?(jmuizelaar)
I'll write a test for this when my tree is in a better state.
Attachment #538172 - Flags: review?(jmuizelaar) → review+
Keywords: checkin-needed
Whiteboard: [needs landing]
Attached patch testcase (deleted) — Splinter Review
Attachment #538685 - Flags: review?(jmuizelaar)
http://hg.mozilla.org/integration/mozilla-inbound/rev/e01a18d61dd6 Once this lands on central, I would like to get it into ff5/beta if possible.
Keywords: checkin-needed
Whiteboard: [needs landing] → [inbound]
Backed out on m-i due to test_canvas.html failures: http://hg.mozilla.org/integration/mozilla-inbound/rev/3d1b0a500fbf
Whiteboard: [inbound]
Attached patch fix test failure (deleted) — Splinter Review
We need to make sure the input pattern is set correctly when we draw the shadow.
Attachment #538172 - Attachment is obsolete: true
http://hg.mozilla.org/integration/mozilla-inbound/rev/a858cef629f0 http://hg.mozilla.org/integration/mozilla-inbound/rev/70c1c696f691 I pushed the test on the assumption it will be OK. If Jeff doesn't like it we can change it later.
Whiteboard: [inbound]
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla7
Comment on attachment 538795 [details] [diff] [review] fix test failure Probably too late for beta, but might as well draw the attention of release drivers. This is a canvas regression in an edge case, but the fix is very simple.
Attachment #538795 - Flags: approval-mozilla-beta?
Attachment #538795 - Flags: approval-mozilla-aurora?
Comment on attachment 538795 [details] [diff] [review] fix test failure too late for beta. we'll evaluate for 6.
Attachment #538795 - Flags: approval-mozilla-beta? → approval-mozilla-beta-
Attachment #538795 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Whiteboard: [needs landing]
Attachment #538685 - Flags: review?(jmuizelaar) → review+
Flags: in-testsuite+
Bug 668742 is about that this issue is still present in the current Firefox 5 release. Can we expect any improvement in a Firefox 5 update or do we need to wait until Firefox 6 is released? Then I know what to tell to people on he support forum that have this problem in Firefox 5. You can see a list of support threads that show this problem in Firefox 5, so we can expect to see a lot more reports about this issue. (cross posting this in bug Bug 668742 as this bug is resolved)
> Can we expect any improvement in a Firefox 5 update or do we need to wait until > Firefox 6 is released? Most likely the latter. That should happen in mid-August....
Thanks for confirming that. I was already expecting that (porting code to another branch is not easy), but a confirmation helps to avoid confusion and disappointment.
It's not just porting; there are just no plans to do any Firefox 5 updates before Firefox 6 ships unless a critical issue comes up; if one does an update would contain a fix for just that issue. Once Firefox 6 ships, it becomes the security update to Firefox 5, with automatic updates from 5 to 6.
Mozilla/5.0 (Windows NT 5.1; rv:7.0) Gecko/20100101 Firefox/7.0 Verified on Win XP, Win 7, Ubuntu 11.04 and Mac OS X 10.6 using the STR from the description. Image is now loaded and can be dragged into the canvas. Setting resolution to VERIFIED FIXED.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: