Closed
Bug 5979
Opened 26 years ago
Closed 26 years ago
unit test reveals error
Categories
(Core :: DOM: Editor, defect, P3)
Tracking
()
VERIFIED
FIXED
M7
People
(Reporter: buster, Assigned: mozeditor)
Details
nsHTMLEditRules::DidDoAction returns an error during InsertBreak.
I suggest something like this:
NS_IMETHODIMP
nsHTMLEditRules::DidDoAction(nsIDOMSelection *aSelection,
nsRulesInfo *aInfo, nsresult aResult)
{
!> nsTextRulesInfo *info = NS_STATIC_CAST(nsTextRulesInfo*, aInfo);
!> if (!info) {return NS_OK;}
!>
!> switch (info->action)
!> {
!> case kInsertBreak:
!> return NS_OK;
!> default:
return nsTextEditRules::DidDoAction(aSelection, aInfo, aResult);
}
!> }
|
Assignee | |
Updated•26 years ago
|
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•26 years ago
|
Target Milestone: M6
|
|
Assignee | |
Comment 1•26 years ago
|
||
setting milestone to M6 - will check in fix once tree opens
|
|
Assignee | |
Updated•26 years ago
|
Target Milestone: M6 → M7
|
|
Assignee | |
Comment 2•26 years ago
|
||
moving non-crashers to M7
|
|
Assignee | |
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 3•26 years ago
|
||
fixed.
Steve, can you verify this works now and mark this bug VERIFIED-FIXED?
thanks!
You need to log in
before you can comment on or make changes to this bug.
Description
•