GNOME Bugzilla – Bug 686406
MetaPluginManager: don't send events to Clutter twice
Last modified: 2012-11-03 23:37:10 UTC
When support for multiple plugins was removed, the logic that was supposed to send events to Clutter directly *only if there is no filter function from a plugin* was broken, so events were being sent to Clutter twice if Clutter didn't consume them the first time.
Created attachment 226768 [details] [review] MetaPluginManager: don't send events to Clutter twice
Review of attachment 226768 [details] [review]: I thought the point was that if you returned FALSE from the filter, you were marked as not sending the event to Clutter? I guess I understood it wrong, then.
The comment above explains: /* We need to make sure that clutter gets certain events, like * ConfigureNotify on the stage window. If there is a plugin that * provides an xevent_filter function, then it's the responsibility * of that plugin to pass events to Clutter. Otherwise, we send the * event directly to Clutter ourselves. */
I somehow misread that... a few times.
Attachment 226768 [details] pushed as f258556 - MetaPluginManager: don't send events to Clutter twice