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+

Description Anders Carlsson 2013-05-12 10:31:43 PDT
Simplify AutodrainedPool
Comment 1 Anders Carlsson 2013-05-12 10:35:55 PDT
Created attachment 201495 [details]
Patch
Comment 2 Andreas Kling 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!
Comment 3 Anders Carlsson 2013-05-12 10:50:56 PDT
Committed r149965: <http://trac.webkit.org/changeset/149965>
Comment 4 Alexey Proskuryakov 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?
Comment 5 Anders Carlsson 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.