Closed Bug 1484129 Opened 6 years ago Closed 6 years ago

Split HTMLEditor::GetNextSelectedCell() for internal use

Categories

(Core :: DOM: Editor, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: masayuki, Assigned: masayuki)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

      No description provided.
nsITableEditor::GetNextCellElement() is an XPCOM method but it's used internally
a lot.  So, HTMLEditor should implement it with non-virtual method and
internal users should use the non-virtual method.

Therefore, this patch creates HTMLEditor::GetNextSelectedTableCellElement().
Comment on attachment 9003723 [details]
Bug 1484129 - part 1: Create HTMLEditor::GetNextSelectedTableCellElement() for internal use of nsITableEditor::GetNextCellElement()

Makoto Kato [:m_kato] has approved the revision.
Attachment #9003723 - Flags: review+
Comment on attachment 9003721 [details]
Bug 1484129 - part 0: Add automated tests for nsITableEditor::GetNextSelectedCell()

Makoto Kato [:m_kato] has approved the revision.
Attachment #9003721 - Flags: review+
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/bfbee4aff230
part 0: Add automated tests for nsITableEditor::GetNextSelectedCell() r=m_kato
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/9da197304355
part 1: Create HTMLEditor::GetNextSelectedTableCellElement() for internal use of nsITableEditor::GetNextCellElement() r=m_kato
https://hg.mozilla.org/mozilla-central/rev/bfbee4aff230
https://hg.mozilla.org/mozilla-central/rev/9da197304355
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: