Bug 132326

Summary: [GTK] Simplify Vector handling in WorkQueue::registerSocketEventHandler()
Product: WebKit Reporter: Zan Dobersek <zan>
Component: New BugsAssignee: Zan Dobersek <zan>
Status: RESOLVED FIXED    
Severity: Normal CC: cgarcia
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 130081    
Bug Blocks:    
Attachments:
Description Flags
Patch none

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.