ver 1.9.0

[nodaguti]

No description provided.

[nodaguti]

Review Comments:

---

1. Synchronous SQL should not be used. The use of synchronous SQL via the storage system leads to severe responsiveness issues, and should be avoided at all costs. Please use asynchronous SQL via Sqlite.jsm (http://mzl.la/sqlite-jsm) or the executeAsync method, or otherwise switch to a simpler database such as JSON files or IndexedDB.
2. This add-on is creating DOM nodes from HTML strings containing potentially unsanitized data, by assigning to innerHTML, jQuery.html, or through similar means. Aside from being inefficient, this is a major security risk. For more information, see https://developer.mozilla.org/en/XUL_School/DOM_Building_and_HTML_Insertion.
3. Long inline script. Please store complex scripts in .js files rather than inline script nodes.
   chrome/content/chaika/support.xhtml
4. Markup parsing error. The markup file has more closing tags than it has opening tags.
   chrome/content/chaika/defaults/skin/Footer.html

[masal64]

これどうしましょうかね。困りましたね

AMOの中の人の立場では、レビューするための人的リソースが不足しているのでいいかげん修正してほしいということなのでしょうけど、SQL の全非同期化は変更箇所が多岐にわたるのでおそらく困難だと思います。
そこまでするぐらいなら、これを機に SQLite を全廃して JSON などに置き換えた方が前向きな変更だと思います。

これは自分の想像なので確かなことは言えないのですが、メジャーバージョンアップで minVersion を増やすたびに目視レビューが要求されているような気がするので、minVersion="38.0" を変更せずに例えば 1.8.2 として AMO に提出すると自動で通らないでしょうか。

開発方針からすると不本意ではあるんですが、もしこれでＯＫなら、これらの問題点が解消される目処が立つまでの暫定措置ということにもできますし。

[nodaguti]

コメントありがとうございます．

AMO の審査に目視レビュー無しのバージョンってあるのでしたっけ？
いづれにしても

Markup parsing error. The markup file has more closing tags than it has opening tags.
chrome/content/chaika/defaults/skin/Footer.html

が警告として出るのが避けられないので，自動レビューで済むようになるのは難しいのではないかと思います．

[nodaguti]

とりあえずマージします．
ただ，AMO 登録はおそらく難しいと思います．