Make it possible to use tiling with the basic compositor backend (at least on Linux)

RESOLVED FIXED in mozilla37

Status

()

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

People

(Reporter: nical, Assigned: nical)

Tracking

unspecified
mozilla37
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Even if we don't ship it, there is no reason to prevent this configuration directly in the code, which is what we do here: http://dxr.mozilla.org/mozilla-central/source/gfx/layers/client/ClientPaintedLayer.cpp
(Assignee)

Comment 1

3 years ago
Created attachment 8537276 [details] [diff] [review]
Don't disable tiles for the basic backend

This may cause some tooltips and and other kind of widget/popups to have tiled painted layers where they used to not be tiled.

If we don't want that, then we should consider having the same logic on all platforms as what we currently do on b2g, that is only use tiled layers for scrollable content.
Attachment #8537276 - Flags: review?(bas)
Comment on attachment 8537276 [details] [diff] [review]
Don't disable tiles for the basic backend

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

For now I feel this is fine.
Attachment #8537276 - Flags: review?(bas) → review+
(Assignee)

Comment 3

3 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/bf93cbec0450
Backed out for OSX asserts/crashes.
https://hg.mozilla.org/integration/mozilla-inbound/rev/3a1f8d27931e

https://treeherder.mozilla.org/logviewer.html#?job_id=4763848&repo=mozilla-inbound
(Assignee)

Comment 5

3 years ago
Relanded linux-only for now: https://hg.mozilla.org/integration/mozilla-inbound/rev/3233b239e61d
Summary: make it possible to use tiling with the basic compositor backend → Make it possible to use tiling with the basic compositor backend (at least on Linux)
https://hg.mozilla.org/mozilla-central/rev/3233b239e61d
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in before you can comment on or make changes to this bug.