[webkit-reviews] review denied: [Bug 122142] Release expired pages in the page cache more often : [Attachment 213757] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Oct 9 08:51:08 PDT 2013


Brady Eidson <beidson at apple.com> has denied Iago Toral <itoral at igalia.com>'s
request for review:
Bug 122142: Release expired pages in the page cache more often
https://bugs.webkit.org/show_bug.cgi?id=122142

Attachment 213757: Patch
https://bugs.webkit.org/attachment.cgi?id=213757&action=review

------- Additional Comments from Brady Eidson <beidson at apple.com>
This is a good idea.

I think relying on canCachePage() is kind of a crutch.	If we want to free
pages when they expire, we should set a timer for when they expire.

This change can be tested with a WK2 API test.


More information about the webkit-reviews mailing list