WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
115986
Simplify AutodrainedPool
https://bugs.webkit.org/show_bug.cgi?id=115986
Summary
Simplify AutodrainedPool
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+
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug