Checkerboard telemetry code doesn't take into account RTL pages

RESOLVED FIXED in Firefox 16

Status

()

Firefox for Android
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: kats, Assigned: kats)

Tracking

unspecified
Firefox 16
All
Android
Points:
---

Firefox Tracking Flags

(firefox14 wontfix, firefox15 wontfix, firefox16 fixed)

Details

Attachments

(1 attachment)

Created attachment 628349 [details] [diff] [review]
Patch

Looking through LayerRenderer I found this, which I think I missed when I landed the RTL patch. Not as critical since the code isn't exercised in production, but still good to fix.
Attachment #628349 - Flags: review?(chrislord.net)

Comment 1

5 years ago
Comment on attachment 628349 [details] [diff] [review]
Patch

Review of attachment 628349 [details] [diff] [review]:
-----------------------------------------------------------------

Looks fine.
Attachment #628349 - Flags: review?(chrislord.net) → review+
Landed on inbound

https://hg.mozilla.org/integration/mozilla-inbound/rev/088893efefc7

Marking as wontfix for 14 and 15 since this doesn't affect any production code and is not worth uplifting.
status-firefox14: --- → wontfix
status-firefox15: --- → wontfix
status-firefox16: --- → fixed
Target Milestone: --- → Firefox 16

Comment 3

5 years ago
https://hg.mozilla.org/mozilla-central/rev/088893efefc7
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.