From 28c7245f64c261546f892006523b62e615c585a4 Mon Sep 17 00:00:00 2001 From: vinoth Date: Mon, 19 Mar 2018 12:00:55 +0200 Subject: [PATCH] Bug 1391823 - Don't run |finish() called multiple times| for dom/security/test/mixedcontentblocker/test_frameNavigation.html. r=ckerschb Reviewers: ckerschb Reviewed By: ckerschb Subscribers: ckerschb Bug #: 1391823 Differential Revision: https://phabricator.services.mozilla.com/D562 --HG-- extra : rebase_source : 0ff9fc78d2ca00c2c347f14167de34e31ded1967 extra : amend_source : f2a350607bb0645e3140b51cc57353a50f7c2261 --- .../mixedcontentblocker/file_frameNavigation_blankTarget.html | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dom/security/test/mixedcontentblocker/file_frameNavigation_blankTarget.html b/dom/security/test/mixedcontentblocker/file_frameNavigation_blankTarget.html index 9d234b42ffcf..e5e234962adc 100644 --- a/dom/security/test/mixedcontentblocker/file_frameNavigation_blankTarget.html +++ b/dom/security/test/mixedcontentblocker/file_frameNavigation_blankTarget.html @@ -17,7 +17,8 @@ https://bugzilla.mozilla.org/show_bug.cgi?id=841850 var observer = { observe: function(subject, topic, data) { - if(topic == "content-document-global-created" && data =="http://example.com") { + //Subject location check added for Bug 1391823 to avoid removeAsyncObserver from being called multiple times + if(topic == "content-document-global-created" && data =="http://example.com" && subject.location.href == "http://example.com/tests/dom/security/test/mixedcontentblocker/file_frameNavigation_innermost.html?blankTarget") { parent.parent.postMessage({"test": "blankTarget", "msg": "opened an http link with target=_blank from a secure page"}, "http://mochi.test:8888"); SpecialPowers.removeAsyncObserver(observer, "content-document-global-created"); }