Bug 133671

Summary: [iOS WebKit2] Stay in "memory pressure" mode until told otherwise.
Product: WebKit Reporter: Andreas Kling <kling>
Component: PlatformAssignee: Andreas Kling <kling>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Andreas Kling
Reported 2014-06-09 19:52:13 PDT
After receiving a memory pressure notification, we should stay on alert until the system tells us bad times are over.
Attachments
Patch (6.08 KB, patch)
2014-06-09 19:53 PDT, Andreas Kling 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2014-06-09 19:53:02 PDT
Created attachment 232764 [details] Patch
Sam Weinig
Comment 2 2014-06-09 22:24:18 PDT
Comment on attachment 232764 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=232764&action=review > Source/WebCore/platform/MemoryPressureHandler.h:111 > + void respondToMemoryPressure(bool critical); This seems like a good place for an enum.
WebKit Commit Bot
Comment 3 2014-06-10 09:02:35 PDT
Comment on attachment 232764 [details] Patch Clearing flags on attachment: 232764 Committed r169746: <http://trac.webkit.org/changeset/169746>
WebKit Commit Bot
Comment 4 2014-06-10 09:02:38 PDT
All reviewed patches have been landed. Closing bug.
Tim Horton
Comment 5 2014-06-10 09:30:48 PDT
Comment on attachment 232764 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=232764&action=review > Source/WebCore/ChangeLog:17 > + Also propagate the "critical" flag all the way from the system > + to the pressure handler. Note that we don't actually do anything > + differently based on the flag yet. yay!
Note You need to log in before you can comment on or make changes to this bug.