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

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");
       }