Bug 463341 - Don't use unnecessary buggy-repeat fallbacks.
Summary: Don't use unnecessary buggy-repeat fallbacks.
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: firefox
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Martin Stransky
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-09-22 22:53 UTC by Owen Taylor
Modified: 2018-04-11 17:24 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-09-25 06:46:20 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Owen Taylor 2008-09-22 22:53:34 UTC
The Firefox version of cairo unconditionally turns on a "buggy-repeat"
fallback in Cairo that does tiled repeating images by grabbing everything
to the client, doing the repeat in software, then pushing the result
back. This can be extremely slow.

(It may be even slower with the kernel modesetting drivers that we are using in Fedora 10, but it is always going to be really slow.)

It's not completely clear what X server bug was being worked around, but
it doesn't seem to be present in recent X servers, like the Fedora X
server. Quoting from IRC:

<vlad_> try running with MOZ_CAIRO_NO_BUGGY_REPEAT=1 in the environment
<vlad_> we unconditionally force buggy repeat handling because we couldn't figure out a good heuristic to determine which x servers/drivers had the buggy code
<vlad_> so when we last looked at it, the bug existed in 1.3.0.0 and seemed to be gone in at least xorg 1.4.99.901
<vlad_> so what we could do is disable that forcing for > 1.4 (or 1.5, whatever the latest is)
<vlad_> otaylor: filed https://bugzilla.mozilla.org/show_bug.cgi?id=456467

We should make sure that we have this fallback off for F10, either by pulling an upstream patch, by setting the environment variable, or by some either means.

One test case of this is header of http://www.smh.com.au , but it probably will be triggered on many web pages.

Comment 2 Martin Stransky 2008-09-23 12:54:56 UTC
I can handle it.

Comment 3 Martin Stransky 2008-09-24 09:18:32 UTC
It does not affect Fedora because we use system cairo there and it does not contain the original (broken) hack. The affected systems are RHEL-4/5 only.

Comment 4 Owen Taylor 2008-09-24 12:48:25 UTC
The Rawhide Firefox is not using the system cairo, even if it is intended to.

Comment 5 Martin Stransky 2008-09-25 06:46:20 UTC
okay, let's enable it then. added to xulrunner-1.9.0.2-2.fc10


Note You need to log in before you can comment on or make changes to this bug.