Retained tiles should not be used with fixed-position layers

RESOLVED FIXED in Firefox 16

Status

()

Core
Graphics: Layers
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: cwiiis, Assigned: cwiiis)

Tracking

unspecified
mozilla17
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox16 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Fixed position layers don't move, so it doesn't make sense to try to retain tiles. Currently, it just ends up causing weird-looking rendering glitches. We should just not use them for fixed position layers.

Eventually retained tiles will be superseded by on-demand tile rendering work, but this is an easy thing to do that will improve the experience on sites with top/bottom bars, etc.
(Assignee)

Comment 1

5 years ago
Created attachment 651516 [details] [diff] [review]
Don't retain tiles on fixed position layers
Attachment #651516 - Flags: review?(bgirard)
Comment on attachment 651516 [details] [diff] [review]
Don't retain tiles on fixed position layers

r+, nice detailed comment.
Attachment #651516 - Flags: review?(bgirard) → review+
(Assignee)

Comment 3

5 years ago
Pushed to inbound:

https://hg.mozilla.org/integration/mozilla-inbound/rev/84f253d1174c
(Assignee)

Comment 4

5 years ago
Comment on attachment 651516 [details] [diff] [review]
Don't retain tiles on fixed position layers

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Rendering artifacts when panning/zooming on pages with fixed-position elements
User impact if declined: See above
Testing completed (on m-c, etc.): Tested locally
Risk to taking this patch (and alternatives if risky): Low - just disables a feature in a simple and reliable way.
String or UUID changes made by this patch: None
Attachment #651516 - Flags: approval-mozilla-aurora?

Comment 5

5 years ago
https://hg.mozilla.org/mozilla-central/rev/84f253d1174c
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Attachment #651516 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 6

5 years ago
Pushed to aurora: http://hg.mozilla.org/releases/mozilla-aurora/rev/bb764cfc347a

Updated

5 years ago
status-firefox16: --- → fixed
You need to log in before you can comment on or make changes to this bug.