もずはっく日記(2015年7月)
2015年7月19日
Bug-org 1179093 ContentEventHandler::mSelection should be nsRefPtr<Selection>
初回投稿日時: 2015年07月19日12時07分05秒
カテゴリ: Mozilla Core Mozilla42 バグ修正
SNS:
Tweet (list)
現在、nsISelectionはmozilla::dom::Selectionでしか実装されていません。そのため、多くの内部処理はSelectionクラスを直接利用するようになっていますが、ContentEventHandlerは未だにnsCOMPtr<nsISelection>を利用していたため、コードが無用に複雑なままでした(XPCOMのメソッドで値を取得する場合、値は引数で受け取らないといけないため)。そこで、nsRefPtr<Selection>として保存するようにし、コードをシンプルにしようというバグです。
しかし、nsISelectionとして取得したものを現在の設計を理由にSelectionにキャストしているので、安全とは言い切れない修正にもなっています。nsISelection::AsSelection()のようなメソッドを今後追加すべきかもしれません。