1359393 - HTMLEditor::NodeIsInlineStatic() returns false when given node is not an inline element

Status: NEW

(Core :: DOM: Editor, defect, P3)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(Away until 6/30 except 24)]

This is really odd behavior if caller checks its result with |!|.

http://searchfox.org/mozilla-central/rev/baf47b352e873d4516d7656186d6d7c7447d3873/editor/libeditor/HTMLEditRules.cpp#7941,7946,7951

> bool
> HTMLEditRules::IsEmptyInline(nsINode& aNode)
> {
>   NS_ENSURE_TRUE(mHTMLEditor, false);
>   RefPtr<HTMLEditor> htmlEditor(mHTMLEditor);
> 
>   if (IsInlineNode(aNode) && htmlEditor->IsContainer(&aNode)) {
>     bool isEmpty = true;
>     htmlEditor->IsEmptyNode(&aNode, &isEmpty);
>     return isEmpty;
>   }
>   return false;
> }