No. `xblock.children` should be the deserialized values. The next line puts the serialized values into the document which is being constructed for persistence.