Bug 115986

Summary: Simplify AutodrainedPool
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, benjamin, cmarcelo, commit-queue, japhet, kling
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch kling: review+

Anders Carlsson
Reported 2013-05-12 10:31:43 PDT
Simplify AutodrainedPool
Attachments
Patch (9.70 KB, patch)
2013-05-12 10:35 PDT, Anders Carlsson 		kling: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2013-05-12 10:35:55 PDT
Created attachment 201495 [details] Patch
Andreas Kling
Comment 2 2013-05-12 10:46:19 PDT
Comment on attachment 201495 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=201495&action=review > Source/WTF/ChangeLog:9 > + Creating and draining pools is fast enough now that we don't need to try to be clever about it. Great!
Anders Carlsson
Comment 3 2013-05-12 10:50:56 PDT
Committed r149965: <http://trac.webkit.org/changeset/149965>
Alexey Proskuryakov
Comment 4 2013-05-12 18:30:52 PDT
Comment on attachment 201495 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=201495&action=review >> Source/WTF/ChangeLog:9 >> + Remove the ability to cycle an AutodrainedPool, as well as the iteration count. >> + Creating and draining pools is fast enough now that we don't need to try to be clever about it. > > Great! When did it become fast, does that include Lion?
Anders Carlsson
Comment 5 2013-05-13 10:36:02 PDT
(In reply to comment #4) > (From update of attachment 201495 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=201495&action=review > > >> Source/WTF/ChangeLog:9 > >> + Remove the ability to cycle an AutodrainedPool, as well as the iteration count. > >> + Creating and draining pools is fast enough now that we don't need to try to be clever about it. > > > > Great! > > When did it become fast, does that include Lion? It did become fast in Lion.
Note You need to log in before you can comment on or make changes to this bug.