webdino / amethyst

Simple WebViewer App for embedded systems (patches for Firefox/Gecko)
https://gecko-embedded.org/docs/webviewer/
5 stars 1 forks source link

クラッシュレポートを無効化 #15

Closed dynamis closed 5 years ago

dynamis commented 5 years ago

クラッシュレポートは無効化する。単にビルドオプション変えるだけでは実行時設定によって有効化できてしまう可能性があるので注意。applicaiton.ini に記載するだけでは不可で、そもそもクラッシュレポータがビルドされず使われないか、設定を読み取る側に変更が必要。

実行時環境変数だと MOZ_CRASHREPORTER で application.ini 無視して有効化できるが、もちろんこれで有効化できないことを確認する

https://firefox-source-docs.mozilla.org/toolkit/crashreporter/crashreporter/index.html

hATrayflood commented 5 years ago

https://github.com/webdino/amethyst/commit/cd2408e783000f7cbff8908a3974e3e2db8bc05b 対応しました。 ビルドオプションに関係なく常に無効になるようにしました。 また、applicaiton.ini と MOZ_CRASHREPORTER を読み込むコードも削除しました。 @dynamis

kou029w commented 5 years ago

メモ: クラッシュさせて確認

kou029w commented 5 years ago

crashreporter のバイナリが /usr/lib64/webviewer ディレクトリ以下に存在しないことと、実際にクラッシュさせ Mozilla Crash Reporter が起動しないことを確認しました

クラッシュさせる方法:

  1. リモートデバッガでブラウザコンソールに接続
  2. chrome://browser/content/browser.xul のコンテクストで Components.utils.crashIfNotInAutomation() を実行

結果: crashreporter のウィンドウが現れない

備考: https://github.com/webdino/meta-browser/blob/20ae2986334fd70c5b864793077092edcbb9e772/recipes-browser/firefox/firefox/mozconfig#L37 ビルド自体無効にされているが、ビルドオプションに関係なく無効になっている