RESOLVED FIXED132326
[GTK] Simplify Vector handling in WorkQueue::registerSocketEventHandler() 
https://bugs.webkit.org/show_bug.cgi?id=132326
Summary [GTK] Simplify Vector handling in WorkQueue::registerSocketEventHandler()
Zan Dobersek
Reported 2014-04-29 02:21:05 PDT
[GTK] Simplify Vector handling in WorkQueue::registerSocketEventHandler()
Attachments
Patch (2.67 KB, patch)
2014-04-29 02:26 PDT, Zan Dobersek 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Zan Dobersek
Comment 1 2014-04-29 02:26:17 PDT
Created attachment 230364 [details] Patch
Carlos Garcia Campos
Comment 2 2014-04-29 03:37:19 PDT
Comment on attachment 230364 [details] Patch This code should die once bug #130081 is fixed. It's not really possible to register a socket event handler for more than one descriptor.
Zan Dobersek
Comment 3 2014-04-29 11:01:02 PDT
(In reply to comment #2) > (From update of attachment 230364 [details]) > This code should die once bug #130081 is fixed. It's not really possible to register a socket event handler for more than one descriptor. I'll mark this bug as dependent on bug #130081 so I get notified when this patch might become obsolete.
Zan Dobersek
Comment 4 2014-04-29 11:02:34 PDT
Comment on attachment 230364 [details] Patch Removing the review flag until then.
Zan Dobersek
Comment 5 2014-05-29 05:01:33 PDT
(In reply to comment #3) > (In reply to comment #2) > > (From update of attachment 230364 [details] [details]) > > This code should die once bug #130081 is fixed. It's not really possible to register a socket event handler for more than one descriptor. > > I'll mark this bug as dependent on bug #130081 so I get notified when this patch might become obsolete. Bug #130081 landed, making this patch invalid/unnecessary. Closing.
Note You need to log in before you can comment on or make changes to this bug.