• Leszek Swirski's avatar
    Reland "[runtime] Reset clobbered argument in DefineClass" · 85ab0ad7
    Leszek Swirski authored
    This is a reland of 9b5f3985
    
    Reland fixes:
     * Store a Handle instead of a raw pointer in the scope, to make sure
       the saved object stays alive.
    
    Original change's description:
    > [runtime] Reset clobbered argument in DefineClass
    >
    > The caller of DefineClass may not expect its arguments to be mutated, so
    > add an arguments mutation scope which resets the argument clobbered by
    > DefineClass.
    >
    > Bug: chromium:1268738
    > Change-Id: I03e9cd82535ca1f83353012a92e80f822566e64e
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3283077
    > Auto-Submit: Leszek Swirski <leszeks@chromium.org>
    > Commit-Queue: Igor Sheludko <ishell@chromium.org>
    > Reviewed-by: Igor Sheludko <ishell@chromium.org>
    > Cr-Commit-Position: refs/heads/main@{#77921}
    
    Bug: chromium:1268738
    Change-Id: I934ba2063bf2b0e66a3c42f274419ddd178e4b54
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3289146
    Auto-Submit: Leszek Swirski <leszeks@chromium.org>
    Commit-Queue: Igor Sheludko <ishell@chromium.org>
    Reviewed-by: 's avatarIgor Sheludko <ishell@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#77945}
    85ab0ad7
runtime-classes.cc 30 KB