Bug 145641

Summary: [Streams API] ReadableStream should store callbacks as a Deque
Product: WebKit Reporter: youenn fablet <youennf>
Component: WebCore Misc.Assignee: youenn fablet <youennf>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, calvaris, cmarcelo, commit-queue, darin
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Reuploading now that 145210 landed none

youenn fablet
Reported 2015-06-04 02:33:47 PDT
Currently, ReadableStream stores callbacks as Vector, but it should be a Deque as we are doing FIFO here.
Attachments
Patch (3.71 KB, patch)
2015-06-04 02:46 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Reuploading now that 145210 landed (3.71 KB, patch)
2015-06-04 03:33 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 2015-06-04 02:46:23 PDT
Created attachment 254262 [details] Patch
youenn fablet
Comment 2 2015-06-04 03:33:06 PDT
Created attachment 254264 [details] Reuploading now that 145210 landed
WebKit Commit Bot
Comment 3 2015-06-05 12:09:15 PDT
Comment on attachment 254264 [details] Reuploading now that 145210 landed Clearing flags on attachment: 254264 Committed r185260: <http://trac.webkit.org/changeset/185260>
WebKit Commit Bot
Comment 4 2015-06-05 12:09:22 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.