mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-09-24 02:08:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibJS: Throw on failed PutValue into an environment reference

Browse source 
This should really be handled at a different layer of the stack, but
this allows us to make progress on the Object rewrite without breaking
strict mode assignment tests.
This commit is contained in:
Andreas Kling 2021-07-02 00:32:23 +02:00
commit a4897ed7b2
Notes: sideshowbarker 2024-07-18 11:08:59 +09:00
1 changed files with 9 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

10
Userland/Libraries/LibJS/Runtime/Reference.cpp
View file

@ -60,7 +60,15 @@ void Reference::put_value(GlobalObject& global_object, Value value)
return;
}
m_base_environment->put_into_environment(m_name.as_string(), variable);
bool succeeded = m_base_environment->put_into_environment(m_name.as_string(), variable);
if (vm.exception())
return;
if (!succeeded && m_strict) {
// FIXME: This is a hack and will disappear when we support proper variable bindings.
vm.throw_exception<TypeError>(global_object, ErrorType::DescWriteNonWritable, m_name.to_value(vm).to_string_without_side_effects());
return;
}
}
void Reference::throw_reference_error(GlobalObject& global_object)