• Georg Neis's avatar
    [modules] Fix mapping of internal status to external status. · e0174631
    Georg Neis authored
    Internally, the module status type has one additional value, namely
    PreInstantiating. I previously mapped this to Instantiating when
    crossing the API boundary but it really should be mapped to
    Uninstantiated. That's because when instantiation fails, typically all
    modules not yet visited will remain in the PreInstantiating state, yet
    they must appear Uninstantiated to the outside.
    
    A relevant test will be added to chromium shortly.
    
    Bug: v8:1569, chromium:748544
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Change-Id: Icb33c7f90db5e62375b6c09d14e3d2d5342b0879
    Reviewed-on: https://chromium-review.googlesource.com/586602Reviewed-by: 's avatarAdam Klein <adamk@chromium.org>
    Commit-Queue: Georg Neis <neis@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#46911}
    e0174631
api.cc 384 KB