search

wakakusakatuya / adblock-plus-japanese-filter

Automatically exported from code.google.com/p/adblock-plus-japanese-filter
0 stars 0 forks source link

汎用フィルタの非表示要素ルールの改善について意見求ム #7

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
例えば、以下のルールを見てみてください

##*[src*="-ad-"]

これは
　１．ページ内の全要素に対し
　２．src 属性を調べ、「-ad-」を含む場合
　３．非表示(CSS の display:none; に相当)にする
という意味です。それぞれについての問題点は
　１．全要素を対象にすると、パフォーマンスへの悪影響��
�懸念される
　２．src 
で指定される画像やスクリプトは通常のフィルタで対処で��
�る（文字列
長が足りなくて警告マーク！が出るが..）
　３．非表示ルール ## 
ではダウンロードをブロックできない

同様のルールは他に７４個もあり、この部分をまずは改善��
�たいところです

Original issue reported on code.google.com by k2ja...@gmail.com on 13 May 2010 at 8:50

GoogleCodeExporter commented 8 years ago 
https://adblockplus.org/forum/viewtopic.php?t=4067
これを参考にどうぞ

以下の発言は気になります
@Wladimir: I did some testing now, and it seems that rules with . and # cause no
slowdown at all (or it's only a VERY small slowdown). It's only the rules with 
* ^
and $ (partial matches) that make things slow.

Original comment by paranoid...@gmail.com on 17 May 2010 at 10:12

GoogleCodeExporter commented 8 years ago 
@paranoidincarnation
参考リンク読みました。「部分一致がスローダウンの原因��
�も納得

*[src*="TEST"]
は何とかパス内で８文字以上の連続文字列を見つけて、通��
�（Element Hiding じゃない）ルー
ルで扱うべきだよね

ここは、一旦 *[src*="TEST"] 
ルール消してみようと思うけど、どうだろう？

Original comment by k2ja...@gmail.com on 17 May 2010 at 12:16

GoogleCodeExporter commented 8 years ago 
いいと思う
部分一致の非表示ルールはどんどん減らすべき

Original comment by paranoid...@gmail.com on 18 May 2010 at 4:31

GoogleCodeExporter commented 8 years ago 
まずはコメントアウトしてみて、１～２週間クレーム等が��
�るかどうか様子を見てみる

今回コメントアウトしてみたのはこのパターン
● *[src*="SAMPLE"] => ! *[src*="TEST"]

Original comment by k2ja...@gmail.com on 18 May 2010 at 11:33

GoogleCodeExporter commented 8 years ago 
１．上記コメントアウトに関して：特にクレームは無かっ��
�ので削除
　
２．次の段階として、##*[class*="AD"] のように class や id 
の部分一致についてはどう対
処すべきだろうか
　a) そのまま
　b) 
数を減らす（"PR_"のような短い部分一致から削っていく）
　c) id や class 
文字列の部分一致を全て消し、具体的な文字列のみに絞る
　
お気軽に意見をどうぞ

Original comment by k2ja...@gmail.com on 1 Jun 2010 at 11:46

GoogleCodeExporter commented 8 years ago 
c)に一票

普通のユーザはここに見に来ないだと思う
やっぱり以前のようにコメントアウトしてから様子見した��
�うがいいじゃない

Original comment by paranoid...@gmail.com on 13 Jun 2010 at 2:41

GoogleCodeExporter commented 8 years ago 
> paranoidincarnation

> 普通のユーザはここに見に来ないだと思う

確かに...
もう少し部分一致の Element Hiding 
ルールを削ってみるつもり（作業は今週末あたりかな）

Original comment by k2ja...@gmail.com on 14 Jun 2010 at 4:21

GoogleCodeExporter commented 8 years ago 
汎用フィルタの非表示要素(Element 
Hiding)ルールで全要素対象（＊）で PR 
文字列にマッチするルールをコメントアウト（パフォーマ��
�ス改善のテスト：数週間様子を見て、反応無しならルール�
��消す）
● ##*[class$="_PR"]（sample）

Original comment by k2ja...@gmail.com on 20 Jun 2010 at 5:26

GoogleCodeExporter commented 8 years ago

Original comment by k2ja...@gmail.com on 6 Jan 2011 at 6:34