Closed Bug 778031 Opened 7 years ago Closed 7 years ago

don't XSync when already already synced

Categories

(Core :: Graphics, defect)

13 Branch
All
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: karlt, Assigned: karlt)

References

Details

Attachments

(2 files)

No description provided.
This skips more syncs when there are multiple plugins active, or multiple layers with the patch for bug 687831.

I haven't changed the XSyncs where it is (almost) certain that a sync is required.  I originally had plans to make FinishX detect more situations through XFlush, and that would have possibly added unnecessary overhead, but there was no win there, so I dumped that.
Assignee: nobody → karlt
Status: NEW → ASSIGNED
Attachment #646450 - Flags: review?(jones.chris.g)
Comment on attachment 646450 [details] [diff] [review]
add FinishX to XSync less frequently and use it

It's rather surprising that Xlib doesn't do this itself, but looks good!
Attachment #646450 - Flags: review?(jones.chris.g) → review+
Attachment #646451 - Flags: review?(jones.chris.g) → review+
https://hg.mozilla.org/mozilla-central/rev/330f9bebf53f
https://hg.mozilla.org/mozilla-central/rev/deba4b252efe
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.