Closed
Bug 394349
Opened 17 years ago
Closed 17 years ago
use textbox.editor.transactionManager.clear() instead of textbox.editor.enableUndo(true/false)
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 3 alpha8
People
(Reporter: Gavin, Assigned: sciguyryan)
References
()
Details
(Whiteboard: [good first bug])
Attachments
(1 file)
1.35 KB,
patch
|
Gavin
:
review+
|
Details | Diff | Splinter Review |
See URL. We can just call clear() on the transactionmanager directly instead of toggling undo (that's essentially what enableUndo(false) does).
Reporter | ||
Updated•17 years ago
|
Whiteboard: [good first bug]
Reporter | ||
Updated•17 years ago
|
Keywords: helpwanted
Assignee | ||
Comment 1•17 years ago
|
||
Patch v1.0 Is this what you were looking for Gavin or are there other instances to be fixed also?
Assignee: nobody → sciguyryan
Status: NEW → ASSIGNED
Attachment #279232 -
Flags: review?(gavin.sharp)
Reporter | ||
Comment 2•17 years ago
|
||
Comment on attachment 279232 [details] [diff] [review] Patch v1.0 Nope, this is it. Thanks!
Attachment #279232 -
Flags: review?(gavin.sharp) → review+
Reporter | ||
Updated•17 years ago
|
Keywords: helpwanted → checkin-needed
Comment 3•17 years ago
|
||
Checking in browser/base/content/sanitize.js; /cvsroot/mozilla/browser/base/content/sanitize.js,v <-- sanitize.js new revision: 1.21; previous revision: 1.20 done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3 M8
You need to log in
before you can comment on or make changes to this bug.
Description
•