The default bug view has changed. See this FAQ.

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

RESOLVED FIXED in mozilla17

Status

()

Core
Graphics
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: nrc, Assigned: nrc)

Tracking

15 Branch
mozilla17
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

5 years ago
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.
(Assignee)

Comment 1

5 years ago
Created attachment 647838 [details]
screenshot of the Android bug
(Assignee)

Comment 2

5 years ago
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).
(Assignee)

Comment 3

5 years ago
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...
(Assignee)

Comment 4

5 years ago
Split off bug 779650 for the Windows/smashcat bug
(Assignee)

Updated

5 years ago
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
(Assignee)

Comment 5

5 years ago
Created attachment 648157 [details]
minimal test case
(Assignee)

Comment 6

5 years ago
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
(Assignee)

Comment 7

5 years ago
Created attachment 649189 [details] [diff] [review]
patch
Assignee: nobody → ncameron
Attachment #649189 - Flags: review?(bas.schouten)
(Assignee)

Comment 8

5 years ago
test coming soon...
Attachment #649189 - Flags: review?(bas.schouten) → review+
(Assignee)

Comment 9

5 years ago
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.
(Assignee)

Comment 10

5 years ago
Created attachment 649490 [details] [diff] [review]
patch

was missing an include, carrying r=bas
Attachment #649189 - Attachment is obsolete: true
Attachment #649490 - Flags: review+
(Assignee)

Comment 11

5 years ago
https://tbpl.mozilla.org/?tree=Mozilla-Inbound&rev=309786fce827
https://hg.mozilla.org/mozilla-central/rev/309786fce827
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.