• Michael Lippautz's avatar
    Reland "Reland "[heap] ArrayBufferTracker: Only consider committed size"" · da9691de
    Michael Lippautz authored
    Includes the fix for aborted compaction pages that now require processing 
    with all other ArrayBufferTrackers because the considered length (byteLength)
    may be a HeapNumber allocated on a compacted page.
    
    This is a reland of 46f9d5a2
    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}
    > 
    > 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}
    
    Bug: chromium:775896
    Change-Id: I9b7b2ae865ef6cdb25692abb65108df5c2ecc157
    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/726800Reviewed-by: 's avatarUlan Degenbaev <ulan@chromium.org>
    Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#48753}
    da9691de
array-buffer-tracker.cc 4.23 KB