Closed Bug 779401 Opened 8 years ago Closed 8 years ago

[Azure] Azure/cairo canvas renders Asteroids canvas demo incorrectly on Android

Categories

(Core :: Graphics, defect)

15 Branch
ARM
Android
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: nrc, Assigned: nrc)

References

()

Details

Attachments

(3 files, 1 obsolete file)

For the following, you need a recent nightly with Azure canvas pref'ed on and gfx.canvas.azure.backends = "cairo".

On Windows, http://www.smashcat.org/av/canvas_test/ renders as a blank (white) rectangle.

On Android (tested Nexus 7 (4.1) and HTC One X (4.0)), http://www.kevs3d.co.uk/dev/asteroids/index-debug.html renders very badly (see screenshot, coming soon).

I've filed these as one bug for now, I suspect there might be a single cause, but we might want to split the bug later.
note on the attachment, I think the double awesome bar is a separate bug, due to my build (it doesn't seem to be present on the nightly build I downloaded).
The Windows bug only happens with Direct2D on, so it must be different from the Android bug, so going to split this off into a separate bug...
Split off bug 779650 for the Windows/smashcat bug
OS: All → Android
Hardware: All → ARM
Summary: [Azure] Azure/cairo canvas render errors with some canvas demos → [Azure] Azure/cairo canvas renders Asteroids canvas demo incorrectly on Android
Attached file minimal test case
Seems to be a problem with the 9arg flavour of drawImage. I have no idea why it would only affect Android, and not affect the Thebes canvas
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → ncameron
Attachment #649189 - Flags: review?(bas.schouten)
test coming soon...
Attachment #649189 - Flags: review?(bas.schouten) → review+
test not coming soon after all. This bug is caught by the drawImage.9arg tests in test_canvas when run locally, I don't know why it doesn't fail on Try, but I can't imagine a test which would fail there is the test_canvas ones don't.
Attached patch patchSplinter Review
was missing an include, carrying r=bas
Attachment #649189 - Attachment is obsolete: true
Attachment #649490 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/309786fce827
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.