• Simon Zünd's avatar
    [array] Use 'strict' DeleteProperty in Array#sort · b37f1c0a
    Simon Zünd authored
    This CL changes the generic version of Array#sort to use 'strict'
    DeleteProperty when "moving" holes to the end of the sort range.
    
    This brings V8 not only in line with the proposed Array#sort spec
    change, but also closer to what other engines do. Now all engines
    throw a TypeError when the new test case is run.
    
    R=jgruber@chromium.org, mathias@chromium.org
    
    Bug: v8:8714
    Change-Id: Ic5bcd152ad55fd534c1e9e3218393bfe4a50667e
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1666995
    Commit-Queue: Simon Zünd <szuend@chromium.org>
    Commit-Queue: Mathias Bynens <mathias@chromium.org>
    Auto-Submit: Simon Zünd <szuend@chromium.org>
    Reviewed-by: 's avatarMathias Bynens <mathias@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#62273}
    b37f1c0a
array-sort.js 20 KB