Bug 108517

Summary: Use a Vector for IPC attachments
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: ossy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch sam: review+

Description Anders Carlsson 2013-01-31 13:44:32 PST 
Use a Vector for IPC attachments
Comment 1 Anders Carlsson 2013-01-31 13:51:31 PST 
Created attachment 185853 [details]
Patch
Comment 2 Sam Weinig 2013-01-31 13:54:49 PST 
Comment on attachment 185853 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=185853&action=review

> Source/WebKit2/ChangeLog:7
> +        Use a Vector for IPC attachments
> +        https://bugs.webkit.org/show_bug.cgi?id=108517
> +
> +        Reviewed by NOBODY (OOPS!).
> +

Please state a reason.
Comment 3 Anders Carlsson 2013-01-31 14:01:48 PST 
Committed r141482: <http://trac.webkit.org/changeset/141482>
Comment 4 Csaba Osztrogonác 2013-01-31 14:10:51 PST 
(In reply to comment #2)
> (From update of attachment 185853 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=185853&action=review
> 
> > Source/WebKit2/ChangeLog:7
> > +        Use a Vector for IPC attachments
> > +        https://bugs.webkit.org/show_bug.cgi?id=108517
> > +
> > +        Reviewed by NOBODY (OOPS!).
> > +
> 
> Please state a reason.

So what was the reason? Because I still can't see in the changelog.
Comment 5 Anders Carlsson 2013-01-31 14:14:21 PST 
(In reply to comment #4)
> (In reply to comment #2)
> > (From update of attachment 185853 [details] [details])
> > View in context: https://bugs.webkit.org/attachment.cgi?id=185853&action=review
> > 
> > > Source/WebKit2/ChangeLog:7
> > > +        Use a Vector for IPC attachments
> > > +        https://bugs.webkit.org/show_bug.cgi?id=108517
> > > +
> > > +        Reviewed by NOBODY (OOPS!).
> > > +
> > 
> > Please state a reason.
> 
> So what was the reason? Because I still can't see in the changelog.

Sorry about that, I missed the comment. I updated the ChangeLog in r141484.