nsITableEditor::GetFirstRow() is an XPCOM method, so, for internal use, we should create non-virtual method, that is GetFirstTableRowElement(). This patch makes it never return NS_SUCCESS_EDITOR_ELEMENT_NOT_FOUND since nobody refers it and it's detectable. If the method returns nullptr without error, it's the case of NS_SUCCESS_EDITOR_ELEMENT_NOT_FOUND. Additionally, this patch changes the return type of GetFirstRow() from Node to Element since it always return an Element node if not null.

Comment on attachment 9002438 [details] Bug 1484113 - part 0: Create automated tests for nsITableEditor::GetFirstRow() Makoto Kato [:m_kato] has approved the revision.

Comment on attachment 9002439 [details] Bug 1484113 - part 1: Create HTMLEditor::GetFirstTableRowElement() for internal use of nsITableEditor::GetFirstRow() Makoto Kato [:m_kato] has approved the revision.

Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/af33be3aee82 part 0: Create automated tests for nsITableEditor::GetFirstRow() r=m_kato

Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/090edc3b798b part 1: Create HTMLEditor::GetFirstTableRowElement() for internal use of nsITableEditor::GetFirstRow() r=m_kato

https://hg.mozilla.org/mozilla-central/rev/af33be3aee82 https://hg.mozilla.org/mozilla-central/rev/090edc3b798b