• jbroman's avatar
    Handle errors in v8::ValueDeserializer by throwing exceptions. · 5a97e955
    jbroman authored
    This restores the contract that all API methods that return Maybe<T> or
    MaybeLocal<T> always throw an exception when they return nothing.
    
    Since v8::ValueDeserializer::ReadHeader can now throw exceptions, it
    needs a Local<Context> parameter so that it can set up execution state
    (entering the context, etc.). The old method has been marked for
    deprecation, but since this API is experimental I intend to remove it
    as soon as I've removed the use from Blink.
    
    value-serializer-unittest has been updated to expect an exception in
    all decode failure cases.
    
    BUG=chromium:148757,chromium:641964
    
    Review-Url: https://codereview.chromium.org/2308053002
    Cr-Commit-Position: refs/heads/master@{#39188}
    5a97e955
messages.h 42 KB