r51105 removed the null timer event of the carbon event model. This however breaks the flash of many sites (the ball in http://www.communitymx.com/content/source/E5141/wmodetrans.htm bounces slowly for example).
First off, I apologize for the lack of time for me in working on this bug (especially since I was the initial committer). That said, I have a patch at http://qt.gitorious.org/~girish/qt/girishs-qt/commit/c8483ce0bdc38ba40a7d673340f6f4f04b7bc439. The patch is known to work (it is more than a simple 'revert' since trunk has changed a bit). If someone has the time, please go ahead and get this committed.
PluginViewMac could use a much better name. Looking at this bug, I began to wonder how this change didn't break Safari behavior.
Alexey, I think the intent is to make PluginViewMac, the common plugin implementation used across all ports. To be clear : This task is Qt specific. PluginViewMac is not used by Safari.
Created attachment 49583 [details] Applying patch suggested by Girish Ramakrishnan in bug comment #1.
Attachment 1 looks good to me. Only suggestion I had was to modify the ChangeLog to state why we are reverting it.
Created attachment 49586 [details] Applying patch suggested by Girish Ramakrishnan in bug comment #1.
Comment on attachment 49586 [details] Applying patch suggested by Girish Ramakrishnan in bug comment #1. LGTM, but it would be nice with an explanation to why that 22 constant is used. It was just moved around in this code, so I won't r- it because of that.
Comment on attachment 49586 [details] Applying patch suggested by Girish Ramakrishnan in bug comment #1. Clearing flags on attachment: 49586 Committed r55332: <http://trac.webkit.org/changeset/55332>
All reviewed patches have been landed. Closing bug.