• Michael Lippautz's avatar
    Revert "Reland "[heap] ArrayBufferTracker: Only consider committed size"" · e1e5f6cf
    Michael Lippautz authored
    This reverts commit 46f9d5a2.
    
    Reason for revert: Aborted compaction pages require separate handling now that we consider byteLength which is a Number.
    
    Original change's description:
    > Reland "[heap] ArrayBufferTracker: Only consider committed size"
    > 
    > This is a reland of 6488c9e5
    > Original change's description:
    > > [heap] ArrayBufferTracker: Only consider committed size
    > > 
    > > - Only consider commited size of ABs.
    > > - Compute freed memory from retained sizes byte length might be a
    > >   HeapNumber and thus prohibited from accessing (as it may be already
    > >   collected).
    > > 
    > > CQ_INCLUDE_TRYBOTS=master.tryserver.v8:v8_linux64_tsan_rel;master.tryserver.v8:v8_linux64_tsan_concurrent_marking_rel_ng;master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
    > > 
    > > Bug: chromium:775896
    > > Change-Id: Ia0bed66afac5e4d5ed58194950a55156e19cec72
    > > Reviewed-on: https://chromium-review.googlesource.com/725722
    > > Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    > > Reviewed-by: Ulan Degenbaev <ulan@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#48699}
    > 
    > Tbr: ulan@chromium.org
    > Bug: chromium:775896
    > Change-Id: Ibbec1ffa8fe90d3668f0fe0c1b8b9997b5fd644e
    > Cq-Include-Trybots: master.tryserver.v8:v8_linux64_tsan_rel;master.tryserver.v8:v8_linux64_tsan_concurrent_marking_rel_ng;master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
    > Reviewed-on: https://chromium-review.googlesource.com/726579
    > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#48707}
    
    TBR=ulan@chromium.org,mlippautz@chromium.org
    
    Change-Id: If678ad73326ceb24e85f3a7bf6350df05991005f
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: chromium:775896
    Cq-Include-Trybots: master.tryserver.v8:v8_linux64_tsan_rel;master.tryserver.v8:v8_linux64_tsan_concurrent_marking_rel_ng;master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
    Reviewed-on: https://chromium-review.googlesource.com/726799Reviewed-by: 's avatarMichael Lippautz <mlippautz@chromium.org>
    Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#48708}
    e1e5f6cf
array-buffer-tracker-inl.h 3.55 KB