• Michael Lippautz's avatar
    Revert "[heap] ArrayBufferTracker: Only consider committed size" · d10b6218
    Michael Lippautz authored
    This reverts commit 6488c9e5.
    
    Reason for revert: wasm grow memory test fails; requires investigation.
    
    https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20shared/builds/20548/steps/Check/logs/grow-memory
    
    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,mlippautz@chromium.org
    
    Change-Id: I0f8b28b876b09f149ff330e532e57cf1871e3961
    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/726440Reviewed-by: 's avatarMichael Lippautz <mlippautz@chromium.org>
    Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#48700}
    d10b6218
array-buffer-tracker.cc 4.09 KB