1359393 - HTMLEditRules::IsEmptyInline() returns false when given node is not an inline element

Status: NEW

Description

[Masayuki Nakano [:masayuki] (JST, +0900) (offline: 9/21-9/30)]

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;
> }