Unreviewed, rolling out r230005.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Mar 2018 23:00:51 +0000 (23:00 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Mar 2018 23:00:51 +0000 (23:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=184115

"it caused a huge regression on iOS" (Requested by saamyjoon
on #webkit).

Reverted changeset:

"memoryStatus() is wrong in certain testing scenarios on iOS"
https://bugs.webkit.org/show_bug.cgi?id=184050
https://trac.webkit.org/changeset/230005

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@230057 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/bmalloc/ChangeLog
Source/bmalloc/bmalloc/AvailableMemory.cpp

index 7e819ff..3fb805e 100644 (file)
@@ -1,3 +1,17 @@
+2018-03-28  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r230005.
+        https://bugs.webkit.org/show_bug.cgi?id=184115
+
+        "it caused a huge regression on iOS" (Requested by saamyjoon
+        on #webkit).
+
+        Reverted changeset:
+
+        "memoryStatus() is wrong in certain testing scenarios on iOS"
+        https://bugs.webkit.org/show_bug.cgi?id=184050
+        https://trac.webkit.org/changeset/230005
+
 2018-03-27  Saam Barati  <sbarati@apple.com>
 
         memoryStatus() is wrong in certain testing scenarios on iOS
index e5ba09f..fcc9b98 100644 (file)
@@ -99,12 +99,8 @@ MemoryStatus memoryStatus()
     mach_msg_type_number_t vmSize = TASK_VM_INFO_COUNT;
     
     size_t memoryFootprint = 0;
-    if (KERN_SUCCESS == task_info(mach_task_self(), TASK_VM_INFO, (task_info_t)(&vmInfo), &vmSize)) {
-        // FIXME: Move back to phys_footprint when it works in all of our
-        // iOS testing infrastructure:
-        // https://bugs.webkit.org/show_bug.cgi?id=184050
-        memoryFootprint = static_cast<size_t>(vmInfo.internal) + static_cast<size_t>(vmInfo.compressed);
-    }
+    if (KERN_SUCCESS == task_info(mach_task_self(), TASK_VM_INFO, (task_info_t)(&vmInfo), &vmSize))
+        memoryFootprint = static_cast<size_t>(vmInfo.phys_footprint);
 
     double percentInUse = static_cast<double>(memoryFootprint) / static_cast<double>(availableMemory());
     double percentAvailableMemoryInUse = std::min(percentInUse, 1.0);