Bug 167872

Summary: [GTK][EFL] Release unused UpdateAtlas when in memory pressure situation
Product: WebKit Reporter: Miguel Gomez <magomez>
Component: WebKitGTKAssignee: Miguel Gomez <magomez>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, commit-queue
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch magomez: commit-queue+

Miguel Gomez
Reported 2017-02-06 02:43:50 PST
The current implementation waits 3 seconds before releasing unused UpdateAtlases, in case we can reuse them. This is nice, but if we are in a memory pressure situation we should try to release them as soon as possible to free resources.
Attachments
Patch (3.91 KB, patch)
2017-02-06 02:52 PST, Miguel Gomez 		no flags
DetailsFormatted DiffDiff
Patch (3.91 KB, patch)
2017-02-06 03:30 PST, Miguel Gomez 		magomez: commit-queue+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Miguel Gomez
Comment 1 2017-02-06 02:52:03 PST
Created attachment 300701 [details] Patch
Carlos Garcia Campos
Comment 2 2017-02-06 03:12:33 PST
Comment on attachment 300701 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=300701&action=review > Source/WebKit2/WebProcess/WebPage/CoordinatedGraphics/CompositingCoordinator.h:69 > + enum ReleaseAtlasPolicy { > + ReleaseInactive, > + ReleaseUnused > + }; This should also be private.
Miguel Gomez
Comment 3 2017-02-06 03:30:11 PST
Created attachment 300704 [details] Patch
WebKit Commit Bot
Comment 4 2017-02-06 04:23:34 PST
Comment on attachment 300704 [details] Patch Clearing flags on attachment: 300704 Committed r211721: <http://trac.webkit.org/changeset/211721>
WebKit Commit Bot
Comment 5 2017-02-06 04:23:38 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.