Setting content from HTML results in unwanted paragraphs

[jonnyandrew]

Summary

Setting content from HTML results in extra unwanted paragraphs being inserted into the content.

Steps to reproduce

• Initialise the composer with HTML containing paragraphs and newlines, for example:

<p>
  paragraph 1
</p>
<p>
  paragraph 2
</p>

• Get the composer content as HTML.

Expected

Output is the same as the input.

Actual

Output contains an extra paragraph:

<p>
  paragraph 1
</p>
<p>
</p>
<p>
  paragraph 2
</p>

Failing test

#[test]
fn set_content_from_html_paragraphs() {
    let mut model = cm("|");
    model
        .set_content_from_html(&utf16(
            "<p>\nparagraph 1\n</p>\n<p>\nparagraph 2\n</p>",
        ))
        .unwrap();
    assert_eq!(
        tx(&model),
        "<p>\nparagraph 1\n</p>\n<p>\nparagraph 2\n</p>|"
    );
}

Related issues

• Causes Editing multi-line message creates extra new lines element-hq/element-x-android#1259