もずはっく日記(2010年4月)
2010年4月12日
Bug-org 552914 nsEditor::mFlags is never modified by SetFlags()
初回投稿日時: 2010-04-12 14:52:37
カテゴリ: Mozilla Core バグ修正
SNS:
Tweet (list)
エディタ内でフラグが二重に管理されていて、同期できていなかった(というかしていなかった)バグです。
ns*Editorクラスは、実際の編集処理をns*EditRulesクラスに任せています。ns*EditRulesクラスは各ns*Editorクラスのインスタンスの初期化時に行われますが、ここでフラグの内容がコピーされていました。しかし、nsIEditor::SetFlags()が呼ばれてもns*EditorRulesへの反映処理は全くありませんでした。
今回の修正でフラグは全てnsEditorで保持するように一本化し、ns*EditRulesは逐一、nsEditorに問い合わせるように修正しています。
またこのバグでコードを整理中に!flags & kFlagXといったミスを発見しました。もちろんコメントでは!(flags & kFlagX)を意図していました。この手のミスはありがちですし、コーディングルールの桁数制限上、非常に見づらいので全てのフラグの問い合わせがメソッドでできるように修正しています。