Bug 184019

Summary: memoryStatus() is wrong in certain testing scenarios on iOS
Product: WebKit Reporter: Saam Barati <saam>
Component: JavaScriptCoreAssignee: Saam Barati <saam>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, benjamin, fpizlo, ggaren, gskachkov, jfbastien, joepeck, keith_miller, mark.lam, msaboff, rmorisset, rniwa, simon.fraser, ticaiolima, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 184050    
Attachments:
Description Flags
patch mark.lam: review+

Description Saam Barati 2018-03-26 11:51:59 PDT 
phys_footprint is a number that in my experiments remains constant at around ~9MB. I think what we want is "internal + compressed".
Comment 1 Saam Barati 2018-03-26 16:20:33 PDT 
(In reply to Saam Barati from comment #0)
> phys_footprint is a number that in my experiments remains constant at around
> ~9MB. I think what we want is "internal + compressed".

I'm actually not 100% sure of this. Doing some more investigation now.
Comment 2 Saam Barati 2018-03-27 11:48:56 PDT 
<rdar://problem/37959258>
Comment 3 Saam Barati 2018-03-27 11:51:27 PDT 
Created attachment 336602 [details]
patch
Comment 4 Saam Barati 2018-03-27 11:52:26 PDT 
Comment on attachment 336602 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=336602&action=review

> Source/bmalloc/ChangeLog:3
> +        Move back to phys_footprint when it works for our testing scenarios on iOS

Will change bug title to match before landing.
Comment 5 Mark Lam 2018-03-27 11:53:40 PDT 
Comment on attachment 336602 [details]
patch

rs=me
Comment 6 Saam Barati 2018-03-27 13:52:53 PDT 
landed in:
https://trac.webkit.org/changeset/230005/webkit