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+

Anders Carlsson
Reported 2013-01-31 13:44:32 PST
Use a Vector for IPC attachments
Attachments
Patch (8.49 KB, patch)
2013-01-31 13:51 PST, Anders Carlsson 		sam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2013-01-31 13:51:31 PST
Created attachment 185853 [details] Patch
Sam Weinig
Comment 2 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.
Anders Carlsson
Comment 3 2013-01-31 14:01:48 PST
Committed r141482: <http://trac.webkit.org/changeset/141482>
Csaba Osztrogonác
Comment 4 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.
Anders Carlsson
Comment 5 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.
Note You need to log in before you can comment on or make changes to this bug.