もずはっく日記(2014年3月)
2014年3月11日
Bug-org 981947 Use PeekMessage() instead of GetMessage() when we try removing found message with PeekMessage(PM_NOREMOVE)
初回投稿日時: 2014年03月11日12時28分37秒
カテゴリ: Mozilla Core Mozilla28 Mozilla29 Mozilla30 Windows バグ修正
SNS:
Tweet (list)
mozilla::widget::NativeKey::RemoveFollowingCharMessage()は、WM_KEYDOWNメッセージのハンドリング中に呼び出され、生成されているWM_CHAR等をキューから取り除くメソッドなのですが、この最中に想定外のことがあると、テキスト入力はおかしくなるものの、バグ報告が来ないことが予想され、また、そうそう起きないであろうケースと踏んで、MOZ_CRASH()を利用して、クラッシュレポートで想定外のバグを調査できるようにしていました。
しかし、このクラッシュが予想外に大量に発生しているということで、対応を続けています。クラッシュレポートでも、有益なコメントが付けられているものはほとんど無いのですが、原因のひとつには、Doragon Naturally Speakingというソフトがあるようです。このアプリは、音声からキー入力イベントを生成しているようですが、どうもその生成が下手なようで、他のアプリでもうまく動かないものがあるというコメントも見られました。
このバグは、オリジナルであるBug-org 962140での取り扱いに限界が出たため、分割登録しなおした最初のバグです。
これまでは、PeekMessage()でWM_*CHARを発見した場合、GetMessage()で見つけたメッセージを取得していましたが、PeekMessage()をPM_REMOVEとセットで呼び出した場合の挙動の違いが事態をややこしくしている可能性がありましたので、PeekMessage()をPM_NOREMOVEを利用して呼び出した後には、必ず、PeekMessage()とPM_REMOVEを利用するようにしました。
また、実際にバグ修正のヒントとなるであろう、メッセージキューの情報を、MOZ_CRASH()を呼び出す直前に取得して、リポートにログを追加するようにしました。