0ct0pu5/ladybird
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

LibJS: Port create_iterator_result_object() to NonnullGCPtr

Browse source
This commit is contained in:
Linus Groh 2023-04-15 16:26:42 +02:00
parent aff1ec6014
commit d4eaaf905c
Notes: sideshowbarker 2024-07-17 02:23:25 +09:00
4 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
Userland/Libraries/LibJS/Runtime/AsyncFromSyncIteratorPrototype.cpp
View file

@ -53,7 +53,7 @@ static Object* async_from_sync_iterator_continuation(VM& vm, Object& result, Pro
// 8. Let unwrap be a new Abstract Closure with parameters (value) that captures done and performs the following steps when called:
auto unwrap = [done](VM& vm) -> ThrowCompletionOr<Value> {
// a. Return CreateIterResultObject(value, done).
return create_iterator_result_object(vm, vm.argument(0), done);
return create_iterator_result_object(vm, vm.argument(0), done).ptr();
};
// 9. Let onFulfilled be CreateBuiltinFunction(unwrap, 1, "", « »).
@ -117,7 +117,7 @@ JS_DEFINE_NATIVE_FUNCTION(AsyncFromSyncIteratorPrototype::return_)
// 7. If return is undefined, then
if (return_method == nullptr) {
// a. Let iterResult be CreateIterResultObject(value, true).
auto* iter_result = create_iterator_result_object(vm, vm.argument(0), true);
auto iter_result = create_iterator_result_object(vm, vm.argument(0), true);
// b. Perform ! Call(promiseCapability.[[Resolve]], undefined, « iterResult »).
MUST(call(vm, *promise_capability->resolve(), js_undefined(), iter_result));

2
Userland/Libraries/LibJS/Runtime/IteratorOperations.cpp
View file

@ -184,7 +184,7 @@ Completion async_iterator_close(VM& vm, Iterator const& iterator_record, Complet
}
// 7.4.10 CreateIterResultObject ( value, done ), https://tc39.es/ecma262/#sec-createiterresultobject
Object* create_iterator_result_object(VM& vm, Value value, bool done)
NonnullGCPtr<Object> create_iterator_result_object(VM& vm, Value value, bool done)
{
auto& realm = *vm.current_realm();

2
Userland/Libraries/LibJS/Runtime/IteratorOperations.h
View file

@ -29,7 +29,7 @@ ThrowCompletionOr<bool> iterator_complete(VM&, Object& iterator_result);
ThrowCompletionOr<Value> iterator_value(VM&, Object& iterator_result);
Completion iterator_close(VM&, Iterator const&, Completion);
Completion async_iterator_close(VM&, Iterator const&, Completion);
Object* create_iterator_result_object(VM&, Value, bool done);
NonnullGCPtr<Object> create_iterator_result_object(VM&, Value, bool done);
ThrowCompletionOr<MarkedVector<Value>> iterable_to_list(VM&, Value iterable, Optional<Value> method = {});
using IteratorValueCallback = Function<Optional<Completion>(Value)>;

2
Userland/Libraries/LibWeb/XHR/FormDataIterator.cpp
View file

@ -73,7 +73,7 @@ JS::Object* FormDataIterator::next()
if (m_iterator_kind == JS::Object::PropertyKind::Value)
return create_iterator_result_object(vm, entry_value, false);
return create_iterator_result_object(vm, JS::Array::create_from(realm(), { JS::PrimitiveString::create(vm, entry.name), entry_value }), false);
return create_iterator_result_object(vm, JS::Array::create_from(realm(), { JS::PrimitiveString::create(vm, entry.name), entry_value }), false).ptr();
}
}