• Vlad Tsyrklevich's avatar
    Refactor unreliable Proxy tests · 7a75d582
    Vlad Tsyrklevich authored
    While working on crrev.com/c/1141045 I caused 3 assertThrows() tests
    under the 'Deeply nested target' tests to fail. The tests for
    defineProperty, isExtensible, and preventExtensions began to fail under
    a couple build configurations because my change modified the stack check
    code such that it no longer inhibited tail call optimization. Under some
    build configurations the methods responsible for causing a stack oveflow
    for those 3 methods were tail call optimized and the tests no longer
    threw an exception.
    
    Other built-in implementations of proxy handler methods could also fail
    in the future due to refactors moving variables off the stack. Change
    the test to ensure v8 doesn't crash but don't rely on stack overflow
    exceptions being thrown for the 'deeply nested target' test.
    
    BUG=chromium:864705
    
    Change-Id: Iefeaa1d5402986c1831d0f259f83025452756387
    Reviewed-on: https://chromium-review.googlesource.com/1159356Reviewed-by: 's avatarGeorg Neis <neis@chromium.org>
    Commit-Queue: Georg Neis <neis@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#54878}
    7a75d582
regress-7716.js 2.01 KB