Bug 109736 - EventDispatcher should be a WorkQueueMessageReceiver
Summary: EventDispatcher should be a WorkQueueMessageReceiver
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Anders Carlsson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-13 12:46 PST by Anders Carlsson
Modified: 2013-02-13 13:53 PST (History)
1 user (show)

See Also:


Attachments
Patch (9.08 KB, patch)
2013-02-13 12:47 PST, Anders Carlsson
kling: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Anders Carlsson 2013-02-13 12:46:33 PST
EventDispatcher should be a WorkQueueMessageReceiver
Comment 1 Anders Carlsson 2013-02-13 12:47:15 PST
Created attachment 188155 [details]
Patch
Comment 2 Andreas Kling 2013-02-13 12:53:17 PST
Comment on attachment 188155 [details]
Patch

WorkQueue all the things!
Comment 3 Anders Carlsson 2013-02-13 12:59:49 PST
Committed r142778: <http://trac.webkit.org/changeset/142778>
Comment 4 Csaba Osztrogonác 2013-02-13 13:33:46 PST
(In reply to comment #3)
> Committed r142778: <http://trac.webkit.org/changeset/142778>

It broke the WK2 build, the Mac WK2 build too. What if you
try to build before committing? As far as I remember WK2
patches must build on the core Mac WK2 platform.
Comment 5 Csaba Osztrogonác 2013-02-13 13:53:03 PST
Thanks for the quick fix - http://trac.webkit.org/changeset/142784.
(But the comment is a little bit misleading ... It fixed all 
!ENABLE(THREADED_SCROLLING) WK2 platforms, not only Lion.)