LibJS: Change create_iterator_result_object's return type to Object*

This always returns an Object, so there's no reason to return a less
explicit Value.

Userland/Libraries/LibJS/Runtime/IteratorOperations.cpp

@@ -148,7 +148,7 @@ void iterator_close(Object& iterator)
 }
 
 // 7.4.8 CreateIterResultObject ( value, done ), https://tc39.es/ecma262/#sec-createiterresultobject
-Value create_iterator_result_object(GlobalObject& global_object, Value value, bool done)
+Object* create_iterator_result_object(GlobalObject& global_object, Value value, bool done)
 {
     auto& vm = global_object.vm();
     auto* object = Object::create(global_object, global_object.object_prototype());

Userland/Libraries/LibJS/Runtime/IteratorOperations.h

@@ -24,7 +24,7 @@ Object* iterator_step(GlobalObject&, Object& iterator);
 bool iterator_complete(GlobalObject&, Object& iterator_result);
 Value iterator_value(GlobalObject&, Object& iterator_result);
 void iterator_close(Object& iterator);
-Value create_iterator_result_object(GlobalObject&, Value value, bool done);
+Object* create_iterator_result_object(GlobalObject&, Value value, bool done);
 MarkedValueList iterable_to_list(GlobalObject&, Value iterable, Value method = {});
 
 enum class CloseOnAbrupt {