Summary: | [Streams API] Implement ReadableStreamReader.releaseLock | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | youenn fablet <youennf> | ||||||||
Component: | WebCore Misc. | Assignee: | youenn fablet <youennf> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | calvaris, commit-queue, darin | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 138967 | ||||||||||
Attachments: |
|
Description
youenn fablet
2015-05-21 23:40:33 PDT
Created attachment 254954 [details]
Patch
Comment on attachment 254954 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=254954&action=review > Source/WebCore/Modules/streams/ReadableStream.h:70 > + bool hasReadPendingRequests() { return m_readRequests.size(); } Often we use !isEmpty instead of size for such things. In theory isEmpty might be faster than !size. > Source/WebCore/Modules/streams/ReadableStreamReader.h:36 > +#include "ExceptionCode.h" If we need only the type for ExceptionCode, then we typically declare it with: typedef int ExceptionCode; rather than including the header. Created attachment 255096 [details]
Patch for landing
Created attachment 255097 [details]
Patch for landing
Comment on attachment 255097 [details] Patch for landing Clearing flags on attachment: 255097 Committed r185697: <http://trac.webkit.org/changeset/185697> All reviewed patches have been landed. Closing bug. |