Refactor: libcrmcommon: Skip unnecessary doc in replace_xml_child()

The copy of the update argument doesn't need a doc of its own, so we
don't need copy_xml(). When it replaces child, it becomes part of the
doc that child was originally in.

Ref T185

Signed-off-by: Reid Wahl <nrwahl@protonmail.com>