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

Unified Diff: lib/notificationHelper.js

Issue 29772555: Issue 6647 - Stop converting domains from punycode to unicode (Closed)
Patch Set: Created May 6, 2018, 2:42 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
Index: lib/notificationHelper.js
===================================================================
--- a/lib/notificationHelper.js
+++ b/lib/notificationHelper.js
@@ -23,7 +23,6 @@
const {Utils} = require("./utils");
const {Notification: NotificationStorage} =
require("../adblockpluscore/lib/notification");
-const {stringifyURL} = require("./url");
const {initAntiAdblockNotification} =
require("../adblockplusui/lib/antiadblockInit");
const {Prefs} = require("./prefs");
@@ -314,7 +313,7 @@
ext.pages.onLoading.addListener(page =>
{
- NotificationStorage.showNext(stringifyURL(page.url));
+ NotificationStorage.showNext(page.url.href);
});
NotificationStorage.addShowListener(showNotification);

Powered by Google App Engine
This is Rietveld