• jkummerow's avatar
    [defineProperty] Fix non-throwing access check failure · 70b74001
    jkummerow authored
    ReportFailedAccessCheck() sometimes doesn't throw an exception (it's up
    to the embedder). Pretend that OrdinaryDefineOwnProperty() succeeded in
    that case. This is consistent with existing behavior of other methods:
    - JSObject::DefineOwnPropertyIgnoreAttributes
    - JSObject::SetPropertyWithFailedAccessCheck
    - Object::SetPropertyWithAccessor
    
    See also commit f66c3f5c.
    
    BUG=chromium:574217
    LOG=n
    R=neis@chromium.org
    
    Review URL: https://codereview.chromium.org/1580823002
    
    Cr-Commit-Position: refs/heads/master@{#33262}
    70b74001
objects.cc 700 KB