Closed Bug 778031 Opened 7 years ago Closed 7 years ago
don't XSync when already already synced
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://tbpl.mozilla.org/?tree=Try&rev=47e0b459eb76 https://hg.mozilla.org/integration/mozilla-inbound/rev/330f9bebf53f https://hg.mozilla.org/integration/mozilla-inbound/rev/deba4b252efe
Hardware: x86_64 → All
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.