Bug 132326 - [GTK] Simplify Vector handling in WorkQueue::registerSocketEventHandler()
Summary: [GTK] Simplify Vector handling in WorkQueue::registerSocketEventHandler()
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: Zan Dobersek
URL:
Keywords:
Depends on: 130081
Blocks:
  Show dependency tree / graph
 
Reported: 2014-04-29 02:21 PDT by Zan Dobersek
Modified: 2014-05-29 05:01 PDT (History)
1 user (show)

See Also:

Attachments
Patch (2.67 KB, patch)
2014-04-29 02:26 PDT, Zan Dobersek 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Zan Dobersek 2014-04-29 02:21:05 PDT 
[GTK] Simplify Vector handling in WorkQueue::registerSocketEventHandler()
Comment 1 Zan Dobersek 2014-04-29 02:26:17 PDT 
Created attachment 230364 [details]
Patch
Comment 2 Carlos Garcia Campos 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.
Comment 3 Zan Dobersek 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.
Comment 4 Zan Dobersek 2014-04-29 11:02:34 PDT 
Comment on attachment 230364 [details]
Patch

Removing the review flag until then.
Comment 5 Zan Dobersek 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.