Noissue - Add webrtc filter type to filter documentation

pages/filters.html

Index: pages/filters.html
diff --git a/pages/filters.html b/pages/filters.html
index c10c8b5a912dba2419ddda87be520ff5771d97b7..867202821574800934b4fb9b63365e3868326788 100644
--- a/pages/filters.html
+++ b/pages/filters.html
@@ -152,6 +152,7 @@ title=Writing Adblock Plus filters
         <li>{{s64 <code><fix>subdocument</fix></code> — embedded pages, usually included via HTML frames}}</li>
         <li>{{ping <code><fix>ping</fix></code> — requests started by <a href="https://developer.mozilla.org/docs/Web/HTML/Element/a#attr-ping"><code><fix>&lt;a ping&gt;</fix></code></a> or <a href="https://developer.mozilla.org/docs/Web/API/Navigator/sendBeacon"><code><fix>navigator.sendBeacon()</fix></code></a> (Adblock Plus 2.7.1 or higher required)}}</li>
         <li>{{websocket <code><fix>websocket</fix></code> — requests initiated via <a href="https://developer.mozilla.org/docs/Web/API/WebSocket"><code><fix>WebSocket</fix></code> object</a> (Adblock Plus 2.8 or higher required)}}</li>
+        <li>{{webrtc <code><fix>webrtc</fix></code> — requests initiated via <a href="https://developer.mozilla.org/docs/Web/API/RTCPeerConnection"><code><fix>RTCPeerConnection</fix></code> object</a> (Adblock Plus for Chrome and Opera 1.13.3 or higher required, Adblock Plus for Firefox and Safari not yet supported)}}</li>

[Sebastian Noack, 2017/05/22 13:59:37]

While it's probably not worth to add support for the legacy blocking mode on
Safari, I wonder whether connection to WebRTC ICE servers are actually blocked
through Content Blockers? If not done, yet all it would take is mapping $webrtc
filters to "raw". Same for $websocket. Manish, can you check this?

Also, I wonder whether we should specifically mention ICE servers here, as the
$webrtc option is essentially blocking requests to ICE servers, not to the
actual peers.

[kzar, 2017/05/22 14:06:41]

I see what you mean, but equally I'm trying to keep this short and simple. I've
made a small tweak to make it clearer that the connections are being made via a
peer connection instance, what do you think?

[Sebastian Noack, 2017/05/22 14:23:07]

Why not using the correct terminology and talk about ICE servers? In fact one
could argue that the actual peer connection (which is not blocked) is made by
the same instance (after all that's what the API is named like), based on the
peers discovered through the ICE server(s).

[kzar, 2017/05/22 14:26:44]

Do you have a suggestion of how we can do that without making it unclear to
someone who isn't a developer who's familiar with how the WebRTC API works?

[Sebastian Noack, 2017/05/22 14:55:32]

I don't have a particular suggestion, but how is mentioning the
RTCPeerConnection API/class any less technical than the existence of ICE
servers? I probably would keep mentioning both, so if you know about WebRTC but
not about ICE servers, you still know what this is about.

[Manish Jethani, 2017/05/29 13:38:19]

Safari does not support RTCPeerConnection. I just checked WebSocket, and it does
indeed block it.

I think we can map $websocket filters directly.

[Sebastian Noack, 2017/05/29 13:47:06]

So I guess let's map both, $websocket and $webrtc to "raw", in abp2blocklist.
Even if RTCPeerConnection isn't supported yet by Safari, when they add support
it might be blocked by the "raw" type as well.

As for the documentation change here, let's just not mention Safari.

         <li>{{s65 <code><fix>document</fix></code> — the page itself (only <a href="#whitelist">exception rules</a> can be applied to the page)}}</li>
         <li>{{s66 <code><fix>elemhide</fix></code> — for exception rules only, similar to <code><fix>document</fix></code> but only disables <a href="#elemhide">element hiding rules</a> on the page rather than all filter rules (Adblock Plus 1.2 or higher required)}}</li>
         <li>{{generichide <code><fix>generichide</fix></code> — for exception rules only, similar to <code><fix>elemhide</fix></code> but only disables <a href="#generic-specific">generic</a> element hiding rules on the page (Adblock Plus 2.6.12 or higher required)}}</li>