Review Request: khtml adblock filter lookup API

Maksim Orlovich mo85 at cornell.edu
Mon Sep 21 15:53:09 BST 2009


> Dear all,
>
> I've seen it suggested on another ML that here is the place to discuss
> KHTML changes/additions.  If you weren't aware of it already, would it
> be possible for an expert to look at http://reviewboard.kde.org/r/1382
> and give a yes/no whether it is acceptable?
>
> This is an addition to the KHTML API (binary and source compatible) to
> allow the adblock filter matching a URL to be identified - the current
> API provides only a boolean match.  This is so that the matching
> filter can be shown in the adblock dialogue, allowing the filter that
> caused something to be blocked to be identified.  There should be no
> performance impact under normal operation.

Looks fine, though I can't be 100% sure that what it returns in complex
cases is identical to the input.

Sorry for missing your previous e-mail.






More information about the kfm-devel mailing list