バグ #1207
closed
タイムライン上にマウスカーソルを移動するとクラッシュする
Description
現在のdevelopブランチのmikutterをUbuntuで実行したとき、タイムライン（要するにMiraclePainterが描画している領域）にカーソルを移動させると、確実にクラッシュします。
WSL上のUbuntu（私）と、MacBookAir上の実機（ https://social.mikutter.hachune.net/@d_flat_aug7/99869543045799080 ）での発生を確認しています。
エラーメッセージは以下の通り。
RuntimeError failed to initialize
{MIKUTTER_DIR}/core/mui/cairo_miracle_painter.rb:223:in `initialize'
{MIKUTTER_DIR}/core/mui/cairo_miracle_painter.rb:223:in `new'
{MIKUTTER_DIR}/core/mui/cairo_miracle_painter.rb:223:in `set_cursor'
{MIKUTTER_DIR}/core/lib/uithreadonly.rb:22:in `block (2 levels) in singleton class'
{MIKUTTER_DIR}/core/mui/cairo_miracle_painter.rb:183:in `point_moved'
{MIKUTTER_DIR}/core/lib/uithreadonly.rb:22:in `block (2 levels) in singleton class'
{MIKUTTER_DIR}/core/mui/cairo_cell_renderer_message.rb:184:in `block in event_hooks'
{MIKUTTER_DIR}/core/mui/gtk_extension.rb:35:in `block in safety_signal_connect'
{MIKUTTER_DIR}/core/mui/cairo_cell_renderer_message.rb:73:in `signal_emit'
{MIKUTTER_DIR}/core/mui/cairo_cell_renderer_message.rb:73:in `block in tree='
{MIKUTTER_DIR}/core/mui/gtk_extension.rb:35:in `block in safety_signal_connect'
{MIKUTTER_DIR}/core/plugin/gtk/mainloop.rb:10:in `main'
{MIKUTTER_DIR}/core/plugin/gtk/mainloop.rb:10:in `mainloop'
./mikutter.rb:68:in `boot!'
./mikutter.rb:104:in `<main>'
これ以降は私の環境でのみ確認（他のUbuntu環境では未確認）
breeze-cursor-themeパッケージをインストール
$ sudo apt install breeze-cursor-theme
すると、このエラーは出なくなりますが、低確率で以下のエラーにより落ちます。
TypeError wrong argument type Gdk::DisplayX11 (expected Data)
{MIKUTTER_DIR}/core/mui/cairo_miracle_painter.rb:223:in `initialize'
{MIKUTTER_DIR}/core/mui/cairo_miracle_painter.rb:223:in `new'
{MIKUTTER_DIR}/core/mui/cairo_miracle_painter.rb:223:in `set_cursor'
{MIKUTTER_DIR}/core/lib/uithreadonly.rb:22:in `block (2 levels) in singleton class'
{MIKUTTER_DIR}/core/mui/cairo_miracle_painter.rb:183:in `point_moved'
{MIKUTTER_DIR}/core/lib/uithreadonly.rb:22:in `block (2 levels) in singleton class'
{MIKUTTER_DIR}/core/mui/cairo_cell_renderer_message.rb:184:in `block in event_hooks'
{MIKUTTER_DIR}/core/mui/gtk_extension.rb:35:in `block in safety_signal_connect'
{MIKUTTER_DIR}/core/mui/cairo_cell_renderer_message.rb:73:in `signal_emit'
{MIKUTTER_DIR}/core/mui/cairo_cell_renderer_message.rb:73:in `block in tree='
{MIKUTTER_DIR}/core/mui/gtk_extension.rb:35:in `block in safety_signal_connect'
{MIKUTTER_DIR}/core/plugin/gtk/mainloop.rb:10:in `main'
{MIKUTTER_DIR}/core/plugin/gtk/mainloop.rb:10:in `mainloop'
./mikutter.rb:68:in `boot!'
./mikutter.rb:104:in `<main>'
https://cobodo.hateblo.jp/entry/2018/04/16/143723 に貼ったプラグインでモンキーパッチして、Gdk::Cursorの旧形式コンストラクタを呼ぶように変更すると、今のところ落ちなくなるようです。
Files
再現手順
・developブランチのmikutterを起動
・マウスカーソルをタイムライン上に移動
Updated by toshi_a 初音 about 7 years ago
明言しているわけでわないですが、WSLやMacは環境としてかなり特殊なので、なんとも言えないですね。
とはいえ、前の方法がdeprecatedだったわけでもないようですし、新たに提案された方法はそのチケットで解決したい問題ではなかったので、動いていた環境で動かなくなったなら、その部分は修正したいです。
Updated by cob odo about 7 years ago
書き方が悪かったんですが、「MacBookAir上の実機（ https://social.mikutter.hachune.net/@d_flat_aug7/99869543045799080 ）」はハードウェアとしてのMacBookAirの上で動くUbuntu16.04らしいです。
Updated by toshi_a 初音 about 7 years ago
ああそうだったんですね。だとすると尚更まずいですね。いずれにせよ、対応することには肯定的です。
Updated by Yuto Tokunaga about 7 years ago
- File 1207-1.patch 1207-1.patch added
Ubuntu 16.04で問題を確認しました．Gdkのgdk_cursor_new_from_name関数がGdkCursorを返すかNULLを返すかが環境に依ることが原因のようです．ruby-gtkのnative extensionではNULLが返ってきた場合に適切にハンドリングできておらず，RuntimeErrorが生じるようになっています．解決策として添付したパッチのような方法もあるとは思うのですが，あまりすっきりしないですね．
あと，gtk3-demoというアプリケーションのカーソルのデモでは同じAPIを使って適切にカーソルを設定できているようなのですが，なぜこちらでは上手くいくのにmikutterではできないのかもよく分かりません．
Updated by Yuto Tokunaga about 7 years ago
Ubuntu 18.04でもクラッシュを確認しました．gtk2のライブラリのバージョンに依存すると考え調べてみたのですが，Ubuntu 18.04はlibgtk2.0-0のバージョンが2.24.32なのに対し，正常に動作している手元環境(Arch linux)のgtk2のバージョンが2.24.32なので関係ないことが分かりました．
Updated by Akira Ouchi about 7 years ago
以下のチケットを適当に見た感じ、環境によってカーソルがあったりなかったりするのではないでしょうか。
https://github.com/ruby-gnome2/ruby-gnome2/issues/720
例外をスルーしたらとりあえずコケなくなりました（カーソルは変わらなくなりますが）。
(カーソルが変更できるかのチェックが出来ればそのほうが優しいとは思います)
diff --git a/core/mui/cairo_miracle_painter.rb b/core/mui/cairo_miracle_painter.rb index 8774ef4b..d507d3a6 100644 --- a/core/mui/cairo_miracle_painter.rb +++ b/core/mui/cairo_miracle_painter.rb @@ -223,10 +223,13 @@ class Gdk::MiraclePainter < Gtk::Object # ==== Args # [name] カーソルの名前(String) private def set_cursor(name) - window = @tree.get_ancestor Gtk::Window - display = window.screen.display - window.window.cursor = Gdk::Cursor.new(display, name) - self + begin + window = @tree.get_ancestor Gtk::Window + display = window.screen.display + window.window.cursor = Gdk::Cursor.new(display, name) + self + rescue => e + end end # MiraclePainterが選択解除されたことを通知する
Updated by toshi_a 初音 about 7 years ago
- Assignee set to toshi_a 初音
カーソルがない場合は他のところで既に使っている方法で取得すれば良さそうではありますが、どちらの場合も必ず従来の方法であれば正しく動いているので、そちらの方法で揃えたいと思っています。
Updated by toshi_a 初音 about 7 years ago
修正案の一つとして topic/1207-gdk-cursor をpushしました( 977f77ef )。
一度recommendedな方法にしたのを戻しているので心苦しいですが、最低限しか変えていないので万が一あとから巧いやり方がわかっても実装しやすいように配慮しています。
これでクラッシュせず、マウスカーソルがちゃんと変わっているでしょうか。
Updated by Akira Ouchi about 7 years ago
topic/1207-gdk-cursorをcheckoutしてクラッシュしなくなったことを確認しました。
Updated by cob odo about 7 years ago
topic/1207-gdk-cursor でクラッシュしなくなることは確認しましたが、リンクをmouseoverしてもカーソル形状がXTERMのままで、HAND2になりません。
これは、 topic/1207-gdk-cursor がscoreブランチmerge後のdevelopから生えていて、 Gdk::MiraclePainter#cursor_name_of がmessage.links(=entity)を参照しているためかなと思います。
Updated by cob odo about 7 years ago
↑こう書いておいて何ですが、落ちるよりは遥かに良いので、とりあえずmergeしてしまって、カーソル形状は別チケットで処理した方が良い気がしますね。
Updated by toshi_a 初音 about 7 years ago
- Status changed from 実装待ち to 終了
developにmergeしました。
topic/1207-gdk-cursor でクラッシュしなくなることは確認しましたが、リンクをmouseoverしてもカーソル形状がXTERMのままで、HAND2になりません。
これは、 topic/1207-gdk-cursor がscoreブランチmerge後のdevelopから生えていて、 Gdk::MiraclePainter#cursor_name_of がmessage.links(=entity)を参照しているためかなと思います。
これは直ってると思います