Closed Bug 363290 Opened 13 years ago Closed 13 years ago

Make DeleteRangeTxn scriptable through XPCOM

Categories

(Core :: DOM: Editor, defect)

defect
Not set

Tracking

()

RESOLVED WONTFIX

People

(Reporter: WeirdAl, Assigned: WeirdAl)

Details

Attachments

(2 obsolete files)

editor/libeditor/base has some fairly useful core transactions (DeleteRangeTxn, InsertElementTxn, etc.).  I'd like to expose creating some of these transactions to the world of XPCOM, so that Verbosio can use them in its core JS code without having to reinvent the wheel.

Many of the transaction classes there have most of the five methods nsITransaction.idl defines; a first step would be in making a class QI to nsITransaction.  NS_IMPL_ISUPPORTS, anyone?
Attached patch patch for DeleteRangeTxn (obsolete) — Splinter Review
What do you think, Daniel?
Assignee: nobody → ajvincent
Status: NEW → ASSIGNED
Attachment #248107 - Flags: review?(daniel)
Summary: Make TransactionFactory and friends scriptable through XPCOM → Make DeleteRangeTxn scriptable through XPCOM
Comment on attachment 248107 [details] [diff] [review]
patch for DeleteRangeTxn

Something's changed with the mozilla.org build tree; I can't build this cleanly as it now stands.  Withdrawing review request until I have a new patch.
Attachment #248107 - Flags: review?(daniel)
Attached patch patch for DeleteRangeTxn (obsolete) — Splinter Review
Attachment #248107 - Attachment is obsolete: true
Attachment #250012 - Flags: review?(daniel)
Comment on attachment 250012 [details] [diff] [review]
patch for DeleteRangeTxn

hm, I don't need this anymore.
Attachment #250012 - Attachment is obsolete: true
Attachment #250012 - Flags: review?(daniel)
no longer needed
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.