• svenpanne@chromium.org's avatar
    Make the runtime entry for setting/changing accessors "atomic". · 1729e3c0
    svenpanne@chromium.org authored
    Previously, there were 1 or 2 calls to the runtime when accessors were changed
    or set. This doesn't really work well with property attributes, leading to some
    hacks and complicates things even further when trying to share maps in presence
    of accessors. Therefore, the runtime entry now takes the full triple (getter,
    setter, attributes), where the getter and/or the setter can be null in case they
    shouldn't be changed.
    
    For now, we do basically the same on the native side as we did before on the
    JavaScript side, but this will change in future CLs, the current CL is already
    large enough.
    
    Note that object literals with a getter and a setter for the same property still
    do 2 calls, but this is a little bit more tricky to fix and will be handled in a
    separate CL.
    
    Review URL: https://chromiumcodereview.appspot.com/9616016
    
    git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10956 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
    1729e3c0
object-define-property.js 31.5 KB