Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: safari/ext/common.js

Issue 29503587: Issue 5464 - Upgrade to new asynchronous version of abp2blocklist (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set: Patch RegExpFilter.typeMap with WEBRTC Created Aug. 14, 2017, 5:21 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« safari/contentBlocking.js ('K') | « safari/contentBlocking.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: safari/ext/common.js
===================================================================
--- a/safari/ext/common.js
+++ b/safari/ext/common.js
@@ -12,16 +12,23 @@
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
*/
(function()
{
+ window.addEventListener("load", () =>
Manish Jethani 2017/08/14 17:23:49 We have to do this in the load handler because the
Sebastian Noack 2017/08/15 15:25:01 This seems to be the wrong place to do this. If yo
Manish Jethani 2017/08/16 10:23:01 This seems to do the trick. Done.
+ {
+ // Patch RegExpFilter.typeMap with WEBRTC, because abp2blocklist needs it
+ // and our version of core doesn't support it yet. See issue #5464.
+ require("filterClasses").RegExpFilter.typeMap.WEBRTC = 256;
+ });
+
/* Message passing */
var MessageProxy = ext._MessageProxy = function(messageDispatcher)
{
this._messageDispatcher = messageDispatcher;
this._responseCallbacks = Object.create(null);
this._responseCallbackCounter = 0;
};
« safari/contentBlocking.js ('K') | « safari/contentBlocking.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld